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?

Files: 
CommentFileSizeAuthor
#2 install_another_module_link-1446650-2.patch516 bytessuperspring
PASSED: [[SimpleTest]]: [MySQL] 46,435 pass(es).
[ View ]
#1 install_another_module_link-1446650-1.patch599 bytessuperspring
PASSED: [[SimpleTest]]: [MySQL] 39,582 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new599 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,582 pass(es).
[ View ]

This patch adds the link to the 'install complete' review screen.

Version:7.12» 8.x-dev
StatusFileSize
new516 bytes
PASSED: [[SimpleTest]]: [MySQL] 46,435 pass(es).
[ View ]

This patch adds the same link to the D8 'install complete' review screen.

Component:user interface text» update.module
Status:Needs review» Reviewed & tested by the community

A 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

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

Status:Reviewed & tested by the community» Fixed

Looks sensible. Committed/pushed to 8.x.

Version:8.x-dev» 7.x-dev
Status:Fixed» Patch (to be ported)

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

Status:Patch (to be ported)» Needs review

Status:Needs review» Patch (to be ported)

The patch in #1 still works for Drupal 7.x. I am setting the status to "patch (to be ported)" again.

Status:Patch (to be ported)» Reviewed & tested by the community

#1 is RTBC IMO

Status:Reviewed & tested by the community» Fixed

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

I also went ahead and filed the obvious followup issue here: #1862094: After installing a theme, display a link to "install another theme"

Drupal 7.18 was a security release only, so this issue is now scheduled for Drupal 7.19 instead.

Fixing tags accordingly.

Status:Fixed» Closed (fixed)

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

Drupal 7.19 was a security release only, so this issue is now scheduled for Drupal 7.20 instead.

Fixing tags accordingly.

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

Issue tags:+7.22 release notes

Fixing 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 :)