Now that #253569: DX: Add hook_modules to act on other module status changes has landed, module developers no longer have to remove their own schema on uninstall.

We can move this into system_modules_uninstalled()

See http://api.drupal.org/api/function/hook_modules_uninstalled/7 for details.

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Title: DX: Remove module table automatically » DX: Remove module tables automatically on uninstall
matt2000’s picture

matt2000 commented

According to the documentation, this is already done?

http://api.drupal.org/api/function/hook_uninstall/7

dmitrig01’s picture

dmitrig01 commented
Status: Active » Fixed

Line 568 of install.inc.

function drupal_uninstall_modules($module_list = array()) {
  foreach ($module_list as $module) {
... 
   drupal_uninstall_schema($module);

Status: Fixed » Closed (fixed)

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