EZP-26145: Unable to undo in the RichText editor (#664)

ezsystems · Aug 29, 2016 · 7963cbd · 7963cbd
1 parent 2b84f14
commit 7963cbd
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/Resources/public/js/alloyeditor/plugins/yui3.js b/Resources/public/js/alloyeditor/plugins/yui3.js
@@ -11,12 +11,14 @@ YUI.add('ez-alloyeditor-plugin-yui3', function (Y) {
     }
 
     function cleanUpIds(editor) {
+        editor.undoManager.lock();
         Array.prototype.forEach.call(
             editor.element.$.querySelectorAll('[id]'),
             function (element) {
                 element.removeAttribute('id');
             }
         );
+        editor.undoManager.unlock();
     }
 
     /**

diff --git a/Tests/js/alloyeditor/plugins/assets/ez-alloyeditor-plugin-yui3-tests.js b/Tests/js/alloyeditor/plugins/assets/ez-alloyeditor-plugin-yui3-tests.js
@@ -68,6 +68,21 @@ YUI.add('ez-alloyeditor-plugin-yui3-tests', function (Y) {
                 "The id should have been removed"
             );
         },
+
+        "Should lock and unlock the undo manager": function () {
+            var nativeEditor = this.editor.get('nativeEditor');
+
+            nativeEditor.undoManager = new Mock(nativeEditor.undoManager);
+            Mock.expect(nativeEditor.undoManager, {
+                method: 'lock',
+            });
+            Mock.expect(nativeEditor.undoManager, {
+                method: 'unlock',
+            });
+            nativeEditor.execCommand('bold');
+
+            Mock.verify(nativeEditor.undoManager);
+        },
     });
 
     Y.Test.Runner.setName("eZ AlloyEditor yui3 plugin tests");