My feature tells me "Needs review", with no further explanation of what this might mean or what the problem might be.

Review how? What am I meant to do? what should I be looking for?

Comments

Assigned:Unassigned» yhahn
Category:bug» task

Yes, this needs some more helptext.

For the time being, there is a description of why this shows up in API.txt.

I'm also curious about this :). In the description it states "User input is necessary to resolve this state" but I've never actually had to do anything (or had an option to do anything afaik). Basically on some of my sites, (all with identical code bases running the same updates (aegir migration)) I get overridden and others I get "needs review". The only difference in this particular case was that the site with "needs review" was newer and the feature had never been reverted before.

As far as I can tell, the "user input" needed is either to update or revert. I ran a "drush fr" and it resolved the issue for me.

Assigned:Unassigned» yhahn
Status:Closed (works as designed)» Active

Make sure you know what you're doing before following abeger's advice

Both "needs review" and "overridden" statuses mean that the feature in the database does not match the feature in code. Running "drush fr" (feature revert) will replace the feature in the database with the feature in code, which may or may not be what you want to do.

My workflow when this happens is as follows:
1. Make sure the existing feature code is committed to version control.
2. Run "drush fu feature-name" to replace the code for feature-name with the configuration from the database
3. Diff the file with the version in version control so you understand the changes and can pick and choose which you want.
4. Commit you changed feature code and/or revert the feature in Drupal

Remember, revert for Drupal features means copy configuration from code to the database.

Assigned:yhahn» Unassigned
Status:Active» Closed (works as designed)

Assigned:yhahn» Unassigned
Issue summary:View changes
Status:Active» Closed (works as designed)

I think this got its status flipped by drupal.org's upgrade to D7.

Hello!

The last question is - how bad is "Needs review" state and who has precedence in this case - DB or code? I mean - what values will be used - stored in DB or in code?

thanks!