Changes between Initial Version and Version 12 of Ticket #13385


Ignore:
Timestamp:
Jul 8, 2015, 2:50:59 PM (9 years ago)
Author:
Piotrek Koszuliński
Comment:

Fixed on master with git:fafab77.

I was also unable to reproduce the mentioned issue in the mentioned way, but there was definitely a bug in getSnapshot(). Therefore, I'll change the ticket description to match a bug that we fixed. If the bug that @wginals could reproduce will be still reproducible, please open a new ticket.

Please also note that there must also be another bug somewhere because after editor is destroyed nothing should try to get snapshots. However, it would be very hard to make sure that it's always true, because short after editor is destroyed a lot may happen. Therefore, we chose the easier path to make sure that at least there's no error.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13385

    • Property Status changed from new to closed
    • Property Summary changed from JavaScriptException in undo/plugin.js to Editor.getSnapshot() should return a string, but it returns a boolean
    • Property Version changed from 4.4.7 to
    • Property Milestone changed from to CKEditor 4.5.2
    • Property Owner set to Szymon Cofalik
    • Property Resolution changed from to fixed
  • Ticket #13385 – Description

    initial v12  
     1If editor is destroyed, editor.getSnapshot() returns a non-string value. Better if it always return a string.
     2
     3Please read comment:1 and comment:12.
     4
     5== Original TC (we did not confirm it): ==
     6
    17We have the following issue, when we combine IE, inline editing and the codesnippt plugin.
    28
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy