From ff64a8340cfd2d82ded628b7f904ca539e786025 Mon Sep 17 00:00:00 2001 From: gorhill Date: Wed, 18 Jan 2017 13:35:10 -0500 Subject: [PATCH] code review: only built-in assets are candidates for removal when updating assets.json --- src/js/assets.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/js/assets.js b/src/js/assets.js index f4daf8f28..42211c33a 100644 --- a/src/js/assets.js +++ b/src/js/assets.js @@ -352,9 +352,12 @@ var updateAssetSourceRegistry = function(json) { getAssetSourceRegistry(function(oldDict) { var assetKey; - // Remove obsolete entries + // Remove obsolete entries (only those which were built-in). for ( assetKey in oldDict ) { - if ( newDict[assetKey] === undefined ) { + if ( + newDict[assetKey] === undefined && + oldDict[assetKey].submitter === undefined + ) { unregisterAssetSource(assetKey); } }