Looking at other CKEditor plugin modules and some of the documentation on drupal.org, it seems to be a defacto standard to have ckeditor plugins in libraries/ckeditor/plugins/pluginname.

This makes sense since it removes the risk of confusion and conflicts with non-ckeditor libraries that might have the same name and CKEditor's default instructions are to put plugins in a plugins sub-directory, which is what Drupal core does with its default CKEditor plugins.

If we can get consistency with where CKEditor plugin modules store their libraries it also makes it easier to manage all CKEditor plugins in a uniform manner.

Comments

rooby created an issue. See original summary.

mark_fullmer’s picture

+1 for this. I also think that additional libraries path locations should be supported, such as such as profiles/[profilename]/libraries or sites/default/libraries

See companion issue for the CKEditor A11yChecker at #3068819: Support alternate libraries path

Patch forthcoming...

mark_fullmer’s picture

Status: Active » Needs review
StatusFileSize
new2.93 KB

The attached patch is based largely on the Libraries module, though it does not require the Libraries module as a dependency. It will look in the following locations for the a11ychecker plugin:

- libraries/
- profiles/[activeprofile]/libraries
- sites/default/libraries
- sites/all/libraries/ (backwards-compatibility)
- libraries/ckeditor/plugins
- profiles/[activeprofile]/libraries/ckeditor/plugins
- sites/default/libraries/ckeditor/plugins
- sites/all/libraries/ckeditor/plugins (backwards-compatibility)

pifagor’s picture

Status: Needs review » Reviewed & tested by the community

  • pifagor committed 2cf62bf on 8.x-1.x authored by mark_fullmer
    Issue #3046778 by mark_fullmer, rooby: Support installing plugin to...
pifagor’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.