Picture currently implements hook_uninstall()in order to remove a small number of variables when it's uninstalled. However, the hook implementation is located in picture.module instead of picture.install which is in opposition to the hook documentation:

The uninstall hook must be implemented in the module's .install file.

Comments

devin carlson’s picture

Status: Active » Needs review
StatusFileSize
new845 bytes

A patch to move the hook implementation.

attiks’s picture

Assigned: Unassigned » jelle_s
jelle_s’s picture

Assigned: jelle_s » Unassigned
Status: Needs review » Fixed

Fixed in latest dev. Thanks for the patch!

Status: Fixed » Closed (fixed)

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