Inbox::streamNotices() with deletion compensation: inbox paging should more or less...
authorBrion Vibber <brion@pobox.com>
Thu, 4 Feb 2010 22:50:20 +0000 (14:50 -0800)
committerBrion Vibber <brion@pobox.com>
Thu, 4 Feb 2010 23:15:42 +0000 (15:15 -0800)
commitbb16898b1c73073e0442de72f2af133a3bd39713
treec9079487b6ee4c79dcefeeaa05a03c016417b970
parent37f3a3d558ba55a085c9ee5427948b572c197bc3
Inbox::streamNotices() with deletion compensation: inbox paging should more or less work with deleted items now.
No change in efficiency for the common case where nothing's deleted: does the same bulk fetch of just the notices we think we'll need as before, then if we turned up short keeps checking one by one until we've filled up to our $limit.

This can leave us with overlap between pages, but we already have that when new messages come in between clicks; seems to be the lesser of evils versus not getting a 'before' button.
More permanent fix for that will be to switch timeline paging in the UI to use notice IDs.
classes/Inbox.php
classes/User.php