Similar to http://drupal.org/node/93212. Submitting a separate issue as per chx's request.

When disabling multiple modules, mymodule1 and mymodule2, you can access mymodule1 hooks from mymodule2_disable(). This patch uses the same method as the above issue. I tested the behavior with two sample modules and this patch successfully refreshes the module list cache after each disable.

CommentFileSizeAuthor
#1 module_disable_0.patch4.15 KBJaza
module.disable.patch1.64 KBRobRoy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jaza’s picture

Jaza CreditAttribution: Jaza commented
FileSize
module_disable_0.patch4.15 KB

New patch that uses the same approach as the committed patch at http://drupal.org/node/93212 for improved efficiency. Tested for disabling modules, and it works the same as the original patch.

Note: this patch includes adding a $refresh parameter to the module_implements() function, which is also needed (not critically, it seems - since things work right now without errors) by the code that was committed in http://drupal.org/node/93212

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Fixed

Committed to HEAD.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)