Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Explosion when destroying editor during initialization phase. #2891

Closed
oskarwrobel opened this issue Jun 28, 2018 · 0 comments · Fixed by ckeditor/ckeditor5-core#135
Closed

Explosion when destroying editor during initialization phase. #2891

oskarwrobel opened this issue Jun 28, 2018 · 0 comments · Fixed by ckeditor/ckeditor5-core#135
Assignees
@oskarwrobel
Labels
package:core type:bug This issue reports a buggy (incorrect) behavior.
Milestone
iteration 19

Comments

@oskarwrobel
Copy link
Contributor

Calling Editor#destroy() during initialization phase leads to exceptions. We need to wait to for the editor#ready before destroying process begins.
@oskarwrobel oskarwrobel self-assigned this Jun 28, 2018
pjasiun referenced this issue in ckeditor/ckeditor5-core Jun 29, 2018
Merge pull request #135 from ckeditor/t/134
ad1da26 
Fix: Editor#destroy waits for the initialization. Closes #134.
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-core Oct 9, 2019
@mlewand mlewand added this to the iteration 19 milestone Oct 9, 2019
@mlewand mlewand added status:confirmed type:bug This issue reports a buggy (incorrect) behavior. package:core labels Oct 9, 2019
@oskarwrobel oskarwrobel mentioned this issue Oct 11, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oskarwrobel oskarwrobel

Labels
package:core type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Milestone
iteration 19
Development

Successfully merging a pull request may close this issue.

Editor#destroy waits for the initialization ckeditor/ckeditor5-core
2 participants
@oskarwrobel @mlewand