From 8b9c76fd29a12a69a0a600d423ac92e2f91724af Mon Sep 17 00:00:00 2001
From: rgrp <rufus.pollock@okfn.org>
Date: Thu, 15 Dec 2011 16:13:43 +0000
Subject: [PATCH] [view/data-explorer][m]: (refs #14) introduce ability to
 change number of rows viewed.

* This also necessitated a nice refactor whereby DataExplorer subviews running off a common Backbone Collection (DocumentList) re-rendering themselves in response to changes in that Collection.
---
 demo/style/style.css | 10 +++++++++
 src/model.js         |  9 ++++++--
 src/view.js          | 51 ++++++++++++++++++++++++++++++--------------
 3 files changed, 52 insertions(+), 18 deletions(-)

diff --git a/demo/style/style.css b/demo/style/style.css
index 5e073b3a..40c295f4 100755
--- a/demo/style/style.css
+++ b/demo/style/style.css
@@ -239,3 +239,13 @@ span.tooltip-status {
   right: 0px;
   background: white;
 }
+
+.nav-pagination {
+  display: inline;
+  float: right;
+  list-style-type: none;
+}
+
+.nav-pagination input {
+  width: 40px;
+}
diff --git a/src/model.js b/src/model.js
index 7a783da7..0e73a5d6 100644
--- a/src/model.js
+++ b/src/model.js
@@ -8,6 +8,10 @@ var my = {};
 // A Dataset model.
 my.Dataset = Backbone.Model.extend({
   __type__: 'Dataset',
+  initialize: function() {
+    this.currentDocuments = new my.DocumentList();
+  },
+
   getLength: function() { 
     return this.rowCount;
   },
@@ -19,13 +23,14 @@ my.Dataset = Backbone.Model.extend({
   // this does not fit very well with Backbone setup. Backbone really expects you to know the ids of objects your are fetching (which you do in classic RESTful ajax-y world). But this paradigm does not fill well with data set up we have here.
   // This also illustrates the limitations of separating the Dataset and the Backend
   getRows: function(numRows, start) {
+    var self = this;
     var dfd = $.Deferred();
     this.backend.getRows(this.id, numRows, start).then(function(rows) {
       var docs = _.map(rows, function(row) {
         return new my.Document(row);
       });
-      var docList = new my.DocumentList(docs);
-      dfd.resolve(docList);
+      self.currentDocuments.reset(docs);
+      dfd.resolve(self.currentDocuments);
     });
     return dfd.promise();
   }
diff --git a/src/view.js b/src/view.js
index e1913d6f..0b2b9ac7 100644
--- a/src/view.js
+++ b/src/view.js
@@ -16,19 +16,38 @@ my.DataExplorer = Backbone.View.extend({
         <input type="radio" id="nav-graph" name="nav-toggle" value="graph" /> \
         <label for="nav-graph">Graph</label> \
       </span> \
+      <ul class="nav-pagination"> \
+        <li><form class="display-count"><label for="per-page">Display count</label> <input name="displayCount" type="text" value="{{displayCount}}" /></form></li> \
+      </ul> \
     </div> \
     <div class="data-view-container"></div> \
   ',
 
   events: {
-    'change input[name="nav-toggle"]': 'navChange'
+    'change input[name="nav-toggle"]': 'navChange',
+    'submit form.display-count': 'displayCountUpdate'
   },
 
-  initialize: function() {
+  initialize: function(options) {
     this.el = $(this.el);
+    this.config = options.config || {};
+    _.extend(this.config, {
+      displayCount: 10
+    });
+    this.draw();
+  },
+
+  displayCountUpdate: function(e) {
+    e.preventDefault();
+    this.config.displayCount = parseInt(this.el.find('input[name="displayCount"]').val());
+    this.draw();
+  },
+
+  draw: function() {
+    var self = this;
+    this.el.empty();
     this.render();
     this.$dataViewContainer = this.el.find('.data-view-container');
-    var self = this;
     // retrieve basic data like headers etc
     // note this.model and dataset returned are the same
     this.model.fetch().then(function(dataset) {
@@ -42,11 +61,13 @@ my.DataExplorer = Backbone.View.extend({
       self.flotGraph.el.hide();
       self.$dataViewContainer.append(self.dataTable.el)
       self.$dataViewContainer.append(self.flotGraph.el);
+      self.model.getRows(self.config.displayCount);
     });
   },
 
   render: function() {
-    $(this.el).html($(this.template));
+    var template = $.mustache(this.template, this.config);
+    $(this.el).html(template);
   },
 
   navChange: function(e) {
@@ -75,12 +96,11 @@ my.DataTable = Backbone.View.extend({
   className: "data-table-container",
 
   initialize: function() {
-    this.el = $(this.el);
     var self = this;
-    this.model.getRows().then(function(documentList) {
-      self._currentDocuments = documentList;
-      self.render()
-    });
+    this.el = $(this.el);
+    _.bindAll(this, 'render');
+    this.model.currentDocuments.bind('add', this.render);
+    this.model.currentDocuments.bind('reset', this.render);
     this.state = {};
     // this is nasty. Due to fact that .menu element is not inside this view but is elsewhere in DOM
     $('.menu li a').live('click', function(e) {
@@ -235,7 +255,7 @@ my.DataTable = Backbone.View.extend({
       , htmls = $.mustache(template, this.toTemplateJSON())
       ;
     this.el.html(htmls);
-    this._currentDocuments.forEach(function(doc) {
+    this.model.currentDocuments.forEach(function(doc) {
       var tr = $('<tr />');
       self.el.find('tbody').append(tr);
       var newView = new my.DataTableRow({
@@ -342,18 +362,17 @@ my.FlotGraph = Backbone.View.extend({
 ',
 
   initialize: function(options, chart) {
+    var self = this;
     this.el = $(this.el);
+    _.bindAll(this, 'render');
+    this.model.currentDocuments.bind('add', this.render);
+    this.model.currentDocuments.bind('reset', this.render);
     this.chart = chart;
     this.chartConfig = {
       group: null,
       series: [],
       graphType: 'line'
     };
-    var self = this;
-    this.model.getRows().then(function(documentList) {
-      self._currentDocuments = documentList;
-      self.render()
-    });
   },
 
   events: {
@@ -412,7 +431,7 @@ my.FlotGraph = Backbone.View.extend({
     if (this.chartConfig) {
       $.each(this.chartConfig.series, function (seriesIndex, field) {
         var points = [];
-        $.each(self._currentDocuments.models, function (index, doc) {
+        $.each(self.model.currentDocuments.models, function (index, doc) {
           var x = doc.get(self.chartConfig.group);
           var y = doc.get(field);
           if (typeof x === 'string') {