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

buddypress-trac noreply at wordpress.org
Wed Mar 4 18:58:58 UTC 2015


#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  |
-----------------------------------------+------------------------------

Comment (by boonebgorges):

 The `$_POST` thing is such an awful hack. Not your fault - it's
 embarrassing that we do this check in the template. I assume this is for
 AJAX pagination or something? In a future version - maybe with a new
 template pack - we should stop doing this, maybe by having separate
 template parts for AJAX vs non-AJAX. (Or by fetching activity items via an
 API and building the page in JS - a man can dream!)

 The `bp_after_activity_entry` part I'm not as worried about.

 Looking forward to seeing the results of your exploration into the other
 option. I suspect that it's going to be pretty ugly...

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


More information about the buddypress-trac mailing list