Legacy: Activity stream's "Load More" button doesn't work

[r-a-y]

If you have the Legacy template pack enabled, the activity stream's "Load More" button does not work.

See:
​https://buddypress.org/support/topic/activity-load-more-not-working/
​https://buddypress.org/support/topic/bp-activity-oldestpage-pagination-cookie-bug-on-page-reload/

The problem is due to the bp-activity-oldestpage cookie being removed from initialization in #7699.

I'm not sure how to address this without either reintroducing the cookie (which we don't want to do) or adding some changes to the javascript.

[r-a-y]

01.patch always purges the bp-activity-oldestpage cookie, which will enable the "Load More" button to work again.

This allows for the original intention behind #7699 to still be respected (kinda).

[DJPaul]

This is a hack. As it removes a cookie, it's not re-introducing creation of the cookie all the time, so it's not against the idea with #7699. If this works, go for it.

We need browser tests to capture these sorts of regressions, and that's something I hope to spearhead through the next few months.

[DJPaul]

If this cookie is basically reset on each subsequent page load, how about just switching the data storage to a data attribute on the form element somewhere? (Nouveau only - we leave Legacy as it is).

[boonebgorges]

Replying to DJPaul:

If this cookie is basically reset on each subsequent page load, how about just switching the data storage to a data attribute on the form element somewhere? (Nouveau only - we leave Legacy as it is).

I like this idea, and I think we should do it in Legacy as well, for GDPR-related reasons. See #7856.

[boonebgorges]

7896.02.patch​ converts it to a simple JS variable. Any reason why this won't work?

[r-a-y]

02.patch works well. Going to commit it!

[r-a-y]

In 12181:

Legacy: Fix activity stream's "Load More" button.

In the bp-legacy template pack, the "Load More" button used to be tied to
the bp-activity-oldestpage cookie. In v3.0, we made steps to remove
this cookie, but this broke activity pagination when used via AJAX (see

This commit outright removes dependencies on the bp-activity-oldestpage
cookie and uses a local variable to track the current activity pagination
number.

Props boonebgorges.

Fixes #7896 (3.x branch).

[r-a-y]

In 12182:

Legacy: Fix activity stream's "Load More" button.

In the bp-legacy template pack, the "Load More" button used to be tied to
the bp-activity-oldestpage cookie. In v3.0, we made steps to remove
this cookie, but this broke activity pagination when used via AJAX (see

This commit outright removes dependencies on the bp-activity-oldestpage
cookie and uses a local variable to track the current activity pagination
number.

Props boonebgorges.

See #7896 (trunk).