Make sure to display a 404 if a Members (sub)nav misses a valid screen function

[imath]

Working on documenting the BP_Component class, I noticed the BP Rewrites API wasn't displaying a 404 if a plugin was adding a members nav/subnav without a valid screen function. The Legacy URL parser was doing this fine.

To maintain this behavior in 12.0.0, we need to check the parsed query once the Members primary/secondary navigations are set to verify the current Members’ component action has a valid screen function.

Displaying a 404 if a Members (sub)nav misses a valid screen function is preserving what was happening in this case with the BP Legacy URL parser.

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

[imath]

In 13512:

Check the requested Member’s page matches a valid nav screen function

The BP_Members_Component::check_parsed_query() method happens once the query is parsed & once the single displayed Member's navigation is set. It checks the current action matches a navigation slug & whether the corresponding screen function exists and is callable. If it's not the case a 404 is displayed, just like it's currently the case when the BP Legacy URL parser is in use.

See #4954
Fixes #8932
Closes ​https://github.com/buddypress/buddypress/pull/125

[imath]

In 13534:

Restore the Member’s activity single view

A regression introduced in [13512] was preventing single activities to successfully load into the BP Nouveau Template Pack. We need to only perform the parse query check when viewing a single user. In the case of a single activity there is no need to check for an existing Member’s (sub)navigation item.

See #8932
Fixes #8956
Closes ​https://github.com/buddypress/buddypress/pull/143

[imath]

BP_Members_Component::check_parsed_query() is not taking care of not existing Member's main nav. In this case it should display a 404.

This case wasn't taken in account into previous patch/fix.

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

[imath]

In 13621:

Improve 404 management in BP_Members_Component::check_parsed_query()

• Make sure to do a 404 if corresponding Members primary navigation does not exist.
• Take in account Members navigations not generated by a BP Component.

Fixes #8932

Closed by ​https://buddypress.trac.wordpress.org/changeset/13621