We've got "Revert All" features task, and I've got a "update all features" task in https://www.drupal.org/project/aegir_config, so might as well put it in hosting_tasks_extras instead?

CommentFileSizeAuthor
#3 2896002-task-fua.patch2.08 KBjon pugh

Comments

Jon Pugh created an issue. See original summary.

  • Jon Pugh committed ec5a743 on 2896002-task-fua 
    Issue #2896002: Add "Update all Features" task
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
StatusFileSize
new 2896002-task-fua.patch2.08 KB
helmo’s picture

helmo commented

+1

minor nitpicking:

  1. +++ b/drush/provision_tasks_extra.drush.inc
@@ -224,6 +227,21 @@ function drush_provision_tasks_extra_provision_features_revert_all() {
+ * Implements the provision-features_revert_all command.

    s/revert/update/ 

  2. +++ b/drush/provision_tasks_extra.drush.inc
@@ -224,6 +227,21 @@ function drush_provision_tasks_extra_provision_features_revert_all() {
+  ¶

    whitespace

  • helmo committed 5bed43c on 2896002-task-fua 
    Issue #2896002: minor nitpicking
  • Jon Pugh committed ec5a743 on 7.x-3.x, 2896002-task-fua
    Issue #2896002: Add "Update all Features" task

  • helmo committed 4500d2f on 2896002-task-fua 
    Issue #2896002: Add matching permission
  • helmo committed fe3fe0d on 2896002-task-fua
    Issue #2896002: Add form validation to check if features module is...

  • helmo committed 4500d2f on 7.x-3.x 
    Issue #2896002: Add matching permission
  • helmo committed 5bed43c on 7.x-3.x
    Issue #2896002: minor nitpicking
  • helmo committed baf3548 on 7.x-3.x
    Issue #2896002: Merge remote-tracking branch 'origin/2896002-task-fua'...
  • helmo committed fe3fe0d on 7.x-3.x
    Issue #2896002: Add form validation to check if features module is...
helmo’s picture

helmo commented
Status: Needs review » Fixed

I added a bit more validation/permission code.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.