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?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

superspring’s picture

Status: Active » Needs review
FileSize
599 bytes

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

superspring’s picture

Version: 7.12 » 8.x-dev
FileSize
516 bytes

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

dww’s picture

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

benjifisher’s picture

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

catch’s picture

Status: Reviewed & tested by the community » Fixed

Looks sensible. Committed/pushed to 8.x.

benjifisher’s picture

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.

benjifisher’s picture

Status: Patch (to be ported) » Needs review
benjifisher’s picture

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.

Josh Waihi’s picture

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

#1 is RTBC IMO

David_Rothstein’s picture

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.

David_Rothstein’s picture

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

David_Rothstein’s picture

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.

David_Rothstein’s picture

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

Fixing tags accordingly.

David_Rothstein’s picture

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.

David_Rothstein’s picture

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

benjifisher’s picture