Skip to:
Content

bbPress.org

Opened 13 years ago

Closed 13 years ago

#1580 closed defect (bug) (fixed)

bbp_logout_url() interferes with existing 'redirect_to' parameter

Reported by: r-a-y's profile r-a-y Owned by:
Milestone: 2.0 Priority: normal
Severity: normal Version: 2.0
Component: Front-end Keywords: has-patch
Cc:

Description

If there is an existing "redirect_to" parameter in the logout URL, bbp_logout_url() doesn't take this into account, it adds an additional "redirect_to" the logout URL.

Tested with the BuddyPress bp-default theme's logout link and bbP 2.0 beta 3.

---

Also fixes $redirect_to parameter for WP installs in a sub-directory by using wp_guess_url() instead of home_url( $_SERVERREDIRECT_URL? ).

eg.

before - localhost/bbpress/bbpress/REST_OF_LINK (invalid)
now - localhost/bbpress/REST_OF_LINK

Attachments (1)

1580.01.patch​ (1.5 KB) - added by r-a-y 13 years ago.

Download all attachments as: .zip

Change History (3)

@r-a-y
13 years ago

#1 @johnjamesjacoby
13 years ago

  • Milestone changed from Awaiting Review to 2.0

Nice catch! Confirmed.

#2 @johnjamesjacoby
13 years ago

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

(In [3387]) Allow previously set $redirect_to override in bbp_logout_url(). Fixes #1580.

Note: See TracTickets for help on using tickets.