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.
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
Comment #1
yhahn CreditAttribution: yhahn commentedYes, this needs some more helptext.
For the time being, there is a description of why this shows up in API.txt.
Comment #2
hadsie CreditAttribution: hadsie commentedI'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.
Comment #3
abeger CreditAttribution: abeger commentedAs 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.
Comment #4
ianthomas_ukMake 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.
Comment #5
mpotter CreditAttribution: mpotter commentedComment #6
ianthomas_ukI think this got its status flipped by drupal.org's upgrade to D7.
Comment #7
euk CreditAttribution: euk commentedHello!
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!
Comment #8
mitchelljj CreditAttribution: mitchelljj commentedAfter recreating a feature I sometimes get "Needs Review" displayed but it many cases it goes away once I have cleared the Drupal Cache.
Comment #9
dman CreditAttribution: dman as a volunteer and at Sparks Interactive commented"needs some more helptext" is still an issue in the current UI.
I had to come to Google to figure out what this was telling me.
There's no explanation in /admin/help/features or any link to online docs like this where it's needed.
I got it today when applying a rebuilt feature (that included a version number update) over top of a recently pulled DB.
Hint : enabling the 'diff' module (always handy with Features) showed me that there was a trivial change where the upstream DB had diverged from the one I was rebuilding the feature from (content type field weights again, *sigh*). Using diff helped me confirm that yes, I did want to revert to the code version.
Comment #10
alexharries CreditAttribution: alexharries commentedI'd like to "+1" dman's comment - it would be a huge UX improvement if we could annotate the "Needs review" text with something like, "(download and install the Diff module to view these changes)".
Comment #11
Greg BoggsDoes anyone know exactly what triggers needs review vs overridden?
Comment #12
jlstreckerYes. From the API.txt file mentioned in #1: