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.

Members fund testing for the Drupal project. Drupal Association Learn more


ezyang’s picture

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.