The uninstall hook needs checked for this module, as it doesn't work due to a reference to oauth2_server_token_bundles() which isn't present at the state of the module being disabled.

This issue directly prevents the module from being uninstalled.

The uninstall function also does not include removal of the database tables that are added by the module.

CommentFileSizeAuthor
#3 oauth2_server-2573447-3.patch559 bytespjcdawkins

Comments

Nate_P created an issue. See original summary.

Nate_P’s picture

Nate_P commented
Issue summary: View changes
pjcdawkins’s picture

pjcdawkins commented
Status: Active » Needs review
StatusFileSize
new oauth2_server-2573447-3.patch559 bytes

Database tables are removed by default, if the uninstall works. This patch addresses the other bug.

othermachines’s picture

othermachines commented

Issue #2578879 is a duplicate but a better fix IMHO since it uses API function drupal_load().

pjcdawkins’s picture

pjcdawkins commented
Status: Needs review » Closed (duplicate)

OK... although I think there's no actual advantage in using the API function, it doesn't cost anything. Let's head over to that issue.

othermachines’s picture

othermachines commented

Total nitpick, I know. I probably wouldn't have noticed if there weren't two patches to choose from. Thanks much -