not-node 6.3.10 → 6.3.11

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "not-node",
-	"version": "6.3.10",
+	"version": "6.3.11",
 	"description": "node complimentary part for client side notFramework.",
 	"main": "index.js",
 	"scripts": {

package/src/auth/rules.js

@@ -81,9 +81,9 @@ function roleRequireAuthState(requiredAuth, userAuth) {
  * @return {boolean}
  */
 function compareAuthStatus(rule, auth) {
-    if (objHas(rule, "auth")) {
+    if (objHas(rule, "auth") && typeof rule.auth !== "undefined") {
         return roleRequireAuthState(rule.auth, auth);
-    } else if (objHas(rule, "user")) {
+    } else if (objHas(rule, "user") && typeof rule.user !== "undefined") {
         return roleRequireAuthState(rule.user, auth);
     } else {
         return true;
@@ -94,7 +94,7 @@ function compareAuthStatus(rule, auth) {
  *	Check rule against presented credentials
  *	@param	{import('../types').notRouteRule}		        rule	        action rule
  *	@param  {boolean}		                                auth	        user state of auth
- *	@param  {string|Array<string>}	                                role	user state of role
+ *	@param  {Array<string>}	                        role	        user state of role
  *	@param  {boolean}		                                root            user state of root
  *	@return {boolean}		                                                pass or not
  */

package/src/manifest/result.filter.js

@@ -71,11 +71,22 @@ module.exports = class notManifestRouteResultFilter {
         if (!filteringTarget) {
             return;
         }
-        this.filterByRule(
-            filteringTarget,
-            filteringRule,
-            this.getFilteringStrictMode(notRouteData)
-        );
+        if (Array.isArray(filteringTarget)) {
+            filteringTarget.forEach((filteringTargetItem) => {
+                this.filterByRule(
+                    filteringTargetItem,
+                    filteringRule,
+                    this.getFilteringStrictMode(notRouteData)
+                );
+            });
+        } else {
+            this.filterByRule(
+                filteringTarget,
+                filteringRule,
+                this.getFilteringStrictMode(notRouteData)
+            );
+        }
+
         return result;
     }
 
@@ -104,7 +115,7 @@ module.exports = class notManifestRouteResultFilter {
      * if pathToProperty targets array in target, each item of array will be filtered by
      * @param {object} target
      * @param {object} rule         map of properties.
-     * @param {boolean} strict      filtering mode
+     * @param {boolean} [strict = DEFAULT_STRICT_MODE]      filtering mode
      * example:
      * {
      *      'user': ['username', 'id', 'email'], //filtering properties of object target.user
@@ -126,7 +137,7 @@ module.exports = class notManifestRouteResultFilter {
     static filterByMapRule(target, rule, strict = DEFAULT_STRICT_MODE) {
         //to form ['id', 'user.username', 'files']
         const filteringArray = Object.keys(rule);
-        //filtering direct chilren if in strict mode
+        //filtering direct children if in strict mode
         if (strict) {
             this.filterStrict(target, filteringArray);
         }
@@ -212,7 +223,7 @@ module.exports = class notManifestRouteResultFilter {
      * @returns {string}  path of sub object in notPath notation
      */
     static getFilteringTargetPath(notRouteData) {
-        let path = notPath.PATH_START_OBJECT;
+        let path = `${notPath.PATH_START_OBJECT}`;
         if (notRouteData) {
             if (objHas(notRouteData.rule, PROP_NAME_RETURN_ROOT)) {
                 path += notRouteData.rule[PROP_NAME_RETURN_ROOT];

package/test/result.filter.js

@@ -35,6 +35,23 @@ describe("notManifestRouteResultFilter", function () {
             notManifestRouteResultFilter.filter(notRouteData, reqRes);
             expect(reqRes).to.be.deep.equal({ some: "data" });
         });
+
+        it("array of objects", function () {
+            const reqRes = [
+                { id: 1, _id: 2 },
+                { id: 11, _id: 12 },
+                { id: 21, _id: 22 },
+            ];
+            const notRouteData = {
+                rule: { return: ["id"] },
+            };
+            notManifestRouteResultFilter.filter(notRouteData, reqRes);
+            expect(reqRes).to.be.deep.equal([
+                { id: 1 },
+                { id: 11 },
+                { id: 21 },
+            ]);
+        });
     });
 
     describe("filterByRule", function () {