#issue15 - Observable notifications (#16)

Fix duplicate observable notifications
ErikSchierboom · Jun 4, 2016 · 21055dd · 21055dd
1 parent c0397d6
commit 21055dd
Show file tree

Hide file tree

Showing 3 changed files with 211 additions and 156 deletions.
diff --git a/knockout-paging.js b/knockout-paging.js
@@ -96,7 +96,7 @@
       'default': new DefaultPageGenerator(),
       'sliding': new SlidingPageGenerator()
     }
-  }
+  };
 
   // This extender adds paging functionality to a Knockout observable array.
   // The target must be an observable array, otherwise an error is thrown.
@@ -128,6 +128,7 @@
     target.pageItems = ko.pureComputed(function() {
       return target().slice(target.firstItemOnPage() - 1, target.lastItemOnPage());
     });
+    target.pageItems.extend({ rateLimit: 1 });
 
     target.pageCount = ko.pureComputed(function() {
       if (target.itemCount() <= 0) {

diff --git a/package.json b/package.json
@@ -48,7 +48,9 @@
     "gulp-replace": "^0.5.4",
     "gulp-uglify": "^1.4.2",
     "mocha": "^2.3.3",
-    "mocha-lcov-reporter": "^1.0.0"
+    "mocha-lcov-reporter": "^1.0.0",
+    "sinon": "^1.17.4",
+    "sinon-chai": "^2.8.0"
   },
   "dependencies": {
     "knockout": "^3.2.0"
@@ -63,4 +65,4 @@
       ]
     }
   }
-}
+}