fix: @feathersjs/express: replace reduce with map (#1429)

feathersjs · Jul 3, 2019 · 44542e9 · 44542e9
1 parent 9585ac0
commit 44542e9
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/packages/express/lib/rest/getHandler.js b/packages/express/lib/rest/getHandler.js
@@ -32,16 +32,16 @@ function getAllowedMethods (service, routes) {
 }
 
 function makeArgsGetter (argsOrder) {
-  return (req, params) => argsOrder.reduce((result, argName) => {
+  return (req, params) => argsOrder.map((argName) => {
     switch (argName) {
       case 'id':
-        return [ ...result, req.params.__feathersId || null ];
+        return req.params.__feathersId || null;
       case 'data':
-        return [ ...result, req.body ];
+        return req.body;
       case 'params':
-        return [ ...result, params ];
+        return params;
     }
-  }, []);
+  });
 }
 
 // A function that returns the middleware for a given method and service