Implement the new _uninstall hook for decisions.install per http://drupal.org/node/64279#uninstall

This should:
1. Drop the decisions table.
2. Delete all decisions nodes.
3. Delete all votes, options, and cached results via VotingAPI.

This should be accomplished by first implementing the hook to delete this info when a node is deleted, and then to add #1 and #2 to _uninstall().

CommentFileSizeAuthor
#2 decisions_uninstall.patch1.98 KBChrisKennedy

Comments

ChrisKennedy’s picture

ChrisKennedy commented

One other area to add:
4. Delete any drupal variables we have created.

ChrisKennedy’s picture

ChrisKennedy commented
Status: Active » Needs review
StatusFileSize
new decisions_uninstall.patch1.98 KB

Here's an initial implementation. It drops the tables and deletes the drupal variables, but does not yet delete decisions data in the VotingAPI tables.

anarcat’s picture

anarcat commented

This patch was committed by Chris, and I have committed a patch that removes the votes and that needs testing.

anarcat’s picture

anarcat commented
Priority: Normal » Critical

Make this appear higher on the list: we don't want to screw up things needlessly.

A warning should appear when deinstalling decisions to make clear that the decisions will be destroyed.

anarcat’s picture

anarcat commented
Assigned: Unassigned » anarcat
Status: Needs review » Fixed

*ugh*... there *is* already a warning before the hook_uninstall is ran, so that is done... I have just committed the last bit of this to remove the nodes from the node table, so considering this as fixed.

Anonymous’s picture

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