systopia / de.systopia.fastactivity

CiviCRM Extension for high performance activity features
Other
4 stars 8 forks source link

Fix Paginator skipping activities in some edge case #40

Closed pfigel closed 3 years ago

pfigel commented 3 years ago

This fixes an issue where the paginator skips over the last activities under certain circumstances when the row count is close to the per-page limit.

I've also included bee2bb527a1db2979f4faebb227667df2d28b7c7, which is a minor change that enforces consistent activity_id-based sorting on top of existing ORDER BY options that effectively falls back on the INSERT order for activities with the same activity_date_time.

bjendres commented 3 years ago

@pfigel Analysing the changes would be a lot of work - but I trust you on this :)