Groups Paginated Wire List hrefs may be wrong, and not work correctly.

[valilenk]

In groups pagination links looks like that: "http:/?wpage=2"

In profiles it's fine: "​http://site/members/nickname/?wpage=2"

[Jason_JM]

I don't have errors with this though I think it may have to do with rewrite rules.

I'm using NginX (as is buddypress.org I believe), your mileage may vary.

Stock implementation (without using the supplied patch) works fine on my end.

[Jason_JM]

Flagged minor for now, we need more info.

[boonebgorges]

Weird that you don't see the problem, Jason_JM. I was definitely experiencing it without the patch. The version of bp_wire_posts_template in trunk builds the pagination URLs using $bp->displayed_user->domain, which is null for group pages, and even if it weren't null it wouldn't make for the right URL.

[Jason_JM]

I'm with you, it's wierd. There is no way i'm down playing this ticket though. Just need more info and I'd like for the rest of the developers to chime in too.

Now, I use nginx i probably have a re-write for it. I have to check tonight.

The buddypress.org home uses nginx too I believe (as does wordpress.com) and you can see that the URLs there are working. This was the first thing I checked. I agree with you however, that the testbp site is not working. I confirm that for sure.

But check out the org page: http://buddypress.org/groups/bp-events/wire/

[jason_jm]

I need some server info, going to hit up Barry.

[jason_jm]

Got the word.
buddypress.org uses lightspeed
testbp.org uses apache.

Weird indeed...

[johnjamesjacoby]

Because the wire can be "inherited" by other plugins, it will need to be smarter than just looking at profiles and blogs. The events and album plugins are good examples.

The included patch will fix the current core but isn't future proof quite yet.

[jason_jm]

Going to try and test this tonight

[apeatling]

Not valid anymore in trunk and 1.2