From 2a151461ab7f5703b24e260b4fbde1854d98eac4 Mon Sep 17 00:00:00 2001
From: amercader <amercadero@gmail.com>
Date: Wed, 23 Jan 2013 22:43:27 +0000
Subject: [PATCH 1/3] [README][xs] Micro tweak

---
 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 65851a30..ba90eed4 100755
--- a/README.md
+++ b/README.md
@@ -51,7 +51,7 @@ For larger changes:
 * [Max Ogden](http://maxogden.com/)
 * [John Glover](https://github.com/johnglover)
 * [James Casbon](http://casbon.me/)
-* [Adria Mercader](http://amercader.net/)
+* [Adrià Mercader](http://amercader.net/)
 * [Dominik Moritz](https://github.com/domoritz)
 * [Friedrich Lindenberg](http://pudo.org/)
 * And [many more](https://github.com/okfn/recline/graphs/contributors)

From 2d5291b201f5072d776231d5310e779d71cff354 Mon Sep 17 00:00:00 2001
From: John Glover <j@johnglover.net>
Date: Tue, 5 Feb 2013 17:32:06 +0100
Subject: [PATCH 2/3] Rename x-axis and y-axis in view.graph.js.

Changed to Axis 1 and Axis 2 to avoid confusion
in bar graph (when axes are inverted).
---
 src/view.graph.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/view.graph.js b/src/view.graph.js
index d0371d69..dc640ca0 100644
--- a/src/view.graph.js
+++ b/src/view.graph.js
@@ -320,7 +320,7 @@ my.GraphControls = Backbone.View.extend({
           <option value="columns">Columns</option> \
           </select> \
         </div> \
-        <label>Group Column (x-axis)</label> \
+        <label>Group Column (Axis 1)</label> \
         <div class="input editor-group"> \
           <select> \
           <option value="">Please choose ...</option> \
@@ -344,7 +344,7 @@ my.GraphControls = Backbone.View.extend({
 ',
   templateSeriesEditor: ' \
     <div class="editor-series js-series-{{seriesIndex}}"> \
-      <label>Series <span>{{seriesName}} (y-axis)</span> \
+      <label>Series <span>{{seriesName}} (Axis 2)</span> \
         [<a href="#remove" class="action-remove-series">Remove</a>] \
       </label> \
       <div class="input"> \

From 4f2e58b51428ba9d0746cf9278c0ecc1ca8212f7 Mon Sep 17 00:00:00 2001
From: John Glover <j@johnglover.net>
Date: Wed, 6 Feb 2013 17:51:59 +0100
Subject: [PATCH 3/3] Add horizontal bar chart back to view.flot.js

---
 src/view.flot.js | 69 ++++++++++++++++++++++++++++++++++--------------
 1 file changed, 49 insertions(+), 20 deletions(-)

diff --git a/src/view.flot.js b/src/view.flot.js
index 3a07f98c..d810b723 100644
--- a/src/view.flot.js
+++ b/src/view.flot.js
@@ -12,10 +12,11 @@ this.recline.View = this.recline.View || {};
 // * model: recline.Model.Dataset
 // * state: (optional) configuration hash of form:
 //
-//        { 
+//        {
 //          group: {column name for x-axis},
 //          series: [{column name for series A}, {column name series B}, ... ],
-//          graphType: 'line'
+//          graphType: 'line',
+//          graphOptions: {custom [flot options]}
 //        }
 //
 // NB: should *not* provide an el argument to the view but must let the view
@@ -80,7 +81,7 @@ my.Flot = Backbone.View.extend({
     // There are issues generating a Flot graph if either:
     // * The relevant div that graph attaches to his hidden at the moment of creating the plot -- Flot will complain with
     //   Uncaught Invalid dimensions for plot, width = 0, height = 0
-    // * There is no data for the plot -- either same error or may have issues later with errors like 'non-existent node-value' 
+    // * There is no data for the plot -- either same error or may have issues later with errors like 'non-existent node-value'
     var areWeVisible = !jQuery.expr.filters.hidden(this.el[0]);
     if ((!areWeVisible || this.model.records.length === 0)) {
       this.needToRedraw = true;
@@ -191,7 +192,7 @@ my.Flot = Backbone.View.extend({
 
       return label;
     };
-    
+
     var xaxis = {};
     xaxis.tickFormatter = tickFormatter;
 
@@ -222,7 +223,7 @@ my.Flot = Backbone.View.extend({
     var yaxis = {};
     yaxis.autoscale = true;
     yaxis.autoscaleMargin = 0.02;
-    
+
     var legend = {};
     legend.position = 'ne';
 
@@ -231,8 +232,8 @@ my.Flot = Backbone.View.extend({
     grid.clickable = true;
     grid.borderColor = "#aaaaaa";
     grid.borderWidth = 1;
-    
-    var optionsPerGraphType = { 
+
+    var optionsPerGraphType = {
       lines: {
         legend: legend,
         colors: this.graphColors,
@@ -259,22 +260,43 @@ my.Flot = Backbone.View.extend({
         grid: grid
       },
       bars: {
+        legend: legend,
+        colors: this.graphColors,
+        lines: { show: false },
+        xaxis: yaxis,
+        yaxis: xaxis,
+        grid: grid,
+        bars: {
+          show: true,
+          horizontal: true,
+          shadowSize: 0,
+          align: 'center',
+          barWidth: 0.8
+        }
+      },
+      columns: {
         legend: legend,
         colors: this.graphColors,
         lines: { show: false },
         xaxis: xaxis,
         yaxis: yaxis,
+        grid: grid,
         bars: {
           show: true,
           horizontal: false,
           shadowSize: 0,
           align: 'center',
           barWidth: 0.8
-        },
-        grid: grid
+        }
       }
     };
-    return optionsPerGraphType[typeId];
+
+    if (self.state.get('graphOptions')) {
+      return _.extend(optionsPerGraphType[typeId],
+                      self.state.get('graphOptions'));
+    } else {
+      return optionsPerGraphType[typeId];
+    }
   },
 
   createSeries: function() {
@@ -289,22 +311,29 @@ my.Flot = Backbone.View.extend({
         // time series
         var xtype = xfield.get('type');
         var isDateTime = (xtype === 'date' || xtype === 'date-time' || xtype  === 'time');
-        
+
         if (isDateTime) {
-          // datetime
-          if (self.state.attributes.graphType != 'bars') {
+          if (self.state.attributes.graphType != 'bars' &&
+              self.state.attributes.graphType != 'columns') {
             x = new Date(x).getTime();
           } else {
             x = index;
           }
         } else if (typeof x === 'string') {
-          x = index;
+          x = parseFloat(x);
+          if (isNaN(x)) {
+            x = index;
+          }
         }
 
         var yfield = self.model.fields.get(field);
         var y = doc.getFieldValue(yfield);
-        
-        points.push([x, y]);
+
+        if (self.state.attributes.graphType == 'bars') {
+          points.push([y, x]);
+        } else {
+          points.push([x, y]);
+        }
       });
       series.push({
         data: points,
@@ -329,9 +358,10 @@ my.FlotControls = Backbone.View.extend({
           <option value="lines">Lines</option> \
           <option value="points">Points</option> \
           <option value="bars">Bars</option> \
+          <option value="columns">Columns</option> \
           </select> \
         </div> \
-        <label>X-Axis</label> \
+        <label>Group Column (Axis 1)</label> \
         <div class="input editor-group"> \
           <select> \
           <option value="">Please choose ...</option> \
@@ -344,7 +374,7 @@ my.FlotControls = Backbone.View.extend({
         </div> \
       </div> \
       <div class="editor-buttons"> \
-        <button class="btn editor-add">Add Additional Y-Axis Plot</button> \
+        <button class="btn editor-add">Add Series</button> \
       </div> \
       <div class="editor-buttons editor-submit" comment="hidden temporarily" style="display: none;"> \
         <button class="editor-save">Save</button> \
@@ -355,7 +385,7 @@ my.FlotControls = Backbone.View.extend({
 ',
   templateSeriesEditor: ' \
     <div class="editor-series js-series-{{seriesIndex}}"> \
-      <label>Y-Axis (<span>{{seriesName}})</span> \
+      <label>Series <span>{{seriesName}} (Axis 2)</span> \
         [<a href="#remove" class="action-remove-series">Remove</a>] \
       </label> \
       <div class="input"> \
@@ -470,4 +500,3 @@ my.FlotControls = Backbone.View.extend({
 });
 
 })(jQuery, recline.View);
-