Skip to:
Content

Opened 5 years ago

Last modified 4 years ago

#2431 new defect

Missing uninstall hooks and function calls

Reported by: Offereins Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: trunk
Component: Tools Keywords:
Cc: lmoffereins@…

Description

Using 2.4, currently bbp_uninstall() creates a dedicated hook in core/sub-actions.php, but the function itself isn't hooked anywhere. It's only the phpDoc of bbp_activation() that reminds us vaguely of a former call to register_uninstall_hook(). Additionally the function bbp_do_uninstall has a similar problem, as well as bbp_delete_user_options. I tried to find any other 'uninstall' functions, terms or comments in 2.4, but couldn't find any.
I do not tend to uninstall bbPress, but I noticed when browsing the source.

Change History (4)

#1 @Offereins
5 years ago

  • Cc lmoffereins@… added

#2 @johnjamesjacoby
5 years ago

  • Component changed from General to Tools
  • Milestone changed from Awaiting Review to 2.7

Thanks for the reminder here. We'll probably try to work this into a future release (2.7?)

Ultimately, we may end up needing to go the uninstall.php route: http://codex.wordpress.org/Function_Reference/register_uninstall_hook#uninstall.php

#3 @Offereins
5 years ago

Why can an incomplete uninstall procedure live on for two more releases? Can you explain the rationale behind those missing pieces and why not to fix it sooner?

#4 @johnjamesjacoby
4 years ago

  • Milestone changed from 2.7 to 2.8

Bumping all 2.7 to 2.8 milestone.

Note: See TracTickets for help on using tickets.