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...

CommentFileSizeAuthor
#2 term-display-add-uninstall.patch868 bytesrstamm

Comments

joachim’s picture

joachim commented

Cripes, 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 :)

rstamm’s picture

rstamm commented
Status: Active » Needs review
StatusFileSize
new term-display-add-uninstall.patch868 bytes

Added hook_uninstrall

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Title: Missing uninstall? » Missing uninstall
Status: Needs review » Reviewed & tested by the community

@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!

rstamm’s picture

rstamm commented
Status: Reviewed & tested by the community » Fixed

@nedjo: Thanks

Committed to HEAD and DRUPAL-5.

Status: Fixed » Closed (fixed)

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