neo.mjs 4.0.72 → 4.0.73

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "neo.mjs",
-    "version": "4.0.72",
+    "version": "4.0.73",
     "description": "The webworkers driven UI framework",
     "type": "module",
     "repository": {

package/src/DefaultConfig.mjs

@@ -78,6 +78,14 @@ const DefaultConfig = {
      * @type Boolean
      */
     isInsideSiesta: false,
+    /**
+     * delay in ms for the worker.Manager:loadApplication() call
+     * @default 20
+     * @memberOf! module:Neo
+     * @name config.loadApplicationDelay
+     * @type Number
+     */
+    loadApplicationDelay: 20,
     /**
      * Used by Intl.DateTimeFormat, for details take a look at:
      * https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/DateTimeFormat

package/src/collection/Base.mjs

@@ -1099,8 +1099,8 @@ class Base extends CoreBase {
      * If the toRemoveArray is used, then the index is not used for removing, the entries are found by key and removed from where they are.
      * If index is not passed, toAddArray is appended to the Collection.
      * @param {Number|null} index
-     * @param {Number|Array} [removeCountOrToRemoveArray]
-     * @param {Array| Object} [toAddArray]
+     * @param {Number|Object[]} [removeCountOrToRemoveArray]
+     * @param {Object|Object[]} [toAddArray]
      * @returns {Object} An object containing the addedItems & removedItems arrays
      */
     splice(index, removeCountOrToRemoveArray, toAddArray) {

package/src/worker/Base.mjs

@@ -246,10 +246,7 @@ class Base extends CoreBase {
             let message = me.sendMessage(dest, opts, transfer),
                 msgId   = message.id;
 
-            me.promises[msgId] = {
-                resolve,
-                reject
-            };
+            me.promises[msgId] = { reject, resolve };
         });
     }
 

package/src/worker/Manager.mjs

@@ -252,7 +252,7 @@ class Manager extends Base {
         if (me.constructedThreads === me.activeWorkers) {
             NeoConfig.appPath && setTimeout(() => { // better save than sorry => all remotes need to be registered
                 me.loadApplication(NeoConfig.appPath);
-            }, 20);
+            }, NeoConfig.loadApplicationDelay);
         }
     }
 

package/src/worker/mixin/RemoteMethodAccess.mjs

@@ -52,7 +52,6 @@ class RemoteMethodAccess extends Base {
         if (remote.destination === Neo.workerId) {
             let me        = this,
                 className = remote.className,
-                exists    = false,
                 methods   = remote.methods,
                 pkg       = Neo.ns(className, true);
 
@@ -63,15 +62,8 @@ class RemoteMethodAccess extends Base {
 
                 if (!pkg[method] ) {
                     pkg[method] = me.generateRemote(remote, method);
-                } else {
-                    exists = true;
                 }
             });
-
-            // todo: inspect if this can get removed
-            if (!exists && Neo.workerId !== 'main') {
-                me.fire('remoteregistered', remote);
-            }
         }
     }
 

package/test/siesta/tests/CollectionBase.mjs

@@ -251,4 +251,28 @@ StartTest(t => {
         t.isStrict(collection3.getCount(), 5, 'collection3 count is 5');
         t.isStrict(collection3.allItems.getCount(), 9, 'collection3 allItems count is 9');
     });
+
+    t.it('Add & remove at same time', t => {
+        collection = Neo.create(Collection, {
+            items: [
+                {id: 'a'},
+                {id: 'b'},
+                {id: 'c'},
+                {id: 'd'},
+                {id: 'e'},
+                {id: 'f'}
+            ]
+        });
+
+        collection.splice(2, [{id: 'a'}, {id: 'd'}, {id: 'f'}], [{id: 'x'}, {id: 'y'}, {id: 'z'}]);
+
+        t.isDeeplyStrict(collection.getRange(), [
+            {id: 'b'},
+            {id: 'c'},
+            {id: 'x'},
+            {id: 'y'},
+            {id: 'z'},
+            {id: 'e'}
+        ], 'collection.getRange()');
+    });
 });