Improved autosave cleanup, fixes #2095

diff --git a/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Redactor/Autosave.js b/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Redactor/Autosave.js
index 74fc977ebedadf5a6b5d578049606d166ee67a11..d964565cefb119f96f54ba2aa310d029cacd184f 100644 (file)
--- a/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Redactor/Autosave.js
+++ b/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Redactor/Autosave.js
@@ -160,9 +160,9 @@ define(['Dom/Traverse'], function(DomTraverse) {
                 * @protected
                 */
                _cleanup: function () {
-                       var oneWeekAgo = Date.now() - (7 * 24 * 3600 * 1000);
-                       var key, value;
-                       for (var i = 0, length = window.localStorage.length; i < length; i++) {
+                       var oneWeekAgo = Date.now() - (7 * 24 * 3600 * 1000), removeKeys = [];
+                       var i, key, length, value;
+                       for (i = 0, length = window.localStorage.length; i < length; i++) {
                                key = window.localStorage.key(i);
                                
                                // check if key matches our prefix
@@ -185,12 +185,16 @@ define(['Dom/Traverse'], function(DomTraverse) {
                                }
                                
                                if (!value || value.timestamp < oneWeekAgo) {
-                                       try {
-                                               window.localStorage.removeItem(key);
-                                       }
-                                       catch (e) {
-                                               window.console.warn("Unable to remove from local storage: " + e.message);
-                                       }
+                                       removeKeys.push(key);
+                               }
+                       }
+                       
+                       for (i = 0, length = removeKeys.length; i < length; i++) {
+                               try {
+                                       window.localStorage.removeItem(removeKeys[i]);
+                               }
+                               catch (e) {
+                                       window.console.warn("Unable to remove from local storage: " + e.message);
                                }
                        }
                }