Group Types Default Pagination Broken

[reubenlillie]

Expected: Pagination links appearing for group type listings with over "n" results (n = 20 by default) would navigate to their respective pages.
Actual: Links appear active (e.g., cursor changes on focus/hover, href exists), but both next-page and numbered links fail to load. That is, it is currently impossible for users with a default BP installation to navigate to groups listed beyond the "nth" result when viewing by group type.
Query issue?
Template issue?
NB: Functionality works as expected when viewing "All Groups" listing but not when viewing by "group type" (2.6+).

[r-a-y]

This is a legit bug that only occurs if there is more than one page of groups on a group type directory.

There are two issues:

1. In the BP_Groups_Component::setup_globals() method, we throw a 404 for group pages that is not a single group page or the group creation page. This meant that the AJAX call was throwing a 404.
2. Currently, group type directory detection occurs on frontend screens only and not during AJAX. We need to move the group type directory detection into the BP_Groups_Component class so this can be picked up by AJAX.

Patch fixes this both up. Was an oversight on my part!

Would love to squeeze this in for BP 2.8, since this bug was introduced in 2.7.

[dcavins]

Tests for group directory by type

[dcavins]

@r-a-y I wanted to add some tests for the correct behavior. I've no idea why the last two new tests are failing. (bp_current_action() is returning type, even though I can verify in an actual install that the behavior is correct. Maybe it's a quirk with go_to? Any ideas?)

[r-a-y]

Maybe it's a quirk with go_to? Any ideas?)

This. We only run 'bp_init' on the go_to() method because we can't really test for redirections using wp_redirect(). If we do try to run the 'bp_template_redirect' hook in the go_to() method, PHPUnit will crap out.

Instead of redirecting, we could change invalid group type requests to 404 instead. That way, we would be in line with what we are currently doing with invalid member type requests.

[r-a-y]

02.patch switches out the invalid group type redirection and just 404s invalid group types.

I moved the unit tests written by @dcavins to /routing/groups.php and added some 404 tests as well.

[DJPaul]

Latest patch looks good to me

[boonebgorges]

Approach looks good to me too.

[r-a-y]

In 11405:

Groups: Fix AJAX pagination when on a group type directory page.

This commit moves group type directory detection from the
'template_redirect' hook to our groups component class to ensure
detection is available for AJAX purposes.

Commit also changes invalid group type directory requests to 404 instead
of redirecting back to the main Groups directory.

Props r-a-y, dcavins.

Fixes #7423.

[boonebgorges]

In 11408:

After [11405], be sure member types are reset between group routing tests.

See #7423.