[buddypress-trac] [BuddyPress Trac] #6234: Multiple scopes in activity-loop.php doesn't work correctly when activity auto-refresh is checked

[buddypress-trac]

#6234: Multiple scopes in activity-loop.php doesn't work correctly when activity
auto-refresh is checked
-----------------------------------------+------------------------------
 Reporter:  bphelp                       |       Owner:
     Type:  defect (bug)                 |      Status:  new
 Priority:  normal                       |   Milestone:  Awaiting Review
Component:  Component - Activity         |     Version:  2.2.1
 Severity:  normal                       |  Resolution:
 Keywords:  has-patch reporter-feedback  |
-----------------------------------------+------------------------------
Changes (by imath):

 * keywords:   => has-patch reporter-feedback


Comment:

 6234.patch is a first approach to reply to @bphelp need.

 But i can imagine potential issues, as :
 - i'm playing with the $_POST variable to be sure the activity-loop.php
 template will not output the ul & form elements
 - i rely on a template hook `bp_after_activity_entry` to get the activity
 last recorded time displayed in the activity-loop.php template. If a
 specific theme is removing this action, it will break the process :(

 @bphelp could you test the patch to confirm it's solving your issue ?

 @boonebgorges : i'm going to explore
 > Maybe bp_has_activities() can include in its return value an array of
 parameters that it used to retrieve items; then we could print these as a
 JS variable using wp_localize_script(), and heartbeat could use them to
 retrieve the next items. The more I think about it, the worse this idea
 seems, but I thought I'd throw it out there :)

 It seems to be a better plan imho ;)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6234#comment:11>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac