Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Installing a number of modules in a row can be tedious. You have to navigate back to the module page then back to the install form page.
How about adding a simple link to the install form page within that message box after installing a module (the one that says "enable newly installed module" etc.)?
I would write a patch for that. But how would I do this?
Comment | File | Size | Author |
---|---|---|---|
#2 | install_another_module_link-1446650-2.patch | 516 bytes | superspring |
#1 | install_another_module_link-1446650-1.patch | 599 bytes | superspring |
Comments
Comment #1
superspring CreditAttribution: superspring commentedThis patch adds the link to the 'install complete' review screen.
Comment #2
superspring CreditAttribution: superspring commentedThis patch adds the same link to the D8 'install complete' review screen.
Comment #3
dwwA more far-reaching solution is being explored at #720452: Allow ability to install multiple modules at once. However, that might not be done and committed before freeze, so I think this is a reasonable (and trivial) improvement to make as a back-up in case #720452 never makes it in. We can always remove this line of code again as part of #720452.
Thanks!
-Derek
Comment #4
benjifisher+1 for RTBC. That is almost exactly the way I would do it; the only difference is that I would put the new "Install another module" link second, rather than first. Thanks to @dww for noticing the duplication of effort.
IMHO we can keep this change even if we also commit #720452: Allow ability to install multiple modules at once.
Comment #5
catchLooks sensible. Committed/pushed to 8.x.
Comment #6
benjifisherI hope I have the procedure right. Now that it has been committed to 8.x, we can consider making the same change to 7.x. I think the patch in #1 was written for D7, so I will queue it for re-testing.
Comment #7
benjifisher#1: install_another_module_link-1446650-1.patch queued for re-testing.
Comment #8
benjifisherThe patch in #1 still works for Drupal 7.x. I am setting the status to "patch (to be ported)" again.
Comment #9
Josh Waihi CreditAttribution: Josh Waihi commented#1 is RTBC IMO
Comment #10
David_Rothstein CreditAttribution: David_Rothstein commentedCommitted to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/50f9d3f
(Not sure if I should have committed it since this is a feature request and we are currently over thresholds, but it's been sitting here for a while, and in any case, for very small Drupal 7 features I no longer see the point of caring about the thresholds so I'm not really going to.)
Note that during commit, I also fixed the documentation (docblocks need to start with a verb, so I added "Returns a" at the beginning). Drupal 8 already had different documentation here anyway.
Comment #11
David_Rothstein CreditAttribution: David_Rothstein commentedI also went ahead and filed the obvious followup issue here: #1862094: After installing a theme, display a link to "install another theme"
Comment #12
David_Rothstein CreditAttribution: David_Rothstein commentedDrupal 7.18 was a security release only, so this issue is now scheduled for Drupal 7.19 instead.
Fixing tags accordingly.
Comment #14
David_Rothstein CreditAttribution: David_Rothstein commentedDrupal 7.19 was a security release only, so this issue is now scheduled for Drupal 7.20 instead.
Fixing tags accordingly.
Comment #15
David_Rothstein CreditAttribution: David_Rothstein commentedDrupal 7.20 was a security release only, so this issue is now scheduled for Drupal 7.21 instead. For real this time... I think :)
Fixing tags accordingly.
Comment #16
David_Rothstein CreditAttribution: David_Rothstein commentedFixing tags since Drupal 7.21 only contained a fix to deal with fallout from the Drupal 7.20 security release. Hopefully this is really the last time I do this and this will be released in Drupal 7.22 for real :)
Comment #17
benjifisherI marked #1012850: Add "install another new module" link to "Next steps" after a new module has been installed as a duplicate of this and #1862094: After installing a theme, display a link to "install another theme".