Skip to:
Content

bbPress.org

Opened 11 years ago

Last modified 10 years ago

#2431 new defect (bug)

Missing uninstall hooks and function calls

Reported by: offereins's profile 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
11 years ago

  • Cc lmoffereins@… added

#2 @johnjamesjacoby
11 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
11 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
10 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.