The new 2.x branch currently does not work properly with features... As an example: it still tries to call variable_realm_controller_build(), which has been removed and is currently causing a fatal error during features rebuild ("Call to undefined function ...").

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bforchhammer’s picture

Assigned: Unassigned » bforchhammer

I'm on this one.

bforchhammer’s picture

Here's a patch. Fairly straighforward, so unless something comes up I'll probably commit this later today... :)

bforchhammer’s picture

Assigned: bforchhammer » Unassigned
Status: Active » Needs review
bforchhammer’s picture

Hm, realms also need the "module" key in their info array, otherwise features can't add the right module dependencies.

Jose Reyero’s picture

Looks good to me. Just remove that 'dpm' :-)

bforchhammer’s picture

Status: Needs review » Fixed

Commited.

Status: Fixed » Closed (fixed)

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

xcafebabe’s picture

Using Features 7.x-2.0-beta1+1-dev and Variable 7.x-2.1+7-dev, when I try to include Global: All Variables in my feature this error shows up:

Notice: Undefined index: module in variable_realm_features_export() on line 35 of /sites/all/modules/contrib/variable/variable_realm/variable_realm.features.inc

Some hint to try to help and fix this issue?

acrollet’s picture

@xcafebabe: you need to re-export your feature after updating to -dev and the error should go away.

richsky’s picture

Version: 7.x-2.x-dev » 7.x-2.2
Status: Closed (fixed) » Active

Latest version (2.2) with this patch (#4) and still getting
Notice: Undefined index: module in variable_realm_features_export() (line 35 of E:\Web\Drupal_7\sites\all\modules\variable\variable_realm\variable_realm.features.inc).

artis.bajars’s picture

I can confirm this still being an issue, using Features 2.0-rc1 and Variable 2.2. Feature dependencies are not added correctly because of this.

dasha_v’s picture

candelas’s picture

same problem, same versions.
thanks for any help :)

Jose Reyero’s picture

Status: Active » Needs review
FileSize
684 bytes

Ok, so it seems we need to add the 'module' value to the realm info.

The idea looks good, but the patch has too many checks and may change the results because it does a full replacement of $result[$key] instead of merging.

Let's try this one.

frazras’s picture

OK #14 seemed to have worked for me, well at least its no showin up in the logs nor is it preventing the feature from adding other entries anymore.

Jose Reyero’s picture

Status: Needs review » Fixed

Great, committed.

Status: Fixed » Closed (fixed)

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