fix(findByIds): do not call mongoose for empty _ids array

croeck · nodkz · commit 2e268066fd1d · 2017-11-13T23:06:47.000+06:00
Closes #74
diff --git a/src/resolvers/findByIds.js b/src/resolvers/findByIds.js
@@ -46,7 +46,7 @@ export default function findByIds(
     resolve: (resolveParams: ExtendedResolveParams) => {
       const args = resolveParams.args || {};
 
-      if (!Array.isArray(args._ids)) {
+      if (!Array.isArray(args._ids) || args._ids.length === 0) {
         return Promise.resolve([]);
       }
 

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ export default function findByIds(`
`46`	`46`	`resolve: (resolveParams: ExtendedResolveParams) => {`
`47`	`47`	`const args = resolveParams.args \|\| {};`
`48`	`48`
`49`		`- if (!Array.isArray(args._ids)) {`
	`49`	`+ if (!Array.isArray(args._ids) \|\| args._ids.length === 0) {`
`50`	`50`	`return Promise.resolve([]);`
`51`	`51`	`}`
`52`	`52`