Nouveau - Don't use AJAX to refresh directories when filtered by type

[r-a-y]

When navigating to the Members Directory filtered by type (example.com/members/type/ninja), clicking on the "All Members" tab would not show all members. Same thing happens with the Groups Directory.

This is an older problem we had with bp-legacy - #7291. To fix this, we need to apply a version of r11198 to Nouveau's JS so AJAX isn't used when clicking on a directory tab.

[r-a-y]

@imath - Could I get you to take a look at this patch to see if it is okay? Thanks!

[imath]

Hi @r-a-y

Thanks for pinging me about it. 7903.01.patch is fine, I've tested it and it works as expected. So you can use it, no problem.

But as a few lines above in buddypress-nouveau.js, there's already a 'no-ajax' available class to escape from AJAX, I would recommend to use it. I think it will make more sense for developers to understand why the page is reloaded in this case.

Moreover, I doubt the body class checks would work for Groups type, as so far this 'type' class is only added when a member type is set.

That's why I think 7903.02.patch:

• would make more sense for the 'no-ajax' thing,
• would take the group type in account
• could possibly take in account more components having a directory types (eg: blogs)

[DJPaul]

For changes in any files in either of the template packs, please change the @version tag in the file header to the release the change will be in (e.g. 4.0.0 in this case).

---

It is unclear to me what exactly a "component's type directory" means, so while that needs to be rephrased into something more under stable, the rest of the patch looks good (I have not tested).

[imath]

Hi @DJPaul,

Sorry for the late reply. You're right let's avoid this confusion. I think we don't need to introduce a new function to check if the page is a directory displaying members with a specific member type (or groups with a specific group type).

So I suggest 7903.03.patch to fix the issue.

[djpaul]

In 12224:

Nouveau: fix filtering Member Directory when specified by user type.

This is a problem we had with BP-Legacy's JS, and as it seems this part was
copied into Nouveau, we are implementing the same fix here. This involves not
using Javascript filtering for the Member Directory filters when a user type
is set.

A better fix would be to rewrite the Javascript to solve the underlying issue.

Fixes #7903

Props imath

[djpaul]

In 12225:

Nouveau: fix filtering Member Directory when specified by user type.

This is a problem we had with BP-Legacy's JS, and as it seems this part was
copied into Nouveau, we are implementing the same fix here. This involves not
using Javascript filtering for the Member Directory filters when a user type
is set.

A better fix would be to rewrite the Javascript to solve the underlying issue.

Fixes #7903 (3.0.x branch)

Props imath

[DJPaul]

Sorry @r-a-y - forgot to prop you here. My bad. ❤️