Earnie Boyd <earnie@users.sourceforge.net> writes:
+1 Cascade.
Better use by all modules for the uninstall API is needed.
I cannot parse this sentence.
I can see the points for not uninstalling when disabled but I am upset that no Uninstall is available for any module enabled that adds tables or columns to existing tables.
We could certainly declare hook_uninstall() to be mandatory for any module that declares hook_install(); now that we have drupal_uninstall_schema(), hook_uninstall() is usually trivial (actually, it always was trivial). For that matter, we could provide a default hook_uninstall() that drops all the module's tables. I think the new variable patch also identifies variables by module so we could uninstall them automatically as well. This has nothing really do to with foreign keys, though. Thanks, Barry