Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Hello,
As explained in this issue still opened, when we applied a views revert or vr drush command on a views or list of views containing a non-overridden views, drush will be set in error.
I think drush must only notice that the views haven't been reverted but not return a status 1 (which is the code for an error).
Here's a patch which will change messages and return code when a views is not overridden.
Comments
Comment #1
Vincent B CreditAttribution: Vincent B commentedComment #2
damiankloip CreditAttribution: damiankloip commentedIt may be worth referencing #1549012: Refactor views drush commands to use CTools drush commands for exportable actions here.
Comment #3
igor.ro CreditAttribution: igor.ro commentedsubsribing
Comment #4
damiankloip CreditAttribution: damiankloip commentedigor.ro, you can use the 'subscribe' button at the top of issues to follow them. No need for the whole 'sub' or '+1' thing anymore.
Comment #5
damiankloip CreditAttribution: damiankloip commentedAlso, the issue referenced in #2 would make this closable.
Comment #6
ndobromirov CreditAttribution: ndobromirov commentedUnder AEGIR environment any error breaks the installation of new profiles or sites, so lowering the level from error to warning in the patch below. The patch is based on the one in comment #1.
Comment #7
RumyanaRuseva CreditAttribution: RumyanaRuseva at FFW commentedI updated the patch from #6 for the current 7.x-3.x-dev.
Comment #8
RumyanaRuseva CreditAttribution: RumyanaRuseva at FFW commentedRerolled patch for the current 7.x-3.x-dev.