Skip to:
Content

bbPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2386 closed defect (bug) (fixed)

bbp_is_deactivation() break the bbPress!

Reported by: alex-ye's profile alex-ye Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.4 Priority: normal
Severity: normal Version: 2.3.2
Component: General Keywords: needs-patch
Cc:

Description

When you pass the below GET query in any bbPress page in the front-end:

?action=deactivate&plugin=bbpress/bbpress.php

The page will borken, I don't know if this have a side effects or causing some another bugs, but this bug should be fixed!

In another topic, I am wondering if bbp_is_deactivation() still important.
What is the main purpose for it now ?!

Change History (3)

#1 follow-up: @johnjamesjacoby
11 years ago

  • Milestone changed from Awaiting Review to 2.4

Interesting. Moving to 2.4 to fix this right away.

These functions exist to execute or prevent code execution if bbPress is in the process of being activated or deactivated.

#2 @johnjamesjacoby
11 years ago

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

In 5076:

Add is_admin() and $pagenow checks to bbp_is_deactivation() and bbp_is_activation(). Hat tip alex-ye. Fixes #2386.

#3 in reply to: ↑ 1 @alex-ye
11 years ago

Replying to johnjamesjacoby:

These functions exist to execute or prevent code execution if bbPress is in the process of being activated or deactivated.

I see that, but what this worth?! What if the bbPress code executed in the process of being deactivated, What the bugs will be thrown?

Sorry for the many questions, This is the first time I see this thing in a WordPress plugin and I want to know more about it :)

Note: See TracTickets for help on using tickets.