Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi, good to hear that new version is on!
Btw, I just noticed that term_display_uninstall()
is missing in a new version of .install file?
Is there a reason for it or simply forgotten? imho, it's good to have it for modules...
Comment | File | Size | Author |
---|---|---|---|
#2 | term-display-add-uninstall.patch | 868 bytes | rstamm |
Comments
Comment #1
joachim CreditAttribution: joachim commentedCripes, yes there should be one, as we're adding a table in the hook_install so we need to clean up after ourselves!
I'll do a patch soon unless anyone beats me to it :)
Comment #2
rstamm CreditAttribution: rstamm commentedAdded hook_uninstrall
Comment #3
nedjo@Ralf Stamm: Thanks for the patch. I've assigned you CVS access. Please go ahead and apply the patch. Please always apply to HEAD first and then to any relevant branch--in this case, DRUPAL-5. Thanks!
Comment #4
rstamm CreditAttribution: rstamm commented@nedjo: Thanks
Committed to HEAD and DRUPAL-5.