Since the 3.x branch of Mail System now owns its own MailSystem class, we should try to ensure that the module remains enabled while MailsystemDelegateMailSystem is an available mail class.

Trying to clean up the variable in hook_disable() is probably a non-starter, for the same reasons as explained in #1427612: Variables are not deleted on uninstall of module cause site email to break. Instead, the attached patch uses to hook_system_info_alter() to mark the module as required until the user adjusts the Mail System configuration to the point where MailsystemDelegateMailSystem isn't being used anymore.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Les Lim’s picture

Issue summary: View changes
Status: Needs review » Fixed
FileSize
1014 bytes

Made some minor wording changes, and committed the attached patch to 3.x.

  • Les Lim committed 054fbba on 7.x-3.x
    Issue #1897972 by Les Lim: Added Prevent module disable while '...

Status: Fixed » Closed (fixed)

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