Merge pull request vuejs#1699 from Mat-Moo/dev

yyx990803 · yyx990803 · commit d7ef64363fa3 · 2015-11-04T15:02:28.000-05:00
Add offset facility to limit filter aka mysql limit
diff --git a/src/filters/array-filters.js b/src/filters/array-filters.js
@@ -6,11 +6,13 @@ var toArray = require('../directives/public/for')._postProcess
  * Limit filter for arrays
  *
  * @param {Number} n
+ * @param {Number} offset (Decimal expected)
  */
 
-exports.limitBy = function (arr, n) {
+exports.limitBy = function (arr, n, offset) {
+  offset = offset ? parseInt(offset, 10) : 0
   return typeof n === 'number'
-    ? arr.slice(0, n)
+    ? arr.slice(offset, offset + n)
     : arr
 }
 
diff --git a/test/unit/specs/filters/filters_spec.js b/test/unit/specs/filters/filters_spec.js
@@ -116,6 +116,13 @@ describe('Filters', function () {
     assertArray(res, [1, 2, 3])
     res = filter(arr, -1)
     assertArray(res, [1, 2])
+    // with offsets, note offsets are 0 bound (as expected)
+    res = filter(arr, 1, 1)
+    assertArray(res, [2])
+    res = filter(arr, 2, 1)
+    assertArray(res, [2, 3])
+    res = filter(arr, 1, 2)
+    assertArray(res, [3])
   })
 
   it('filterBy', function () {

Original file line number	Diff line number	Diff line change
`@@ -6,11 +6,13 @@ var toArray = require('../directives/public/for')._postProcess`
`6`	`6`	`* Limit filter for arrays`
`7`	`7`	`*`
`8`	`8`	`* @param {Number} n`
	`9`	`+ * @param {Number} offset (Decimal expected)`
`9`	`10`	`*/`
`10`	`11`
`11`		`-exports.limitBy = function (arr, n) {`
	`12`	`+exports.limitBy = function (arr, n, offset) {`
	`13`	`+ offset = offset ? parseInt(offset, 10) : 0`
`12`	`14`	`return typeof n === 'number'`
`13`		`- ? arr.slice(0, n)`
	`15`	`+ ? arr.slice(offset, offset + n)`
`14`	`16`	`: arr`
`15`	`17`	`}`
`16`	`18`