fix triggering OnAdd on existing key. colyseus/schema#197

Author: endel

haxelib.json

@@ -4,9 +4,9 @@
   "license": "MIT",
   "tags": ["multiplayer", "networking", "websockets", "netcode"],
   "description": "Colyseus Multiplayer SDK",
-  "version": "0.16.6",
+  "version": "0.16.7",
   "classPath": "src/",
-  "releasenote": "onAuth() fix forwarding built-in auth token to server",
+  "releasenote": "schema callbacks: fix triggering onAdd on existing key",
   "contributors": ["endel"],
   "dependencies": {
     "colyseus-websocket": "1.0.14",

src/io/colyseus/serializer/schema/Callbacks.hx

@@ -247,7 +247,7 @@ class SchemaCallbacks<T> {
                         triggerCallbacks2(callbacks, cast OPERATION.ADD, change.value, (change.dynamicIndex == null) ? change.field : change.dynamicIndex);
                     }
 
-                } else if ((change.op & cast OPERATION.ADD) == OPERATION.ADD && change.previousValue == null) {
+                } else if ((change.op & cast OPERATION.ADD) == OPERATION.ADD && change.previousValue != change.value) {
                     // triger onAdd
                     triggerCallbacks2(callbacks, cast OPERATION.ADD, change.value, (change.dynamicIndex == null) ? change.field : change.dynamicIndex);
                 }