@onehat/data 1.6.10 → 1.7.0

package/cypress/integration/Repository/Repository.spec.js

@@ -630,6 +630,15 @@ describe('Repository Base', function() {
 			expect(_.isEqual(entity, dirty[0])).to.be.true;
 		});
 
+		it('isDirty', function() {
+			expect(this.repository.isDirty).to.be.false;
+
+			this.repository.setAutoSave(false);
+			const entity = this.repository.getByIx(0);
+			entity.value = 'test';	
+			expect(this.repository.isDirty).to.be.true;
+		});
+
 		it('getDeleted', function() {
 			this.repository.setAutoSave(false);
 			const entity = this.repository.getByIx(0);
@@ -722,6 +731,7 @@ describe('Repository Base', function() {
 		// undeleteByRange
 		// undeleteBy
 		// undeleteById
+		// undeleteDeleted
 
 	});
 

package/package.json

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

package/src/Entity.js

@@ -257,15 +257,17 @@ class Entity extends EventEmitter {
 	 * This is mainly for updating Entity with new data 
 	 * from remote storage medium.
 	 * Assumes (and sets) isPersisted === true.
+	 * Assumes (and sets) isTempId === false.
 	 * @param {array} originalData - Raw data to load into entity.
 	 */
 	loadOriginalData = (originalData) => {
 		if (this.isDestroyed) {
 			throw Error('this.loadOriginalData is no longer valid. Entity has been destroyed.');
 		}
-		this._originalData = originalData || {};
 		this.isPersisted = true;
+		this._originalData = originalData || {};
 		this.reset();
+		this.getIdProperty().isTempId = false;
 	}
 
 	/**

package/src/Property/Integer.js

@@ -57,7 +57,6 @@ export default class IntegerProperty extends Property {
 		}
 		return id;
 	}
-
 	
 };
 

package/src/Repository/Ajax.js

@@ -28,10 +28,13 @@ class AjaxRepository extends Repository {
 			 * @member {object} api - List of relative URIs to API endpoints.
 			 */
 			api: {
-				add: null,
 				get: null,
+				add: null,
 				edit: null,
 				delete: null,
+				batchAdd: null,
+				batchEdit: null,
+				batchDelete: null,
 				baseURL: '', // e.g. 'https://example.com/myapp/'
 			},
 
@@ -39,8 +42,8 @@ class AjaxRepository extends Repository {
 			 * @member {object} methods - List of methods for all four CRUD operations
 			 */
 			methods: {
-				add: 'POST',
 				get: 'GET',
+				add: 'POST',
 				edit: 'POST',
 				delete: 'POST',
 			},
@@ -377,6 +380,12 @@ class AjaxRepository extends Repository {
 						if (this.debugMode) {
 							console.log('load result ' + this.name, result);
 						}
+
+						if (this.isDestroyed) {
+							// If this repository gets destroyed before it has a chance
+							// to process the Ajax request, just ignore the response.
+							return;
+						}
 						
 						const {
 							root,
@@ -492,7 +501,7 @@ class AjaxRepository extends Repository {
 	 * @returns {promise} - Axios Promise.
 	 * @private
 	 */
-	_doAdd(entity) {
+	_doAdd(entity) { // standard function notation
 		if (!this.api.add) {
 			throw new Error('No "add" api endpoint defined.');
 		}
@@ -522,13 +531,55 @@ class AjaxRepository extends Repository {
 						entity.loadOriginalData(root[0]);
 					});
 	}
+	
+	/**
+	 * Helper for save.
+	 * Add multiple entities to storage medium
+	 * @param {array} entities - Entities
+	 * @returns {promise} - Axios Promise.
+	 * @private
+	 */
+	_doBatchAdd(entities) { // standard function notation
+		if (!this.api.batchAdd) {
+			throw new Error('No "batchAdd" api endpoint defined.');
+		}
+
+		this._operations.add = true;
+
+		const method = this.methods.add,
+			url = this.api.batchAdd,
+			data = _.map(entities, entity => entity.submitValues);
+
+		return this._send(method, url, data)
+					.then(result => {
+						if (this.debugMode) {
+							console.log(this.api.batchAdd + ' result', result);
+						}
+						const {
+							root,
+							success,
+							total,
+							message
+						} = this._processServerResponse(result);
+
+						if (!success) {
+							throw new Error(message);
+						}
+
+						// Reload each entity with new data
+						// TODO: Check this
+						_.each(entities, (entity, ix) => {
+							entity.loadOriginalData(root[ix]);
+						});
+					});
+	}
 
 	/**
 	 * Helper for save.
 	 * @returns {promise} - Axios Promise.
 	 * @private
 	 */
-	_doEdit(entity) {
+	_doEdit(entity) { // standard function notation
 		if (!this.api.edit) {
 			throw new Error('No "edit" api endpoint defined.');
 		}
@@ -559,12 +610,54 @@ class AjaxRepository extends Repository {
 					});
 	}
 
+	/**
+	 * Helper for save.
+	 * Edit multiple entities in storage medium
+	 * @param {array} entities - Entities
+	 * @returns {promise} - Axios Promise.
+	 * @private
+	 */
+	_doBatchEdit(entities) { // standard function notation
+		if (!this.api.batchEdit) {
+			throw new Error('No "batchEdit" api endpoint defined.');
+		}
+
+		this._operations.edit = true;
+
+		const method = this.methods.edit,
+			url = this.api.batchEdit,
+			data = _.map(entities, entity => entity.submitValues);
+
+		return this._send(method, url, data)
+					.then(result => {
+						if (this.debugMode) {
+							console.log(this.api.batchEdit + ' result', result);
+						}
+						const {
+							root,
+							success,
+							total,
+							message
+						} = this._processServerResponse(result);
+
+						if (!success) {
+							throw new Error(message);
+						}
+
+						// Reload each entity with new data
+						// TODO: Check this
+						_.each(entities, (entity, ix) => {
+							entity.loadOriginalData(root[ix]);
+						});
+					});
+	}
+
 	/**
 	 * Helper for save.
 	 * @returns {promise} - Axios Promise.
 	 * @private
 	 */
-	_doDelete(entity) {
+	_doDelete(entity) { // standard function notation
 		if (!this.api.delete) {
 			throw new Error('No "delete" api endpoint defined.');
 		}
@@ -598,9 +691,65 @@ class AjaxRepository extends Repository {
 					});
 	}
 
+	/**
+	 * Helper for save.
+	 * Delete multiple entities from storage medium
+	 * @param {array} entities - Entities
+	 * @returns {promise} - Axios Promise.
+	 * @private
+	 */
+	_doBatchDelete(entities) { // standard function notation
+		if (!this.api.batchDelete) {
+			throw new Error('No "batchDelete" api endpoint defined.');
+		}
+
+		this._operations.delete = true;
+
+		const method = this.methods.delete,
+			url = this.api.batchDelete,
+			ids = _.map(entities, entity => entity.id),
+			data = { ids, };
+
+		return this._send(method, url, data)
+					.then(result => {
+						if (this.debugMode) {
+							console.log(this.api.batchDelete + ' result', result);
+						}
+						const {
+							root,
+							success,
+							total,
+							message
+						} = this._processServerResponse(result);
+						
+						if (!success) {
+							throw new Error(message);
+						}
+
+						// Delete it from this.entities
+						this.entities = _.filter(this.entities, (entity) => {
+							const deleteIt = ids.includes(entity.id);
+							if (deleteIt) {
+								entity.destroy();
+							}
+							return !deleteIt;
+						});
+					});
+	}
+
+	/**
+	 * Helper for save.
+	 * Tells repository to delete entity without ever having saved it 
+	 * to storage medium
+	 * @private
+	 */
 	_doDeleteNonPersisted(entity) {
 		this.entities = _.filter(this.entities, (item) => {
-			return item !== entity;
+			const match = item === entity;
+			if (match) {
+				entity.destroy();
+			}
+			return !match;
 		});
 
 		return true;
@@ -681,19 +830,19 @@ class AjaxRepository extends Repository {
 	 */
 	_finalizeSave = (promises) => {
 		return this.axios.all(promises)
-			.then(this.axios.spread((...batchOperationResults) => {
-				// All requests are now complete
-
-				this.isSaving = false;
-				this.emit('save', batchOperationResults);
-
-				// Do we need to reload?
-				if (this._operations.add || this._operations.delete) {
-					this.reload();
-				} else {
-					this.emit('changeData', this.entities);
-				}
-			}));
+						.then(this.axios.spread((...batchOperationResults) => {
+							// All requests are now complete
+
+							this.isSaving = false;
+							this.emit('save', batchOperationResults);
+
+							// Do we need to reload?
+							if (this._operations.add || this._operations.delete) {
+								this.reload();
+							} else {
+								this.emit('changeData', this.entities);
+							}
+						}));
 	}
 
 }

package/src/Repository/OneBuild.js

@@ -35,10 +35,13 @@ class OneBuildRepository extends AjaxRepository {
 			autoSave: true,
 
 			api: {
-				add: this.name + '/extAdd',
 				get: this.name + '/get',
-				edit: this.name + '/extEdit',
-				delete: this.name + '/extDelete',
+				add: this.name + '/add',
+				edit: this.name + '/edit',
+				delete: this.name + '/delete',
+				batchAdd: this.name + '/batchAdd',
+				batchEdit: this.name + '/batchEdit',
+				batchDelete: this.name + '/batchDelete',
 			},
 
 			methods: {
@@ -51,6 +54,8 @@ class OneBuildRepository extends AjaxRepository {
 			messageProperty: 'message',
 
 			allowsMultiSort: true,
+			// batchAsSynchronous: true, // Add directly to schema for now
+			// combineBatch: true,
 
 			// writer: {
 			// 	type: 'json',

package/src/Repository/Repository.js

@@ -1181,6 +1181,18 @@ export default class Repository extends EventEmitter {
 		return !_.isNil(this.getById(idOrEntity));
 	}
 
+	/**
+	 * Getter of isDirty for this Repository.
+	 * Returns true if any Entities within it are dirty
+	 * @return {boolean} isDirty
+	 */
+	get isDirty() {
+		if (this.isDestroyed) {
+			throw Error('this.isDirty is no longer valid. Repository has been destroyed.');
+		}
+		return !!this.getDirty().length;
+	}
+
 	/**
 	 * Convenience function
 	 * Alias for isInRepository
@@ -1231,23 +1243,26 @@ export default class Repository extends EventEmitter {
 
 			const batchOrder = this.batchOrder.split(',');
 
-			let n;
+			let n,
+				i,
+				entity,
+				entities,
+				operation,
+				result;
 			for (n = 0; n < batchOrder.length; n++) {
-				const operation = batchOrder[n];
-				let entities;
+				operation = batchOrder[n];
 				switch(operation) {
 					case 'add':
 						entities = this.getNonPersisted();
 						if (this.combineBatch) {
 
-							// TODO: Implement combined batch processing
-							throw new Error('Combined batch processing not yet implemented');
-
+								result = this.batchAsSynchronous ? await this._doBatchAdd(entities) : this._doBatchAdd(entities);
+								results.push(result);
+		
 						} else {
 							if (_.size(entities) > 0) {
-								let i;
 								for (i = 0; i < entities.length; i++) {
-									const entity = entities[i];
+									entity = entities[i];
 	
 									if (entity.isDeleted) {
 										// This entity is new, but it's also marked for deletion
@@ -1255,7 +1270,7 @@ export default class Repository extends EventEmitter {
 										continue;
 									}
 	
-									const result = this.batchAsSynchronous ? await this._doAdd(entity) : this._doAdd(entity);
+									result = this.batchAsSynchronous ? await this._doAdd(entity) : this._doAdd(entity);
 									results.push(result);
 								}
 							}
@@ -1267,14 +1282,13 @@ export default class Repository extends EventEmitter {
 						entities = this.getDirty();
 						if (this.combineBatch) {
 
-							// TODO: Implement combined batch processing
-							throw new Error('Combined batch processing not yet implemented');
+							result = this.batchAsSynchronous ? await this._doBatchEdit(entities) : this._doBatchEdit(entities);
+							results.push(result);
 
 						} else {
 							if (_.size(entities) > 0) {
-								let i;
 								for (i = 0; i < entities.length; i++) {
-									const entity = entities[i];
+									entity = entities[i];
 
 									if (entity.isDeleted) {
 										// This entity is new, but it's also marked for deletion
@@ -1282,7 +1296,7 @@ export default class Repository extends EventEmitter {
 										continue;
 									}
 
-									const result = this.batchAsSynchronous ? await this._doEdit(entity) : this._doEdit(entity);
+									result = this.batchAsSynchronous ? await this._doEdit(entity) : this._doEdit(entity);
 									results.push(result);
 								}
 							}
@@ -1292,16 +1306,14 @@ export default class Repository extends EventEmitter {
 						entities = this.getDeleted();
 						if (this.combineBatch) {
 
-							// TODO: Implement combined batch processing
-							throw new Error('Combined batch processing not yet implemented');
+								result = this.batchAsSynchronous ? await this._doBatchDelete(entities) : this._doBatchDelete(entities);
+								results.push(result);
 
 						} else {
 							if (_.size(entities) > 0) {
-								let i;
 								for (i = 0; i < entities.length; i++) {
-									const entity = entities[i];
+									entity = entities[i];
 
-									let result;
 									if (!entity.isPersisted) {
 										result = this.batchAsSynchronous ? await this._doDeleteNonPersisted(entity) : this._doDeleteNonPersisted(entity);
 									} else {
@@ -1319,8 +1331,20 @@ export default class Repository extends EventEmitter {
 		return await this._finalizeSave(results);
 	}
 
+	
+	/**
+	 * Helper for save.
+	 * Add multiple entities to storage medium
+	 * @param {array} entities - Entities
+	 * @private
+	 * @abstract
+	 */
+	_doBatchAdd(entities) { // standard function notation
+		throw new Error('_doBatchAdd must be implemented by Repository subclass');
+	}
+
 	/**
-	 * Helper for save().
+	 * Helper for save.
 	 * Add entity to storage medium
 	 * @param {object} entity - Entity
 	 * @private
@@ -1331,7 +1355,18 @@ export default class Repository extends EventEmitter {
 	}
 
 	/**
-	 * Helper for save().
+	 * Helper for save.
+	 * Edit multiple entities in storage medium
+	 * @param {array} entities - Entities
+	 * @private
+	 * @abstract
+	 */
+	_doBatchEdit(entities) { // standard function notation
+		throw new Error('_doBatchEdit must be implemented by Repository subclass');
+	}
+
+	/**
+	 * Helper for save.
 	 * Mark entity as saved
 	 * @param {object} entity - Entity
 	 * @private
@@ -1342,7 +1377,18 @@ export default class Repository extends EventEmitter {
 	}
 
 	/**
-	 * Helper for save().
+	 * Helper for save.
+	 * Delete multiple entities from storage medium
+	 * @param {array} entities - Entities
+	 * @private
+	 * @abstract
+	 */
+	_doBatchDelete(entities) { // standard function notation
+		throw new Error('_doBatchDelete must be implemented by Repository subclass');
+	}
+
+	/**
+	 * Helper for save.
 	 * Delete entity from storage medium
 	 * @param {object} entity - Entity
 	 * @private
@@ -1353,8 +1399,8 @@ export default class Repository extends EventEmitter {
 	}
 
 	/**
-	 * Helper for save().
-	 * Tells storage medium to delete entity without ever having saved it 
+	 * Helper for save.
+	 * Tells repository to delete entity without ever having saved it 
 	 * to storage medium
 	 * @private
 	 */
@@ -1479,7 +1525,7 @@ export default class Repository extends EventEmitter {
 	}
 
 	/**
-	 * Deletes a single Entity by its index (zero-indexed) on the current page
+	 * Undelete a single Entity by its index (zero-indexed) on the current page
 	 * @param {integer} ix - Index
 	 * @return {object} entity - Entity
 	 */
@@ -1488,7 +1534,7 @@ export default class Repository extends EventEmitter {
 	}
 	
 	/**
-	 * Deletes multiple Entities by their range of indices 
+	 * Undelete multiple Entities by their range of indices 
 	 * (zero-indexed) on the current page
 	 * @param {integer} startIx - Index
 	 * @param {integer} endIx - Index (inclusive)
@@ -1499,7 +1545,7 @@ export default class Repository extends EventEmitter {
 	}
 
 	/**
-	 * Remove multiple Entities by supplied filter function
+	 * Undelete multiple Entities by supplied filter function
 	 * @param {function} fn - Filter function to apply to all entities
 	 * @return {Entity[]} Entities that passed through filter
 	 */
@@ -1508,7 +1554,7 @@ export default class Repository extends EventEmitter {
 	}
 
 	/**
-	 * Remove a single Entity by its id
+	 * Undelete a single Entity by its id
 	 * @param {integer} id - id of record to retrieve
 	 * @return {Entity} The Entity with matching id
 	 */
@@ -1516,6 +1562,14 @@ export default class Repository extends EventEmitter {
 		await this.undelete(this.getById(id));
 	}
 
+	/**
+	 * Undelete all deleted Entities
+	 * @return {Entity[]} Entities that passed through filter
+	 */
+	undeleteDeleted = async () => {
+		await this.undelete(this.getDeleted());
+	}
+