I added this as a comment on the "Drupal 6 port" issue because I was looking at it in the context of using the module under Drupal 6, but it applies equally well for Drupal 5. The uninstall hook invokes loginticket_delete in order to make sure that hook_loginticket is invoked before dropping the table {loginticket}. This code seems unneeded since the modules implementing hook_loginticket will declare a dependency on loginticket and thus will be disabled by the time loginticket can be uninstalled.

I've implemented a patch that takes out the logic around calling loginticket_delete.

CommentFileSizeAuthor
loginticket_uninstall.patch272 bytescrdant

Comments

crdant’s picture

Assigned: Unassigned » crdant
Status: Needs review » Closed (fixed)