Cannot remove contextmenu from plugins

[Brett]

Attempting to create a new config.js file with a command such as:

config.removePlugins = 'contextmenu';

...causes the WYSIWYG to not load. I tested a few more plugins and they caused the same issue to occur (tho, equally, others don't cause this issue to occur)

Windows 2003 Server using the "Replace by Code" method - if that makes a difference.

[Artur Formella]

Ops, menu button doesn't work.

[Artur Formella]

CKEDITOR.plugins.contextMenu is required by buttonMenu.

There are 2 ways to fix this bug:

first - disable context menu if is loaded indirect,

second - rename CKEDITOR.plugins.contextMenu and move it somewhere else (i.e. to menu plugin) and create editor.contextMenu in contextmenu plugin (first part of current plugin)

Patch contains the first solution because is simpler.

[Frederico Caldeira Knabben]

Removing the plugin is not a guarantee that it will not be loaded. There may be dependencies that will require it. What we're missing is having these dependencies well defined in the plugins, so nothing breaks.

For this specific case, we need the contextmenu plugin for the buttonmenu plugin, as identified by Artur. So, it's just a matter of updating the plugin requirements.

I would instead avoid adding the automation for disabling the context menu. It may be "really" required by other plugins. We may instead have a configuration option to disable it, but this is something that should be managed in another ticket.

So, the changes on _source/plugins/menubutton/plugin.js are the only one to be committed.

[Frederico Caldeira Knabben]

The changelog entry is wrong now. It should be something like: "The attempt to remove the contextmenu plugin will not anymore break the editor". Please fix it when committing.

[Artur Formella]

Fixed with [3855]