Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 ...").
Comment | File | Size | Author |
---|---|---|---|
#14 | 1612484-features_export_notice-14.patch | 684 bytes | Jose Reyero |
#12 | variable_realm_features_export_fix-1612484-12.patch | 702 bytes | dasha_v |
#4 | features-fix-1612484-4.patch | 4.16 KB | bforchhammer |
#2 | features-fix-1612484-2.patch | 1.29 KB | bforchhammer |
Comments
Comment #1
bforchhammer CreditAttribution: bforchhammer commentedI'm on this one.
Comment #2
bforchhammer CreditAttribution: bforchhammer commentedHere's a patch. Fairly straighforward, so unless something comes up I'll probably commit this later today... :)
Comment #3
bforchhammer CreditAttribution: bforchhammer commentedComment #4
bforchhammer CreditAttribution: bforchhammer commentedHm, realms also need the "module" key in their info array, otherwise features can't add the right module dependencies.
Comment #5
Jose Reyero CreditAttribution: Jose Reyero commentedLooks good to me. Just remove that 'dpm' :-)
Comment #6
bforchhammer CreditAttribution: bforchhammer commentedCommited.
Comment #8
xcafebabe CreditAttribution: xcafebabe commentedUsing 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?
Comment #9
acrollet CreditAttribution: acrollet commented@xcafebabe: you need to re-export your feature after updating to -dev and the error should go away.
Comment #10
richsky CreditAttribution: richsky commentedLatest 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).
Comment #11
artis.bajars CreditAttribution: artis.bajars commentedI 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.
Comment #12
dasha_v CreditAttribution: dasha_v commentedThe fix from #6 was lost in #1664154: Fix variable hooks invoked during bootstrap (Variable realm breaks pathauto).
Draft for the patch attached.
Comment #13
candelas CreditAttribution: candelas commentedsame problem, same versions.
thanks for any help :)
Comment #14
Jose Reyero CreditAttribution: Jose Reyero commentedOk, 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.
Comment #15
frazras CreditAttribution: frazras commentedOK #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.
Comment #16
Jose Reyero CreditAttribution: Jose Reyero commentedGreat, committed.