Problem/Motivation

In hook_requirements, edit checks that the CKEditor library is located in sites/all/libraries.

if (strpos(libraries_get_path('ckeditor'), 'sites/all/libraries') === FALSE) {

But a library can also be located under the profiles dir or a specific site dir.

Proposed resolution

The libraries_get_path method actually returns NULL if a library isn't located in a sanctioned directory. So we can assume that if libraries can't find the ckeditor plugin, then it's not in the right place.

User interface changes

None.

API changes

None.

Original report by @jessebeach

CommentFileSizeAuthor
#1 lib-dir-checking-2165939-1.patch1.5 KBjessebeach

Comments

jessebeach’s picture

Status: Active » Needs review
StatusFileSize
new1.5 KB

two line fix

wim leers’s picture

Status: Needs review » Fixed

Status: Fixed » Needs work

The last submitted patch, 1: lib-dir-checking-2165939-1.patch, failed testing.

wim leers’s picture

Status: Needs work » Closed (fixed)

… failed to apply… because it was already committed days ago. WTF, d.o?!