@onehat/data 1.20.2 → 1.20.4

package/package.json

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

package/src/Repository/Ajax.js

@@ -1036,7 +1036,10 @@ class AjaxRepository extends Repository {
 									this.assembleTreeNodes();
 									this.emit('changeData', this.entities);
 								} else if (this.isRemotePhantomMode && (this._operations.add || this._operations.deletePhantom)) {
-									// Do nothing, as we don't want to immediately reload on add for a remote phantom mode record. It won't appear, and it will cause all kinds of trouble!
+									if (this._operations.add) {
+										// Do nothing, as we don't want to immediately reload on add for a remote phantom mode record. 
+										// The entity wouldn't appear, and it would cause all kinds of trouble!
+									}
 									if (this._operations.deletePhantom) {
 										// sweep existing deleted records and remove them
 										_.each(this.entities, (entity) => {

package/src/Repository/OneBuild.js

@@ -262,22 +262,31 @@ class OneBuildRepository extends AjaxRepository {
 	
 	/**
 	 * Integrates with RestTrait::reorder in OneBuild API
-	 * @param {entity} dragRecord - which entity was being dragged
+	 * @param {entity|array} dragRecordOrIds - which entity or ids were being dragged
 	 * @param {entity} dropRecord - which entity it was dropped on to
 	 * @param {string} dropPosition - position in which it was dropped; could be 'before' or 'after'
 	 * @return {Promise}
 	 */
-	reorder = (dragRecord, dropRecord, dropPosition) => {
+	reorder = (dragRecordOrIds, dropRecord, dropPosition) => {
 
 		if (!this.isOnline) {
 			this.throwError('Offline');
 			return;
 		}
-		
+
+		let ids;
+		if (_.isArray(dragRecordOrIds)) {
+			ids = dragRecordOrIds;
+		} else if (dragRecordOrIds?.id) {
+			ids = [dragRecordOrIds.id];
+		} else {
+			throw Error('dragRecordOrIds must be an entity or array of ids')
+		}
+
 		const data = {
 			url: this.name + '/reorder',
 			data: qs.stringify({
-				ids: dragRecord.id,
+				ids,
 				dropPosition,
 				dropRecord_id: dropRecord.id,
 			}),
@@ -340,6 +349,9 @@ class OneBuildRepository extends AjaxRepository {
 
 		
 		const duplicateEntity = await this.createStandaloneEntity(root, true, true);
+		if (entity.isRemotePhantomMode) {
+			entity.isRemotePhantom = true;
+		}
 		this._insertBefore(duplicateEntity, entity);
 
 		this.markLoading(false);

package/src/Repository/Repository.js

@@ -1716,6 +1716,10 @@ export default class Repository extends EventEmitter {
 						break;
 					case 'edit':
 						entities = this.getDirty();
+						if (_.isEmpty(entities) && this.isRemotePhantomMode) {
+							// In remote phantom mode, we need to save phantoms even if they're not dirty
+							entities = this.getPhantom();
+						}
 						if (useStaged) {
 							entities = this.getStaged(entities);
 						}