I have some Commons installed alongside Drupal Commerce. When I visit the Feature that handles all these dependencies, I am seeing some PHP warnings.

commons-default_rules_configuration_alter-warning.png

I inserted a krumo dump above to see how many times hook_default_rules_configuration_alter was called. We can see its called 5 times (by various Commerce modules) before commons_groups.module instantiates it.

The commons code assumes $configs['rules_og_member_active'] is available to modify. 5/6 times, its not part of the configs array.

Checking to see if this key exists before assigning anything to the object should prevent this from happening.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

saltednut’s picture

Note: patch looks a little weird because I also adjusted some indentations.

ezra-g’s picture

Status: Needs review » Fixed

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