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

Executing editor.destroy() during image upload throws error #966

Closed
MaksimMakarevich opened this issue Sep 27, 2017 · 2 comments
Closed

Executing editor.destroy() during image upload throws error #966

MaksimMakarevich opened this issue Sep 27, 2017 · 2 comments
Labels
good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. plugin:uploadwidget The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone
4.10.1

Comments

@MaksimMakarevich
Copy link
Contributor

Are you reporting a feature request or a bug?

Bug

Provide detailed reproduction steps (if any)

  1. Start upload large file
  2. Destroy editor

Expected result

What is the expected result of the above steps?
Uploading are canceled.

Actual result

Editor are crashed with error.

What is the actual result of the above steps?

Other details

  • Browser: All
  • OS: Window
  • CKEditor version: 4.7.1
  • Installed CKEditor plugins: default plugins
@MaksimMakarevich MaksimMakarevich mentioned this issue Sep 27, 2017
Closed
@beatadelura beatadelura self-assigned this Oct 24, 2017
@beatadelura
Copy link
Contributor

I'm able to reproduce this issue on SDK Sample with CKEditor version 4.7.3.

@beatadelura beatadelura added type:bug A bug. status:confirmed An issue confirmed by the development team. plugin:uploadwidget The plugin which probably causes the issue. labels Oct 24, 2017
@beatadelura beatadelura removed their assignment Oct 24, 2017
@mlewand mlewand added good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. target:minor Any docs related issue that can be merged into a master or major branch. labels Jan 22, 2018
@mlewand mlewand added this to the Backlog milestone Jan 22, 2018
@jacekbogdanski jacekbogdanski changed the title Bug: Upload Widget crash Executing editor.destroy() during image upload throws error Jan 26, 2018
@jacekbogdanski jacekbogdanski mentioned this issue Jan 29, 2018
Closed
2 tasks
@mlewand mlewand modified the milestones: Backlog, 4.10.1 Jul 19, 2018
@mlewand
Copy link
Contributor

mlewand commented Jul 19, 2018

Fixed with 33d301f.

@mlewand mlewand closed this as completed Jul 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. plugin:uploadwidget The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Projects
None yet
Milestone
4.10.1
Development

Successfully merging a pull request may close this issue.

Fixed executing editor.destroy() during image upload ckeditor/ckeditor4
3 participants
@mlewand @beatadelura @MaksimMakarevich