Missing uninstall [#389730]

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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

joachim CreditAttribution: joachim commented 12 March 2009 at 22:13

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

Comment #2

rstamm CreditAttribution: rstamm commented 15 March 2009 at 09:11

Status:

Active

» Needs review

File	Size
term-display-add-uninstall.patch	868 bytes

Added hook_uninstrall

Comment #3

nedjo

he/him/his

English

CreditAttribution: nedjo commented 15 March 2009 at 15:40

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!