@azrico/webobject 1.5.44 → 1.5.46

package/dist/src/WebObjects/WebObject.js

@@ -590,7 +590,7 @@ var WebObject = exports.WebObject = /*#__PURE__*/function (_SimpleObject) {
             case 0:
               _parseObj = function _parseObj3() {
                 _parseObj = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime().mark(function _callee12(key, obj) {
-                  var _p, keydot, child_changeableKeys, newobj_dlt;
+                  var _p, _child_changeableKeys, keydot, child_changeableKeys, newobj_dlt;
                   return _regeneratorRuntime().wrap(function _callee12$(_context12) {
                     while (1) switch (_context12.prev = _context12.next) {
                       case 0:
@@ -628,7 +628,7 @@ var WebObject = exports.WebObject = /*#__PURE__*/function (_SimpleObject) {
                         return _context12.abrupt("return", _context12.sent);
                       case 6:
                         if (!(obj instanceof WebObject && typeof obj.get_deltaObject === "function")) {
-                          _context12.next = 17;
+                          _context12.next = 18;
                           break;
                         }
                         /* -------------------- parse changeableKeys for children -------------------- */
@@ -637,24 +637,33 @@ var WebObject = exports.WebObject = /*#__PURE__*/function (_SimpleObject) {
                           return s.startsWith(keydot);
                         }).map(function (r) {
                           return r.substring(keydot.length);
-                        }); //child elements have different kind of delta
-                        _context12.next = 11;
+                        });
+                        /**
+                         * if no key is set for children consider it as undefined
+                         * if we send the empty array to get_delta we get no keys
+                         */
+                        if (child_changeableKeys == null || ((_child_changeableKeys = child_changeableKeys) === null || _child_changeableKeys === void 0 ? void 0 : _child_changeableKeys.length) === 0) {
+                          child_changeableKeys = undefined;
+                        }
+                        /* -------------------------------------------------------------------------- */
+                        //child elements have different kind of delta
+                        _context12.next = 12;
                         return obj.get_deltaObject(_objectSpread(_objectSpread({}, p), {}, {
                           isChild: true,
                           changeableKeys: child_changeableKeys
                         }));
-                      case 11:
+                      case 12:
                         newobj_dlt = _context12.sent;
                         if ((0, _object.object_isEmpty)(newobj_dlt)) {
-                          _context12.next = 16;
+                          _context12.next = 17;
                           break;
                         }
                         return _context12.abrupt("return", newobj_dlt);
-                      case 16:
-                        return _context12.abrupt("return", obj);
                       case 17:
                         return _context12.abrupt("return", obj);
                       case 18:
+                        return _context12.abrupt("return", obj);
+                      case 19:
                       case "end":
                         return _context12.stop();
                     }

package/dist/src/structs/SimpleObject.d.ts

@@ -282,7 +282,7 @@ export declare class SimpleObject extends MapableObject {
      * @param result_object
      * @returns
      */
-    static isResultValid(result_object: any, sub?: boolean): any;
+    static isResultValid(result_object: any, depth?: number): boolean;
     /**
      * set the response of last save from site
      * and its type

package/dist/src/structs/SimpleObject.js

@@ -1479,18 +1479,43 @@ var SimpleObject = exports.SimpleObject = /*#__PURE__*/function (_MapableObject)
   }, {
     key: "isResultValid",
     value: function isResultValid(result_object) {
-      var sub = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
+      var _this4 = this;
+      var depth = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : 0;
       if ((0, _object.object_isEmpty)(result_object)) return false;
       switch (_typeof(result_object)) {
+        case "string":
+          try {
+            result_object = JSON.parse(result_object);
+            return this.isResultValid(result_object);
+          } catch (_unused) {
+            return false;
+          }
+        case "number":
+          return result_object > 0;
+        case "boolean":
+          return result_object === true;
         case "object":
           /* ----------------------------- result is error ---------------------------- */
           if (result_object instanceof Error) {
             return false;
           }
-          /* --------- maybe website sends the result as array with 1 element --------- */
+          /* ----------------- maybe website sends the result as array ---------------- */
           if (Array.isArray(result_object)) {
-            if (sub) return false; //avoid going too deep
-            return this.isResultValid((0, _object.array_first)(result_object), true);
+            if (depth > 0) return false; //avoid going too deep
+            var allResults = result_object.map(function (r) {
+              return Boolean(_this4.isResultValid(r, depth + 1));
+            });
+            return allResults.find(function (s) {
+              return s === false;
+            }) == null;
+          }
+          /* ----------------------------- check products ----------------------------- */
+          if (result_object.products) {
+            /**
+             * if one product is not valid the result is not valid
+             * but if all products are valid we still need to check the object itself
+             */
+            if (!this.isResultValid(result_object.products, depth + 1)) return false;
           }
           /* -------------------------- check if acknowledged ------------------------- */
           if ((0, _object.object_isTrue)(result_object.acknowledged)) {
@@ -1500,17 +1525,6 @@ var SimpleObject = exports.SimpleObject = /*#__PURE__*/function (_MapableObject)
             return (0, _object.object_equals)(result_object.expected, result_object.result);
           }
           return !(0, _string.string_isEmpty)(this.getID(result_object));
-        case "string":
-          try {
-            result_object = JSON.parse(result_object);
-            return this.isResultValid(result_object);
-          } catch (_unused) {
-            return false;
-          }
-        case "number":
-          return result_object > 0;
-        case "boolean":
-          return result_object === true;
       }
       return false;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@azrico/webobject",
-    "version": "1.5.44",
+    "version": "1.5.46",
     "description": "Stateful object",
     "author": "Azrideus",
     "license": "MIT",