The Placeholder module can cause errors when installing Commons using the UI (other methods such as Drush work fine). The problem is caused by Placeholder relying on functions from the Libraries module to perform requirements checking during the installation phase, before modules have been installed and loaded, so the Libraries functions are unavailable.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Devin Carlson’s picture

Assigned: Unassigned » Devin Carlson
Status: Active » Needs review
FileSize
698 bytes

The Placeholder module should either perform a check to ensure that the Libraries module exists before attempting to check for requirements using its API functions or only check for requirements during the runtime phase.

The attached patch adds a patch against Placeholder to the Commons makefile which implements the latter solution.

Devin Carlson’s picture

Status: Needs review » Reviewed & tested by the community

Tested #1 with a new Commons installation and it fixed the error messages allowing the install to proceed.

Devin Carlson’s picture

Status: Reviewed & tested by the community » Fixed
ezra-g’s picture

Status: Fixed » Reviewed & tested by the community

Looks great to me.

ezra-g’s picture

Status: Reviewed & tested by the community » Fixed

Fixing status changed in a cross post :).

Status: Fixed » Closed (fixed)

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