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 Owned by: imath
Priority: normal Milestone: 12.0.0
Component: Groups Version:
Severity: normal 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 Make sure to display a 404 if a Group’s (sub)nav misses a valid screen function → 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 → fixed
  • Status assigned → 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.