Loading more activities after an update displays last activity in double

[imath]

Hi,

I hope, i'll succeed to explain this minor "issue" in english!!

Let's say, there's 21 activities in database. When i go on Activity directory, i have 20 activities displayed and the load more link.
Now imagine, i post a new activity, i have 21 activities displayed and the load more link.
Now if i click on the load more link, the activity just before the load more link will be displayed again, and we get the same id in double (for instance : #activity-20)

So i thought, the easiest way to avoid this, is to "remove from display" the activity just before the load more link in case of a new update.

You'll find the javascript enhancement in the diff attached to this ticket

[imath]

4987.01.diff seems to be better. I've noticed in a plugin i'm working on that if the order of the query changes then the previous method is not working well.

[DJPaul]

imath; interested in getting this into 1.8?

The patch needs another look, if only because doing this is unsafe: :)

$qs[] = 'exclude=' . $_POST['exclude_just_posted']

[imath]

Replying to DJPaul:

imath; interested in getting this into 1.8?

Yes !! i'll check this tomorrow to make it safer ;)

[imath]

Just added 4897.02.diff (using wp_parse_id_list() to be sure ids are int)

[boonebgorges]

Great patch, imath. I've cleaned it up a bit, but the technique works well for me.

[boonebgorges]

In 7116:

Exclude dynamically added activity items when clicking Load More in directory

This prevents duplicate items appearing after Load More due to the numeric
offset caused by AJAX-inserted activity items.

Fixes #4897

Props imath

[imath]

Replying to boonebgorges:

Great patch, imath. I've cleaned it up a bit, but the technique works well for me.

Thanks @boone ;)