Skip to:
Content

bbPress.org

Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#3173 closed enhancement (fixed)

Decouple bbp_admin_reset_handler handler

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

Description

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 https://github.com/bbpress/wp-cli-bbpress, 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 7 years ago.

Download all attachments as: .zip

Change History (5)

@espellcaste
7 years ago

#1 @espellcaste
7 years ago

Relate to #3058.

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


6 years ago

#3 @johnjamesjacoby
6 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
6 years ago

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