Download & Extend
CKEditor - WYSIWYG HTML editor » Issues

Fatal error if Libraries module is enabled after CKEditor

Posted by kscheirer on January 25, 2013 at 12:44am
Project:CKEditor - WYSIWYG HTML editor
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

This commit (http://drupalcode.org/project/ckeditor.git/commitdiff/f6abbda281738527d1...) added a call to _ckeditor_requirements_isinstalled(), but fails to load the inc file that defines the function. So I'm seeing a fatal error when installing:

FATAL ERROR: CALL TO UNDEFINED FUNCTION _CKEDITOR_REQUIREMENTS_ISINSTALLED() IN /WWW/COMMONS-7.X-3.X-DEV/PROFILES/COMMONS/MODULES/CONTRIB/CKEDITOR/CKEDITOR.MODULE ON LINE 615

It could be because it was loaded via an install profile instead of just enabled as a new module.

Login or register to post comments

Comments

#1

Posted by kscheirer on January 25, 2013 at 12:52am
Status:active» needs review

This patch moves the lin to load the library up above the call to _ckeditor_requirements_isinstalled().

AttachmentSize
ckeditor-install-lib-1898294-2.patch 734 bytes

#2

Posted by jcisio on January 25, 2013 at 10:09am
Title:Regression: ckeditor fails install» Fatal error if Libraries module is enabled after CKEditor
Status:needs review» reviewed & tested by the community

I was almost going to submit the same patch.

#3

Posted by wwalc on February 2, 2013 at 9:09pm
Status:reviewed & tested by the community» fixed

Big thanks to both of you! Fixed with a89c4ae

#4

Posted by System Message on February 16, 2013 at 9:10pm
Status:fixed» closed (fixed)

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

nobody click here