I just freshly installed features-7.x-1.0-rc3, with no features installed or available, and get the following messages on
admin/structure/features:

Notice: Undefined index: feature in features_get_info() (line 625 of ../features.module).
Warning: array_keys() expects parameter 1 to be array, null given in features_get_info() (line 625 of ../features.module).
Warning: Invalid argument supplied for foreach() in features_get_info() (line 626 of ../features.module).
Warning: array_unique() expects parameter 1 to be array, null given in features_get_info() (line 631 of ../features.module).
Warning: Invalid argument supplied for foreach() in features_get_info() (line 632 of ../features.module)./features.module).

Fix:
In features.module, line 590, replace

  if (empty($cache) || $reset) {
    $data = array();

with

  if (empty($cache) || $reset) {
    $data = array('feature' => array(), 'module' => array());

Otherwise, 'feature' can be left empty.

CommentFileSizeAuthor
#1 features-1635662-1.patch527 bytesdonquixote
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

donquixote’s picture

donquixote CreditAttribution: donquixote commented
Status: Active » Needs review
FileSize
features-1635662-1.patch527 bytes

Patch.

Note: The 'module' => array() is not really required, but just looks more robust.

szantog’s picture

szantog CreditAttribution: szantog commented
Status: Needs review » Reviewed & tested by the community

Patch cannot applied to the latest dev, but the solution is ok, when I applied it manually.

juampynr’s picture

juampynr CreditAttribution: juampynr commented
Version: 7.x-1.0-rc3 » 7.x-1.x-dev
Priority: Normal » Major

@szantog's comment at #2 is wrong. I could apply the patch cleanly and confirmed that it fixes the issue.

Changing priority to major, as this error will happen to anyone installing features for the first time.

Andi-D’s picture

Andi-D CreditAttribution: Andi-D commented

Applied Patch on rc3 and it works
thx

mpotter’s picture

mpotter CreditAttribution: mpotter commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to a77b4d3

IshanComBr’s picture

IshanComBr CreditAttribution: IshanComBr commented

patch #1 works for me also

TKS!

joachim’s picture

joachim CreditAttribution: joachim commented

Might this warrant a new RC release? It's a bit alarming for new users!

caspercash’s picture

caspercash CreditAttribution: caspercash commented

same here.. patch in #1 works.. and i think it would be better to fix the current RC release with this patch..

langworthy’s picture

langworthy CreditAttribution: langworthy commented

certainly warrants a new RC release.

sylvain_a’s picture

sylvain_a CreditAttribution: sylvain_a commented

I support this!

adamelleston’s picture

adamelleston CreditAttribution: adamelleston commented

I would support having a new RC

scuba_fly’s picture

scuba_fly
Location Netherlands, Vinkeveen
CreditAttribution: scuba_fly commented

I also support having a new RC with only this fixed if that speeds up releasing this fix in the RC. other fixes can wait I guess?

aggentle’s picture

aggentle CreditAttribution: aggentle commented

Confirming that I've found the same issue with ...rc3 on D7; is this a cosmetic fault or is this a sign that the installed Features is "broken"?

I can live with the error message on install but please let me know if I need to install the patch.

Thanks.

shadcn’s picture

shadcn CreditAttribution: shadcn commented

+1 for new RC please.

quantos’s picture

quantos CreditAttribution: quantos commented

+1 for me too. I get nervous when I see those error messages and don't know what they mean!

joachim’s picture

joachim CreditAttribution: joachim commented

Filed #1697606: make a new release as a critical to hopefully get the attention of maintainers.

Please feel free to follow that one to lend weight to it :)

mpotter’s picture

mpotter CreditAttribution: mpotter commented

Posting here is all you need to do to get attention. Creating separate issues is not the proper way to handle this. A new release is planned for this week.

imp7’s picture

imp7 CreditAttribution: imp7 commented

Thanks mpotter, I can now drush make an error free drupal, features is awesome :)

Status: Fixed » Closed (fixed)

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

donquixote’s picture

donquixote CreditAttribution: donquixote commented
Issue summary: View changes

This was a great fix, but:
#2562871: Errors when going to admin/structure/features/<invalid-feature> killed it again :(