Hierarchical Select's Features integration is broken because common.inc isn't included before hierarchical_select_common_config_set gets called in hierarchical_select_features_rebuild. The attached patch fixes the problem, though I'm not sure if it's the best place for that code.

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


kyle_mathews’s picture

Version: 6.x-3.7 » 6.x-3.x-dev
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community
909 bytes

I had the same problem. The Features page won't even load without this patch which took me awhile to track down the cause. It doesn't look like a patch got attached so I've attached one I created.

cotto’s picture

How clever of me. The patch kyle_mathews posted is essential the same as mine. Thanks.

davad’s picture

Rerolled Kyle's patch against 6.x-3.7 to include in a drush make file.

kyle_mathews’s picture

Trying another patch for our make file.

davad’s picture

The patch stopped applying correctly when we switched to pressflow. It looks like git-apply applies patches against the base of the working directory. Attached is a rerolled patch against a pressflow installation.

mcjim’s picture

Patch(es) work(s).

Makes sense to move the include to within the if statement, though? Save it being called if not needed.

Wim Leers’s picture

Looks good.

wizonesolutions’s picture

This fixed it for me as well. +1.

jdwfly’s picture

Definitely needs to be committed. Just ran into this issue today. Unusable without this small change.

Wim Leers’s picture

Title: missing include breaks features » Missing include breaks Features
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

jeffschuler’s picture

Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Closed (fixed) » Needs review
594 bytes

Same thing for D7, to fix:

Call to undefined function hierarchical_select_common_config_set()

Wim Leers’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Needs review » Closed (fixed)

#12: that was fixed in [#1847974 ].

Wim Leers’s picture

#12: that was fixed in [#1847974 ].