Currently it is possible to revert a single component of a feature via the UI or via drush if you are in interactive mode (by selecting "no" for the components you do not wish to revert). This patch makes it so you can filter the list of components to revert via an option.

CommentFileSizeAuthor
#4 features-components-drush-1429262.patch2.83 KBneochief
#1 features-drush_revert_component-1429262.patch1.11 KBjoelcollinsdc

Comments

joelcollinsdc’s picture

joelcollinsdc commented
StatusFileSize
new features-drush_revert_component-1429262.patch1.11 KB
joelcollinsdc’s picture

joelcollinsdc commented
Status: Active » Needs review
hefox’s picture

hefox commented
Status: Needs review » Needs work

Seems good, but tiny space issue

neochief’s picture

neochief commented
Status: Needs work » Needs review
StatusFileSize
new features-components-drush-1429262.patch2.83 KB

It looks like original patch intented to work only with one component at a time, since something like this down't work:
drush fr foo --components=c1 c2 --force

Drush understand c2 as another feature to revert, not a component.

Other drawback of this change is that you can't specify components for multiple features in one command.

I have modified the patch and changed it behavior a little bit, so now you can run commands like this:
drush fr foo.node foo.taxonomy bar --force

Examples of command also included.

joelcollinsdc’s picture

joelcollinsdc commented

I assumed the way that you passed multiple space-separated arguments to a single option was to surround them with quotes. Your approach seems better, though, very interesting.

mpotter’s picture

mpotter commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

Nice update to the syntax. Committed and pushed in 6239499.

  • mpotter committed 6239499 on 8.x-3.x 
    Issue #1429262 by joelcollinsdc, neochief: Added Allow component level...