From 9f78cdb8fb986998fadccdccc70ecd966b9b66ab Mon Sep 17 00:00:00 2001 From: Theodore Brown Date: Mon, 10 Mar 2014 10:46:08 -0500 Subject: [PATCH 1/2] Added option to not reset page when adding a filter Also, the page should only reset if the filter is not already applied --- jquery.dynatable.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/jquery.dynatable.js b/jquery.dynatable.js index d08aba3..727b68f 100644 --- a/jquery.dynatable.js +++ b/jquery.dynatable.js @@ -1121,9 +1121,9 @@ } }; - this.add = function(name, value) { - // reset to first page since query will change records - if (settings.features.paginate) { + this.add = function(name, value, noPageReset) { + // only reset the page if noPageReset is not enabled and the filter is not already applied + if (settings.features.paginate && !noPageReset && (!settings.dataset.queries[name] || settings.dataset.queries[name] != value)) { settings.dataset.page = 1; } settings.dataset.queries[name] = value; From c21620fe62738c2afc8379c15c5369c0f2e7ae07 Mon Sep 17 00:00:00 2001 From: Theodore Brown Date: Sun, 22 Nov 2015 11:32:55 -0600 Subject: [PATCH 2/2] Rename parameter to skipPageReset and use a strict equality check --- jquery.dynatable.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/jquery.dynatable.js b/jquery.dynatable.js index 727b68f..51b37dd 100644 --- a/jquery.dynatable.js +++ b/jquery.dynatable.js @@ -1121,9 +1121,9 @@ } }; - this.add = function(name, value, noPageReset) { - // only reset the page if noPageReset is not enabled and the filter is not already applied - if (settings.features.paginate && !noPageReset && (!settings.dataset.queries[name] || settings.dataset.queries[name] != value)) { + this.add = function(name, value, skipPageReset) { + // only reset the page if skipPageReset is not enabled and the filter is not already applied + if (settings.features.paginate && !skipPageReset && (!settings.dataset.queries[name] || settings.dataset.queries[name] !== value)) { settings.dataset.page = 1; } settings.dataset.queries[name] = value;