Opened 8 years ago
Closed 8 years ago
#13056 closed Task (fixed)
Plugin dependencies should be checked
| Reported by: | Wiktor Walc | Owned by: | Piotrek Koszuliński |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.5.0 |
| Component: | General | Version: | 4.5.0 Beta |
| Keywords: | Cc: |
Description (last modified by )
uploadimage requires clipboard to be loaded, but the clipboard plugin is not listed as a requirement:
https://github.com/ckeditor/ckeditor-dev/blob/major/plugins/uploadimage/plugin.js#L22
As a result CKEditor may not load. Might be worth to check other newly introduced plugins for similar issues.
There is a typo in the mentioned line as well ("lister")
Change History (7)
comment:1 Changed 8 years ago by
| Description: | modified (diff) |
|---|---|
| Status: | new → confirmed |
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
| Type: | Bug → Task |
|---|
comment:4 Changed 8 years ago by
It seems to be ok - uploadimage requires uploadwidget which requires clipboard.
comment:5 Changed 8 years ago by
Interesting, I'm quite sure I had error in that line and that listing all plugins manually did the trick, but indeed the requirements are fine. I'm okay with closing this ticket if requirements were checked for new plugins.
comment:6 Changed 8 years ago by
| Owner: | set to Piotrek Koszuliński |
|---|---|
| Status: | confirmed → assigned |
comment:7 Changed 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixed comment:2 and the mentioned typo with git:93d3318.
Note: the notification plugin kind of requires the toolbar plugin, because without it there's no top space.