When installing htmlpurifier.module using an install profile (and possibly other installation use cases), with the required htmlpurifier library installed in sites/all/libraries - htmlpurifier_requirements() returns severity=REQUIREMENT_ERROR and won't continue (wanting the library to be installed in sites/default/module/htmlpurifier/library). I have libraries.module installed (but due to being in installation profile it would not yet be enabled). This is probably because the libraries.module is not a dependency (for legacy reasons).

The attached patch fixes this particular use case by checking for the libraries.module and including it.

CommentFileSizeAuthor
htmlpurifier-libraries-not-enabled.patch859 bytespeter.walter

Comments

ezyang’s picture

ezyang commented
Status: Needs review » Fixed

Commited to CVS, with extra message on why this is necessary. I'll note that this has the failure mode of resulting in warnings if libraries module is present as a file, but not loaded, and you're HTML Purifier library is in the wrong place. A follow-up patch to fix this would be appreciated.

Status: Fixed » Closed (fixed)

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