Posting an activity reload the last activity into the stream

[imath]

Since the introduction of the "heartbeat check", posting an activity uses the offset argument to load the just posted activity and the activities that might have been posted by others in the meantime.

Offset is requesting ">=" and we're passing the last displayed id on the stream. As a result, this last displayed activity is reloaded.

I think it's something we forgot to implement when posting a new activity as we're adding +1 to the offset argument in bp_activity_heartbeat_last_recorded() function.

The patch simply adds +1 when requesting for activities once a new activity is posted in bp_legacy_theme_post_update()

[boonebgorges]

Sorry, this is my fault. Fix looks good.

[imath]

Replying to boonebgorges:

Sorry, this is my fault. Fix looks good.

No problem :) Thanks.

[imath]

In 8048:

Prevent the last displayed activity on stream to be reloaded after a posted update

Since the introduction of the heartbeat feature to automatically check for new activities, when posting an update: an offset argument is used to load all new activities. This offset is given by the last activity id displayed on the stream, to be sure that specific activity is not reloaded, we need to increment this id by 1.

Fixes #5447