chess_uninstall() function is missing from chess.install. SQL tables should should be DROP'd. Content type modifications should be reversed.

CommentFileSizeAuthor
#2 chess.install.zip1.68 KBEoinBailey

Comments

csko’s picture

csko commented
Status: Needs review » Active
EoinBailey’s picture

EoinBailey commented
Assigned: Unassigned » EoinBailey
StatusFileSize
new chess.install.zip1.68 KB

Yep, working on this. Silly not to include it. Sorry about that. The next version will have the uninstall included.

The attached file has it included (this is my first hook_uninstall, let me know if it's missing anything!).

EoinBailey’s picture

EoinBailey commented
Status: Active » Fixed

0.06 has an uninstall.

BUT... if you haven't used the uninstall from the patch above first, you will have to manually remove the tables {chess_games} and {chess_info}, as well as delete the content type chess before installing 0.06.

I rewrote the code so it doesn't depend on CCK anymore, it creates its own chess node type, and the tables have been changed very significantly. I'm hopeful though, that from now on, I will have updates in the releases (once I work out how to do it).

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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