[buddypress-trac] [BuddyPress Trac] #5505: New activity check
buddypress-trac
noreply at wordpress.org
Sun Apr 6 18:54:55 UTC 2014
#5505: New activity check
--------------------------+-----------------------
Reporter: SGr33n | Owner: imath
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 2.0
Component: Activity | Version: 2.0
Severity: normal | Resolution:
Keywords: needs-patch |
--------------------------+-----------------------
Changes (by imath):
* keywords: reporter-feedback => needs-patch
Comment:
Replying to [comment:12 SGr33n]:
> Here it is :)
>
> http://youtu.be/Myx3A2O2V7I
Yeah, this shouldn't happen actually. I think the mention has nothing to
do with it. When a bbPress topic/reply is updated, bbPress is updating the
activity for this topic/reply. But it doesn't update the date of this
activity :
see
https://bbpress.trac.wordpress.org/browser/trunk/src/includes/extend/buddypress/activity.php#L434
The only way i've managed to reproduce was to change this line using
{{{bp_core_current_time()}}} instead of {{{get_post_time( 'Y-m-d H:i:s',
true, $topic_id )}}}
So i'm a bit amazed by the video, maybe there's a bbPress setting i've
missed that would automatically update the topic date in case it has been
edited..
So. If an activity for some reason has his date updated to current time
then all activity with a greater id will load again in the stream. Trying
to build a patch to avoid this.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5505#comment:13>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list