Opened 12 years ago
Closed 12 years ago
#10212 closed Bug (fixed)
[IE7-10] Undo gives JS error after 2+ switches to source mode.
Reported by: | Joel | Owned by: | Piotrek Koszuliński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.0.3 |
Component: | General | Version: | 4.0.1 |
Keywords: | IE IBM | Cc: |
Description
Reproducible in IE9 in http://ckeditor.com/demo and nightly 13-03-13-08-51
- Do any undoable change, like add HR
- Go to source mode and switch back
- Go to source mode and switch back again
- Click undo icon or press Ctrl-Z
The error comes in IE developer tools console. For me the error was
SCRIPT70: Ei käyttöoikeutta ckeditor.js?mjldea, rivi 58 merkki 486
IE9, Win 7 64bit. Not reproducible in Chrome or Firefox.
Change History (8)
comment:1 Changed 12 years ago by
Keywords: | IE added |
---|---|
Status: | new → confirmed |
Summary: | [IE9] Undo gives JS error after 2+ switches to source mode. → [IE7-10] Undo gives JS error after 2+ switches to source mode. |
Version: | 4.0.2 → 4.0.1 |
comment:2 Changed 12 years ago by
Keywords: | IBM added |
---|
Another TC from IBM:
- Open any CK Editor sample, enter some text
- Switch to Source view and back to rich text again.
- Keep cursor in editor body, type some more text, Now click Undo icon or Ctrl + Z
#10216 was marked as duplicate.
comment:3 Changed 12 years ago by
Milestone: | → CKEditor 4.0.3 |
---|
comment:4 Changed 12 years ago by
I pushed t/10212 on dev with simple patch that seems to fix this issue. I'm not putting this ticket on review, since test should be wrriten.
comment:5 Changed 12 years ago by
Owner: | set to Piotrek Koszuliński |
---|---|
Status: | confirmed → assigned |
comment:6 follow-up: 7 Changed 12 years ago by
Status: | assigned → review |
---|
I pushed a test on t/10212. I couldn't reproduce this issue from code on IE9, but test correctly fails on IE8 and IE7.
comment:7 Changed 12 years ago by
Status: | review → review_passed |
---|
Replying to Reinmar:
I pushed a test on t/10212. I couldn't reproduce this issue from code on IE9, but test correctly fails on IE8 and IE7.
There's one browser that is affected by this issue but passes TC without the patch, namely IE10. This is a minor issue though, so R+.
comment:8 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed on master with git:2c54f8d on dev and 2a8e535 on tests.
Problem can be reproduced all versions of IE from CKEditor 4.0.1.
JS Error:
Message: Access Denied
Line: 190
URI: ckeditor4/core/dom/document.js