
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
Comment | File | Size | Author |
---|---|---|---|
#1 | lib-dir-checking-2165939-1.patch | 1.5 KB | jessebeach |
Comments
Comment #1
jessebeach CreditAttribution: jessebeach commentedtwo line fix
Comment #2
wim leersSo much more elegant! Thanks :)
http://drupalcode.org/project/edit.git/commit/3eb10d9
Comment #4
wim leers… failed to apply… because it was already committed days ago. WTF, d.o?!