Don't write pref on initial load, per review

adobe · May 5, 2015 · 8b4c7b2 · 8b4c7b2
1 parent 83eb276
commit 8b4c7b2
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/extensions/default/QuickView/main.js b/src/extensions/default/QuickView/main.js
@@ -739,11 +739,13 @@ define(function (require, exports, module) {
         CommandManager.get(CMD_ENABLE_QUICK_VIEW).setChecked(enabled);
     }
 
-    function setExtensionlessImagePreview(_extensionlessImagePreview) {
+    function setExtensionlessImagePreview(_extensionlessImagePreview, doNotSave) {
         if(extensionlessImagePreview !== _extensionlessImagePreview) {
             extensionlessImagePreview = _extensionlessImagePreview;
-            prefs.set("extensionlessImagePreview", enabled);
-            prefs.save();
+            if (!doNotSave) {
+                prefs.set("extensionlessImagePreview", enabled);
+                prefs.save();
+            }
         }
     }
 
@@ -814,7 +816,7 @@ define(function (require, exports, module) {
 
     // Setup initial UI state
     setEnabled(prefs.get("enabled"), true);
-    setExtensionlessImagePreview(prefs.get("extensionlessImagePreview"));
+    setExtensionlessImagePreview(prefs.get("extensionlessImagePreview"), true);
 
     prefs.on("change", "enabled", function () {
         setEnabled(prefs.get("enabled"), true);