@onehat/data 1.18.3 → 1.18.4

package/package.json

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

package/src/Repository/Ajax.js

@@ -798,7 +798,7 @@ class AjaxRepository extends Repository {
 
 						// Delete it from this.entities
 						const id = entity.id;
-						this.entities = _.filter(this.entities, (entity) => entity.id === id);
+						this.entities = _.filter(this.entities, (entity) => entity.id !== id);
 						entity.destroy();
 					});
 	}
@@ -974,6 +974,14 @@ class AjaxRepository extends Repository {
 							if (!this.eventsPaused) {
 								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.deletePhantom) {
+										// sweep existing deleted records and remove them
+										_.each(this.entities, (entity) => {
+											if (entity.isDeleted && entity.isDestroyed) {
+												this.removeEntity(entity);
+											}
+										})
+									}
 								} else if (this._operations.add || this._operations.delete) {
 									this.reload();
 								} else {

package/src/Repository/Repository.js

@@ -1818,7 +1818,9 @@ export default class Repository extends EventEmitter {
 	 */
 	removeEntity(entity) { // standard function notation so it can be called by child class via super.removeEntity
 		this.entities = _.filter(this.entities, e => e !== entity);
-		entity.destroy();
+		if (!entity.isDestroyed) {
+			entity.destroy();
+		}
 	}
 
 	/**