Opened 13 years ago
Closed 11 years ago
#8663 closed Bug (fixed)
Method dom.element#renameNode() should clear #getName() cache
Reported by: | Piotrek Koszuliński | Owned by: | Piotrek Koszuliński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.3.4 |
Component: | General | Version: | 3.2.2 |
Keywords: | Cc: |
Description
element = CKEDITOR.dom.element.createFromHtml( '<div></div>' ); element.renameNode( 'p' ); assert.areEqual( 'p', element.getName() ); // actual: 'div'
Change History (4)
comment:1 Changed 13 years ago by
Status: | new → confirmed |
---|---|
Version: | → 3.2.2 |
comment:2 Changed 11 years ago by
Milestone: | → CKEditor 4.3.4 |
---|---|
Owner: | set to Piotrek Koszuliński |
Status: | confirmed → review |
Pushed branch:t/8663 on dev and test.
comment:3 Changed 11 years ago by
Status: | review → review_passed |
---|
I've added a new test that guarantees the efficiency of the trick used in this fix. Interesting trick, btw.
comment:4 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed on master with git:662fc078 on dev and 6a62118 on tests.
Can be also checked in browser with the below code:
Reproducible from CKEditor 3.2.2 (Before this version error was thrown).