Hello,

Hook_uninstall() is still having Drupal 6 functions, when you try to uninstall it from Drupal 7, guess what, it crashes ;-)

CommentFileSizeAuthor
#2 0001-1494560-Fix-error-in-hook_uninstall.patch972 bytespol
#1 0001-1494560-Fix-error-in-hook_uninstall.patch971 bytespol

Comments

pol’s picture

pol
Location Brussels
commented
StatusFileSize
new 0001-1494560-Fix-error-in-hook_uninstall.patch971 bytes

Here's the patch for 7.x-1.x.

pol’s picture

pol
Location Brussels
commented
StatusFileSize
new 0001-1494560-Fix-error-in-hook_uninstall.patch972 bytes

Oops !

Fix a typo ($result => $results)

pol’s picture

pol
Location Brussels
commented
Status: Active » Needs review
zzolo’s picture

zzolo commented
Status: Needs review » Fixed

Hey @Pol. Thanks for the catch and the quick patch. I already fixed it and committed. I am using the $conf variable instead of direct DB call. I make a new release.

pol’s picture

pol
Location Brussels
commented

Okay ;-)

Status: Fixed » Closed (fixed)

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