@onehat/data 1.22.4 → 1.22.6

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@onehat/data",
-	"version": "1.22.4",
+	"version": "1.22.6",
 	"description": "JS data modeling package with adapters for many storage mediums.",
 	"main": "src/index.js",
 	"type": "module",

package/src/Property/Json.js

@@ -126,7 +126,7 @@ JsonProperty.className = 'Json';
 JsonProperty.type = 'json';
 
 
-// For the sake of OneBuild, create an alias of Json, that's Tag
+// For the sake of OneBuild backwards compatibility, create an alias of Json, that's Tag
 export class TagProperty extends JsonProperty {}
 TagProperty.className = 'Tag';
 TagProperty.type = 'tag';

package/src/Repository/OneBuild.js

@@ -354,7 +354,8 @@ class OneBuildRepository extends AjaxRepository {
 
 		if (!success) {
 			this.markLoading(false);
-			throw Error(message);
+			this.throwError(message);
+			return;
 		}
 
 		// Click duplicateId. The new row appears directly above the one that was duplicated

package/src/Repository/Tree.js

@@ -186,26 +186,31 @@ class TreeRepository extends OneBuildRepository {
 					this.throwError(message);
 					return;
 				}
+
 				
 				// Set the current entities
-				const oThis = this;
-				const children = _.map(root, (data) => {
-					const entity = Repository._createEntity(oThis.schema, data, this, true);
-					oThis._relayEntityEvents(entity);
-					return entity;
+				const children = [];
+				_.each(root, (data) => {
+					if (data.id === treeNode.id) {
+						// This is the node we're loading, so update it directly
+						treeNode.loadOriginalData(data);
+						return null;
+					}
+					const entity = Repository._createEntity(this.schema, data, this, true);
+					this._relayEntityEvents(entity);
+					children.push(entity);
 				});
-
-				this.entities = this.entities.concat(children);
-
-				this.assembleTreeNodes();
-				
-				this._setPaginationVars();
+				if (children.length) {
+					this.entities = this.entities.concat(children);
+					this.assembleTreeNodes();
+					this._setPaginationVars();
+				}
 
 				this.rehash();
 				// this.emit('changeData', this.entities);
 				this.emit('load', this);
 
-				return children;
+				return treeNode;
 			})
 			.finally(() => {
 				this.markLoading(false);