Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#4029 closed defect (bug) (fixed)

Template screens and actions are hooked to 'wp' instead of 'template_redirect'

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 1.6 Priority: highest
Severity: major Version:
Component: Core Keywords:
Cc:

Description

Back when we audited the actions in 1.5, we kept some things hooked on to 'wp' for future investigation. Now that all of the hooks have been consolidated, we can safely move things onto dedicated template_include and template_redirect actions where needed.

Change History (2)

#1 @johnjamesjacoby
13 years ago

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

(In [5813]) Move bp_screens and bp_actions onto bp_template_redirect action, which is hooked to template redirect.

  • Moves template redirecting where it belongs, the template_redirect action
  • Leaves bp_ready hooked to 'wp' for now, pending testing
  • Anything hooked to bp_ready works the way it always did
  • Anything properly hooked to bp_actions and bp_screens works normally
  • Fully backwards compatible
  • Fixes #4029

#2 @johnjamesjacoby
13 years ago

(In [5817]) Introduce _bp_maybe_remove_redirect_canonical() as a stop-gap to prevent WordPress from confusing BuddyPress faux URLs with existing rewrite rule matches. See #4029.

Note: See TracTickets for help on using tickets.