diff options
author | Garren Smith <garren.smith@gmail.com> | 2013-09-10 16:03:12 +0200 |
---|---|---|
committer | Garren Smith <garren.smith@gmail.com> | 2013-09-10 16:03:12 +0200 |
commit | db85b98565ce0bc691fdac04b9b69eeb9287dccc (patch) | |
tree | 922ccb0f5f7b7a568d16d3f55622d043b46a5ef4 | |
parent | d7d91ac38e5155d848e483328a50f2ff7ca4f10d (diff) | |
download | couchdb-db85b98565ce0bc691fdac04b9b69eeb9287dccc.tar.gz |
Fauxton Pagination improvementsindex-pagination
-rw-r--r-- | src/fauxton/app/modules/databases/routes.js | 1 | ||||
-rw-r--r-- | src/fauxton/app/modules/documents/resources.js | 51 | ||||
-rw-r--r-- | src/fauxton/app/modules/documents/routes.js | 13 | ||||
-rw-r--r-- | src/fauxton/app/modules/documents/tests/resourcesSpec.js | 2 | ||||
-rw-r--r-- | src/fauxton/app/modules/documents/views.js | 13 | ||||
-rw-r--r-- | src/fauxton/app/modules/fauxton/paginate.js | 7 | ||||
-rw-r--r-- | src/fauxton/app/templates/documents/all_docs_list.html | 2 | ||||
-rw-r--r-- | src/fauxton/test/core/paginateSpec.js | 21 |
8 files changed, 96 insertions, 14 deletions
diff --git a/src/fauxton/app/modules/databases/routes.js b/src/fauxton/app/modules/databases/routes.js index 001fab6c8..19772642a 100644 --- a/src/fauxton/app/modules/databases/routes.js +++ b/src/fauxton/app/modules/databases/routes.js @@ -64,7 +64,6 @@ function(app, FauxtonAPI, Databases, Views) { databases.fetch().done(function(resp) { FauxtonAPI.when(databases.map(function(database) { - console.log('fetching', database); return database.status.fetch(); })).always(function(resp) { //make this always so that even if a user is not allowed access to a database diff --git a/src/fauxton/app/modules/documents/resources.js b/src/fauxton/app/modules/documents/resources.js index 065489e7a..911a65d01 100644 --- a/src/fauxton/app/modules/documents/resources.js +++ b/src/fauxton/app/modules/documents/resources.js @@ -253,6 +253,7 @@ function(app, FauxtonAPI) { initialize: function(_models, options) { this.database = options.database; this.params = options.params; + this.skipFirstItem = false; }, url: function(context) { @@ -274,7 +275,9 @@ function(app, FauxtonAPI) { this.params.startkey_docid = '"' + lastId + '"'; this.params.startkey = '"' + lastId + '"'; - this.params.limit = num; + // when paginating forward, fetch 21 and don't show + // the first item as it was the last item in the previous list + this.params.limit = num + 1; return this.url('app'); }, @@ -287,6 +290,7 @@ function(app, FauxtonAPI) { delete this.params.startkey; delete this.params.startkey_docid; } + return this.url('app'); }, @@ -298,14 +302,34 @@ function(app, FauxtonAPI) { return this.viewMeta.update_seq || false; }, + recordStart: function () { + if (this.viewMeta.offset === 0) { + return 1; + } + + if (this.skipFirstItem) { + return this.viewMeta.offset + 2; + } + + return this.viewMeta.offset + 1; + }, + parse: function(resp) { - that = this; + var rows = resp.rows; + this.viewMeta = { total_rows: resp.total_rows, offset: resp.offset, update_seq: resp.update_seq }; - return _.map(resp.rows, function(row) { + + //Paginating, don't show first item as it was the last + //item in the previous page + if (this.skipFirstItem) { + console.log('skipping'); + rows = rows.splice(1); + } + return _.map(rows, function(row) { return { _id: row.id, _rev: row.value.rev, @@ -326,6 +350,7 @@ function(app, FauxtonAPI) { this.idxType = "_view"; this.view = options.view; this.design = options.design.replace('_design/',''); + this.skipFirstItem = false; }, url: function(context) { @@ -366,6 +391,18 @@ function(app, FauxtonAPI) { return this.url('app'); }, + recordStart: function () { + if (this.viewMeta.offset === 0) { + return 1; + } + + if (this.skipFirstItem) { + return this.viewMeta.offset + 2; + } + + return this.viewMeta.offset + 1; + }, + totalRows: function() { return this.viewMeta.total_rows || "unknown"; }, @@ -375,15 +412,21 @@ function(app, FauxtonAPI) { }, parse: function(resp) { + var rows = resp.rows; this.endTime = new Date().getTime(); this.requestDuration = (this.endTime - this.startTime); + if (this.skipFirstItem) { + rows = rows.splice(1); + console.log('skip'); + } + this.viewMeta = { total_rows: resp.total_rows, offset: resp.offset, update_seq: resp.update_seq }; - return _.map(resp.rows, function(row) { + return _.map(rows, function(row) { return { value: row.value, key: row.key, diff --git a/src/fauxton/app/modules/documents/routes.js b/src/fauxton/app/modules/documents/routes.js index 730b4bfde..0edf18d08 100644 --- a/src/fauxton/app/modules/documents/routes.js +++ b/src/fauxton/app/modules/documents/routes.js @@ -159,7 +159,8 @@ function(app, FauxtonAPI, Documents, Databases) { events: { "route:updateAllDocs": "updateAllDocsFromView", "route:updatePreviewDocs": "updateAllDocsFromPreview", - "route:reloadDesignDocs": "reloadDesignDocs" + "route:reloadDesignDocs": "reloadDesignDocs", + "route:paginate": "paginate" }, initialize: function (route, masterLayout, options) { @@ -328,6 +329,16 @@ function(app, FauxtonAPI, Documents, Databases) { })); }, + paginate: function (direction) { + _.extend(this.documentsView.collection.params, app.getParams()); + this.documentsView.forceRender(); + if (direction === 'next') { + this.documentsView.collection.skipFirstItem = true; + } else { + this.documentsView.collection.skipFirstItem = false; + } + }, + reloadDesignDocs: function (event) { this.sidebar.forceRender(); diff --git a/src/fauxton/app/modules/documents/tests/resourcesSpec.js b/src/fauxton/app/modules/documents/tests/resourcesSpec.js index 35bbdb367..e78a3c391 100644 --- a/src/fauxton/app/modules/documents/tests/resourcesSpec.js +++ b/src/fauxton/app/modules/documents/tests/resourcesSpec.js @@ -68,7 +68,7 @@ define([ it('Should return urlNext', function () { var url = collection.urlNextPage(20); - assert.equal(url, 'database/databaseId/_all_docs?limit=20&startkey_docid=%22myId2%22&startkey=%22myId2%22'); + assert.equal(url, 'database/databaseId/_all_docs?limit=21&startkey_docid=%22myId2%22&startkey=%22myId2%22'); }); diff --git a/src/fauxton/app/modules/documents/views.js b/src/fauxton/app/modules/documents/views.js index 08ea6767c..921e0f2f8 100644 --- a/src/fauxton/app/modules/documents/views.js +++ b/src/fauxton/app/modules/documents/views.js @@ -442,7 +442,7 @@ function(app, FauxtonAPI, Paginate, Documents, pouchdb, Codemirror, JSHint, resi establish: function() { if (this.newView) { return null; } - return this.collection.fetch().fail(function() { + return this.collection.fetch({reset: true}).fail(function() { // TODO: handle error requests that slip through // This should just throw a notification, not break the page console.log("ERROR: ", arguments); @@ -455,17 +455,22 @@ function(app, FauxtonAPI, Paginate, Documents, pouchdb, Codemirror, JSHint, resi serialize: function() { var totalRows = 0, - updateSeq = false; + recordStart = 0, + updateSeq = false; if (!this.newView) { totalRows = this.collection.totalRows(); updateSeq = this.collection.updateSeq(); } + recordStart = this.collection.recordStart(); + console.log('record', recordStart); + var info = { updateSeq: updateSeq, + offset: recordStart, totalRows: totalRows, - numModels: this.collection.models.length, + numModels: this.collection.models.length + recordStart - 1, viewList: this.viewList, requestDuration: null }; @@ -530,7 +535,7 @@ function(app, FauxtonAPI, Paginate, Documents, pouchdb, Codemirror, JSHint, resi }, canShowNextfn: function () { - if ((collection.viewMeta.offset + 1) === collection.viewMeta.total_rows) { + if ((collection.viewMeta.offset + collection.length + 2) >= collection.viewMeta.total_rows) { return false; } diff --git a/src/fauxton/app/modules/fauxton/paginate.js b/src/fauxton/app/modules/fauxton/paginate.js index 6dbaa6f91..506b8789f 100644 --- a/src/fauxton/app/modules/fauxton/paginate.js +++ b/src/fauxton/app/modules/fauxton/paginate.js @@ -67,13 +67,16 @@ function(app, FauxtonAPI) { previousClicked: function (event) { event.preventDefault(); - FauxtonAPI.navigate(this.previousUrlfn()); + FauxtonAPI.navigate(this.previousUrlfn(), {trigger: false}); + FauxtonAPI.triggerRouteEvent('paginate', 'previous'); }, nextClicked: function (event) { event.preventDefault(); this.previousIds.push(this.collection.first().id); - FauxtonAPI.navigate(this.nextUrlfn()); + console.log(this.previousIds); + FauxtonAPI.navigate(this.nextUrlfn(), {trigger: false}); + FauxtonAPI.triggerRouteEvent('paginate', 'next'); }, serialize: function () { diff --git a/src/fauxton/app/templates/documents/all_docs_list.html b/src/fauxton/app/templates/documents/all_docs_list.html index f60257551..e5ab8ac05 100644 --- a/src/fauxton/app/templates/documents/all_docs_list.html +++ b/src/fauxton/app/templates/documents/all_docs_list.html @@ -28,7 +28,7 @@ the License. </div> <% } %> <p> - Showing 1-<%= numModels %> of <%= totalRows %> rows + Showing <%=offset%> - <%= numModels %> of <%= totalRows %> rows <% if (updateSeq) { %> -- Update Sequence: <%= updateSeq %> <% } %> diff --git a/src/fauxton/test/core/paginateSpec.js b/src/fauxton/test/core/paginateSpec.js index 413924856..981380e27 100644 --- a/src/fauxton/test/core/paginateSpec.js +++ b/src/fauxton/test/core/paginateSpec.js @@ -55,6 +55,10 @@ define([ }); describe('#next', function () { + beforeEach(function () { + //do this so it doesn't throw an error on other unwired up components + FauxtonAPI.triggerRouteEvent = function () {}; + }); it('Should navigate', function () { var navigateMock = sinon.spy(FauxtonAPI, 'navigate'); @@ -65,6 +69,15 @@ define([ FauxtonAPI.navigate.restore(); }); + it('Should trigger routeEvent', function () { + var navigateMock = sinon.spy(FauxtonAPI, 'triggerRouteEvent'); + + paginate.$('a#next').click(); + + assert.ok(navigateMock.calledOnce); + FauxtonAPI.triggerRouteEvent.restore(); + }); + }); @@ -79,6 +92,14 @@ define([ FauxtonAPI.navigate.restore(); }); + it('Should trigger routeEvent', function () { + var navigateMock = sinon.spy(FauxtonAPI, 'triggerRouteEvent'); + + paginate.$('a#previous').click(); + + assert.ok(navigateMock.calledOnce); + FauxtonAPI.triggerRouteEvent.restore(); + }); }); |