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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Vincent B’s picture

damiankloip’s picture

igor.ro’s picture

subsribing

damiankloip’s picture

igor.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.

damiankloip’s picture

Also, the issue referenced in #2 would make this closable.

ndobromirov’s picture

Under 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.

RumyanaRuseva’s picture

Version: 7.x-3.3 » 7.x-3.x-dev
Issue summary: View changes
FileSize
1.62 KB

I updated the patch from #6 for the current 7.x-3.x-dev.

RumyanaRuseva’s picture

Rerolled patch for the current 7.x-3.x-dev.