Skip to:

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#3173 closed enhancement (fixed)

Decouple bbp_admin_reset_handler handler

Reported by: espellcaste Owned by: johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version: trunk
Component: Tools - Code Improvements Keywords: has-patch


Right now, bbp_admin_reset_handler() cannot be used on CLI contexts because of its checks.

I didn't want to duplicate its functionality on, so I decoupled this function to allow its usage there.

After this change is merged, I can call bbp_admin_reset_bbpress() on CLI.

Attachments (1)

3173.diff (1.3 KB) - added by espellcaste 3 years ago.

Download all attachments as: .zip

Change History (5)

3 years ago

#1 @espellcaste
3 years ago

Relate to #3058.

This ticket was mentioned in Slack in #bbpress by espellcaste. View the logs.

3 years ago

#3 @johnjamesjacoby
3 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 6750:

Admin: split reset handler into 2 functions (handler and logic)

This change improves compatibility with future WP CLI initiatives, specifically allowing removing all bbPress data.

Fixes #3173. Props espellcaste.

#4 @johnjamesjacoby
3 years ago

  • Milestone changed from Awaiting Review to 2.6
Note: See TracTickets for help on using tickets.