From a0a05553c97ad205e51111d1f3b489c987e7ae83 Mon Sep 17 00:00:00 2001 From: Mo Bitar Date: Sun, 3 Jun 2018 13:24:55 -0500 Subject: [PATCH] Sync deleted errored items --- app/assets/javascripts/app/services/modelManager.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/app/services/modelManager.js b/app/assets/javascripts/app/services/modelManager.js index 1c6b2516d..0de7cf210 100644 --- a/app/assets/javascripts/app/services/modelManager.js +++ b/app/assets/javascripts/app/services/modelManager.js @@ -376,8 +376,11 @@ class ModelManager { } getDirtyItems() { - // Items that have errorDecrypting should never be synced back up to the server - return this.items.filter(function(item){return item.dirty == true && !item.dummy && !item.errorDecrypting}) + return this.items.filter((item) => { + // An item that has an error decrypting can be synced only if it is being deleted. + // Otherwise, we don't want to send corrupt content up to the server. + return item.dirty == true && !item.dummy && (!item.errorDecrypting || item.deleted); + }) } clearDirtyItems(items) {