Invalid URL routing for forum profile subnav item

[DJPaul]

On trunk:

​http://testbp.org/members/this_is_made_up/ 404s correctly.
​http://testbp.org/members/this_is_made_up/profile/ does not 404; it renders a lot of the template and generates a lot of PHP Notices.

[djpaul]

(In [4281]) Fix handling of certain subnav URLs in core, xprofile, friends, members, activity, settings, blogs and message components. See #3176

[DJPaul]

Groups and forums components need patching differently because of the way they use the displayed_user (or don't).
/members/admin/forums also seems to render the forums directory template at the moment, which needs fixing first.

[r-a-y]

(Edit) The patch in #3246 will fix the problem noted in the original ticket description.

It doesn't fix /members/INVALID/forums/.

Paul, can you list an example for groups?

[DJPaul]

See #3246

[r-a-y]

#3246 doesn't fix /members/INVALID/forums/. It loads the forum directory like Paul noted above.

I'll take a look at this in a bit.

[r-a-y]

A potential solution involves checking the $bp_unfiltered_uri global in bp_forums_directory_forums_setup().

eg.

function bp_forums_directory_forums_setup() {
	global $bp, $bp_unfiltered_uri;

	if ( bp_is_forums_component() && !isset( $bp_unfiltered_uri[1] ) && ( !bp_current_action() || 'tag' == bp_current_action() ) && !bp_current_item() ) {
		... the rest of the function ...

I'm using the $bp_unfiltered_uri check here because bp_is_directory() isn't true until later in the function.

Did not provide a patch because I'm not sure if the core devs like this approach.

[DJPaul]

Looks good / works for me. I'll put it in

[djpaul]

(In [4501]) Stop forum directory template loading for invalid member forum URLs. Fixes #3176, props r-a-y