This drush command should disable all storm related modules.

It should then uninstall the sub modules in the correct order. It should the uninstall storm core.

All modules should be re-enabled again.

CommentFileSizeAuthor
#4 storm-1817840-storm_drush-3.patch1.43 KBwillwh
#3 storm-1817840-storm_drush-3.patch6.56 KBwillwh
#1 storm-1817840-drush-support-2.patch3.35 KBwillwh

Comments

willwh’s picture

willwh commented
Status: Active » Needs review
StatusFileSize
new storm-1817840-drush-support-2.patch3.35 KB

Here's a first crack at this - does not work quite as it should.

I think the issue here is, all the call_user_func_array() calls run immediately - anyone have any thoughts / tips? :)

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented

I would start by trying to call drush_invoke directly and not wrapping the call in call_user_func_array() which seems unnecessary in this context.

willwh’s picture

willwh commented
StatusFileSize
new storm-1817840-storm_drush-3.patch6.56 KB

Thanks eojthebrave! :)

So - this is definitely not the cleanest way to do this, but in the interests of further the port - this should help people trying to work on it!

It's not pretty but it works :)

willwh’s picture

willwh commented
StatusFileSize
new storm-1817840-storm_drush-3.patch1.43 KB

Need to change my workflow so I am not adding old patches to new ones, d'oh :)

Well, this works! :)

juliangb’s picture

juliangb commented
Category: task » feature
Status: Needs review » Fixed

Committed to both 6.x-2.x and 7.x-1.x branches - thanks!

However - please do watch that the patches don't contain other patch files, even #4 creates a blank patch file as part of the patch!

Status: Fixed » Closed (fixed)

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