Skip to:
Content

BuddyPress.org

Opened 3 years ago

Closed 3 years ago

#8953 closed defect (bug) (fixed)

Make sure to display a 404 if a Group’s URL has no match in registered screens

Reported by: imath's profile imath Owned by: imath's profile imath
Milestone: 12.0.0 Priority: normal
Severity: normal Version:
Component: Groups Keywords: has-patch
Cc:

Description (last modified by imath)

Working on documenting the BP_Group_Extension class, I noticed the BP Rewrites API wasn't displaying a 404 if the current action had no corresponding group screen.

Change History (3)

#1 @imath
3 years ago

  • Description modified (diff)
  • Summary changed from Make sure to display a 404 if a Group’s (sub)nav misses a valid screen function to Make sure to display a 404 if a Group’s URL has no match in registered screens

This ticket was mentioned in ​PR #138 on ​buddypress/buddypress by ​@imath.
3 years ago #2

  • Keywords has-patch added; needs-patch removed

Introduces a check_parsed_query() method to the BP_Groups_Component class to make sure the requested group URL matches an existing group screen.

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/8953

#3 @imath
3 years ago

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

In 13525:

Display a 404 if a group’s URL has no matches in registered screens

Introduces a check_parsed_query() method to the BP_Groups_Component class to make sure the requested group URL matches an existing & registered group screen.

Fixes #8953
Closes ​https://github.com/buddypress/buddypress/pull/138

Note: See TracTickets for help on using tickets.