Opened 11 years ago

Closed 11 years ago

#2911 closed New Feature (wontfix)

Make it possible for CKEDITOR.plugins.addExternal() to load plugins not named "plugin.js"

Reported by: Martin Kou Owned by: Martin Kou
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Review- Cc:

Description

This feature request came from Senthil Kumaran of Oracle.

At the moment CKEDITOR.plugins.addExternal() is only able to load plugins whose filename is plugin.js, because the plugin.js filename is hardcoded into the plugin system.

It is requested that the addExternal() should be able to load arbitrary .js filenames as plugins, e.g. "/my_plugins/abcde.js".

Attachments (1)

2911.patch (696 bytes) - added by Martin Kou 11 years ago.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by Martin Kou

Attachment: 2911.patch added

comment:1 Changed 11 years ago by Martin Kou

Keywords: Review? added

comment:2 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed
Resolution: wontfix
Status: newclosed

The problem with this approach is that, opening it in this way, will inspire people to point the editor to load server side scripts, like plugin.php, or even URLs with querystring parameters. It would just make the system more complex than it needs to be.

I opt for leaving the plugin.js file a requirement. If implementors have specific needs, there may be other ways to satisfy them, like customizing the getUrl function, for example.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy