@onehat/data 1.18.7 → 1.18.9

package/cypress/e2e/Repository/Repository.cy.js

@@ -1139,7 +1139,7 @@ describe('Repository Base', function() {
 					parent_id: null,
 					depth: 0,
 					hasChildren: true,
-					isChildrenLoaded: true,
+					areChildrenLoaded: true,
 				},
 				{
 					id: 2,
@@ -1147,21 +1147,21 @@ describe('Repository Base', function() {
 					parent_id: 1,
 					depth: 1,
 					hasChildren: true,
-					isChildrenLoaded: true,
+					areChildrenLoaded: true,
 				},
 				{
 					id: 3,
 					display: 'Child 2',
 					parent_id: 1,
 					depth: 1,
-					isChildrenLoaded: true,
+					areChildrenLoaded: true,
 				},
 				{
 					id: 4,
 					display: 'Grandchild',
 					parent_id: 2,
 					depth: 2,
-					isChildrenLoaded: true,
+					areChildrenLoaded: true,
 				},
 			],
 			creatRepository = async () => {

package/package.json

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

package/src/Entity/Entity.js

@@ -121,21 +121,24 @@ class Entity extends EventEmitter {
 
 		/**
 		 * @member {TreeNode} parent - The parent TreeNode for this TreeNode
+		 * @public
 		 * For trees only
 		 */
 		this.parent = null;
 
 		/**
 		 * @member {array} children - Contains any children of this TreeNode
+		 * @public
 		 * For trees only
 		 */
 		this.children = this._originalData.children && !_.isEmpty(this._originalData.children) ? this._originalData.children : [];
 
 		/**
-		 * @member {boolean} isChildrenLoaded - Whether child TreeNodes have loaded for this TreeNode
+		 * @member {boolean} areChildrenLoaded - Whether child TreeNodes have loaded for this TreeNode
+		 * @public
 		 * For trees only
 		 */
-		this.isChildrenLoaded = this._originalData.isChildrenLoaded || false;
+		this.areChildrenLoaded = this._originalData.areChildrenLoaded || false;
 
 		/**
 		 * @member {boolean} isPersisted - Whether this object has been persisted in a storage medium
@@ -1639,7 +1642,7 @@ class Entity extends EventEmitter {
 		if (this.isDestroyed) {
 			throw Error('this.getChildren is no longer valid. TreeNode has been destroyed.');
 		}
-		if (!this.isChildrenLoaded) {
+		if (!this.areChildrenLoaded) {
 			await this.loadChildren();
 		}
 		return this.children;
@@ -1667,12 +1670,13 @@ class Entity extends EventEmitter {
 		if (this.isDestroyed) {
 			throw Error('this.loadChildren is no longer valid. TreeNode has been destroyed.');
 		}
-		if (!this.repository?.loadChildren) {
-			throw Error('repository.loadChildren is not defined.');	
+		if (!this.repository?.getChildNodes) {
+			throw Error('repository.getChildNodes is not defined.');	
 		}
 
-		this.children = await this.repository.loadChildren(this, depth); // populates the children with a reference to this in child.parent
-		this.isChildrenLoaded = true;
+		const children = await this.repository.getChildNodes(this, depth);
+		this.areChildrenLoaded = true;
+		return children;
 	}
 
 	/**

package/src/Property/Json.js

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

package/src/Repository/OneBuild.js

@@ -106,7 +106,7 @@ class OneBuildRepository extends AjaxRepository {
 		}
 
 		const headers = _.merge({
-							'Content-Type': 'application/json',
+							// 'Content-Type': 'application/json', // Stops axios from using 'application/x-www-form-urlencoded'
 							Accept: 'application/json',
 						}, this.headers);
 
@@ -178,7 +178,11 @@ class OneBuildRepository extends AjaxRepository {
 		});
 
 		if (this.isLoaded && this.isAutoLoad && !this.eventsPaused) {
-			return this.reload();
+			if (this.isTree) {
+				return this.getRootNodes(1);
+			} else {
+				return this.reload();
+			}
 		}
 	}
 
@@ -199,9 +203,15 @@ class OneBuildRepository extends AjaxRepository {
 		
 		if (!this.eventsPaused) {
 			if (this.isLoaded && this.isAutoLoad) {
-				return this.reload().then(() => {
-					this.emit('changeSorters');
-				});
+				if (this.isTree) {
+					return this.getRootNodes(1).then(() => {
+						this.emit('changeSorters');
+					});
+				} else {
+					return this.reload().then(() => {
+						this.emit('changeSorters');
+					});
+				}
 			} else {
 				this.emit('changeSorters');
 			}
@@ -426,7 +436,7 @@ class OneBuildRepository extends AjaxRepository {
 	/**
 	 * Gets the root nodes of this tree.
 	 */
-	getRootNodes = async (getChildren = false, depth, getChildParams) => {
+	getRootNodes = (depth) => {
 		this.ensureTree();
 		if (this.isDestroyed) {
 			this.throwError('this.setRootNode is no longer valid. Repository has been destroyed.');
@@ -436,26 +446,12 @@ class OneBuildRepository extends AjaxRepository {
 			this.throwError('Offline');
 			return;
 		}
+
 		this.emit('beforeLoad'); // TODO: canceling beforeLoad will cancel the load operation
 		this.markLoading();
 
-		
-		const data = {
-			url: this.name + '/getRootNodes',
-			data: qs.stringify({
-				getChildren,
-				depth,
-				conditions: getChildParams ? getChildParams() : null 
-			}),
-			method: 'POST',
-			baseURL: this.api.baseURL,
-		};
-
-		if (this.debugMode) {
-			console.log('getRootNodes', data);
-		}
-
-		return this.axios(data)
+		const data = _.merge({ depth }, this._baseParams, this._params);
+		return this._send('POST', this.name + '/getNodes', data)
 			.then((result) => {
 				if (this.debugMode) {
 					console.log('Response for getRootNodes', result);
@@ -474,6 +470,11 @@ class OneBuildRepository extends AjaxRepository {
 					message
 				} = this._processServerResponse(result);
 
+				if (!success) {
+					this.throwError(message);
+					return;
+				}
+
 				this._destroyEntities();
 
 				// Set the current entities
@@ -491,10 +492,10 @@ class OneBuildRepository extends AjaxRepository {
 
 				this.areRootNodesLoaded = true;
 
-				this.markLoaded();
-
-				this.emit('changeData', this.entities);
+				
+				// Don't emit events for root nodes...
 				this.emit('load', this);
+				// this.emit('changeData', this.entities);
 
 				return this.getBy((entity) => {
 					return entity.isRoot;
@@ -506,75 +507,125 @@ class OneBuildRepository extends AjaxRepository {
 	}
 
 	/**
-	 * Searches all nodes for the supplied text.
-	 * This basically takes the search query and returns whatever the server sends
+	 * Loads the children of the supplied treeNode
 	 */
-	searchTree = async (q) => {
+	getChildNodes = (treeNode, depth = 1) => {
 		this.ensureTree();
 		if (this.isDestroyed) {
-			this.throwError('this.searchTree is no longer valid. Repository has been destroyed.');
+			this.throwError('this.getChildNodes is no longer valid. Repository has been destroyed.');
 			return;
 		}
 		if (!this.isOnline) {
 			this.throwError('Offline');
 			return;
 		}
-		
-		const data = {
-			url: this.name + '/searchTree',
-			data: qs.stringify({
-				q,
-			}),
-			method: 'POST',
-			baseURL: this.api.baseURL,
-		};
 
-		if (this.debugMode) {
-			console.log('searchTree', data);
+		// If children already exist, remove them from the repository
+		// This way, we can reload just a portion of the tree
+		if (!_.isEmpty(treeNode.children)) {
+			const children = treeNode.children;
+			treeNode.children = [];
+
+			_.each(children, (child) => {
+				this.removeNode(child);
+			});
 		}
+		
+		this.markLoading();
 
-		return this.axios(data)
+		const data = _.merge({ depth, parentId: treeNode.id, }, this._baseParams, this._params);
+		return this._send('POST', this.name + '/getNodes', data)
 			.then((result) => {
 				if (this.debugMode) {
-					console.log('searchTree response', result);
+					console.log('Response for getChildNodes', result);
 				}
 
-				const response = result.data;
-				if (!response.success) {
-					this.throwError(response.data);
+				if (this.isDestroyed) {
+					// If this repository gets destroyed before it has a chance
+					// to process the Ajax request, just ignore the response.
 					return;
 				}
 
-				return response.data;
+				const {
+					root,
+					success,
+					total,
+					message
+				} = this._processServerResponse(result);
+
+				if (!success) {
+					this.throwError(message);
+					return;
+				}
+				
+				// Set the current entities
+				const children = _.map(root, (data) => {
+					const entity = Repository._createEntity(this.schema, data, this, true);
+					this._relayEntityEvents(entity);
+					return entity;
+				});
+
+				this.entities = this.entities.concat(children);
+
+				this.assembleTreeNodes();
+				
+				this._setPaginationVars();
+
+				// this.emit('changeData', this.entities);
+				this.emit('load', this);
+
+				return children;
+			})
+			.finally(() => {
+				this.markLoading(false);
 			});
 	}
+
 	/**
-	 * Loads the children of the supplied treeNode
+	 * Searches all nodes for the supplied text.
+	 * This basically takes the search query and returns whatever the server sends
 	 */
-	loadChildren = async (treeNode, depth = 1) => {
+	searchNodes = (q) => {
 		this.ensureTree();
 		if (this.isDestroyed) {
-			this.throwError('this.setRootNode is no longer valid. Repository has been destroyed.');
+			this.throwError('this.searchNodes is no longer valid. Repository has been destroyed.');
 			return;
 		}
-
-
-		// If children already exist, remove them from the repository
-		// This way, we can reload just a portion of the tree
-		if (!_.isEmpty(treeNode.children)) {
-			const children = treeNode.children;
-			treeNode.children = [];
-
-			_.each(children, (child) => {
-				this.removeNode(child);
-			});
+		if (!this.isOnline) {
+			this.throwError('Offline');
+			return;
 		}
 
+		const data = _.merge({ q, }, this._baseParams, this._params);
+		return this._send('POST', this.name + '/searchNodes', data)
+			.then((result) => {
+				if (this.debugMode) {
+					console.log('Response for searchNodes', result);
+				}
+
+				if (this.isDestroyed) {
+					// If this repository gets destroyed before it has a chance
+					// to process the Ajax request, just ignore the response.
+					return;
+				}
 
-		// TODO: load children here
+				const {
+					root,
+					success,
+					total,
+					message
+				} = this._processServerResponse(result);
 
+				if (!success) {
+					this.throwError(message);
+					return;
+				}
 
-		
+				return root;
+			})
+			.finally(() => {
+				this.markLoading(false);
+			});
 	}
 
 	/**

package/src/Repository/Repository.js

@@ -288,10 +288,10 @@ export default class Repository extends EventEmitter {
 		});
 
 		// Auto load & sort
-		if (this.isAutoLoad) {
+		if (this.isAutoLoad && !this.isTree) {
 			await this.load();
 		}
-		if (!this.isSorted && this.isAutoSort && !this.isRemoteSort) { // load may have sorted, in which case this will be skipped.
+		if (!this.isSorted && this.isAutoSort && !this.isRemoteSort && !this.isTree) { // load may have sorted, in which case this will be skipped.
 			await this.sort();
 		}