Merge pull request #173 from saplingjs/fix/skip

groenroos · web-flow · commit e8b0a1c80f23 · 2021-04-04T19:45:20.000+01:00
Separate skip from limit
diff --git a/lib/Storage.js b/lib/Storage.js
@@ -499,10 +499,10 @@ module.exports = class Storage {
 
 		/* Parse limit options */
 		if ('limit' in request.query) {
-			const limit = request.query.limit.split(',');
-			options.limit = Number(limit[1]) || Number(limit[0]);
-			if (limit.length === 2) {
-				options.skip = Number(limit[0]);
+			options.limit = Number(request.query.limit) || null;
+
+			if ('skip' in request.query) {
+				options.skip = Number(request.query.skip) || null;
 			}
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -499,10 +499,10 @@ module.exports = class Storage {`
`499`	`499`
`500`	`500`	`/* Parse limit options */`
`501`	`501`	`if ('limit' in request.query) {`
`502`		`- const limit = request.query.limit.split(',');`
`503`		`- options.limit = Number(limit[1]) \|\| Number(limit[0]);`
`504`		`- if (limit.length === 2) {`
`505`		`- options.skip = Number(limit[0]);`
	`502`	`+ options.limit = Number(request.query.limit) \|\| null;`
	`503`	`+`
	`504`	`+ if ('skip' in request.query) {`
	`505`	`+ options.skip = Number(request.query.skip) \|\| null;`
`506`	`506`	`}`
`507`	`507`	`}`
`508`	`508`