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:09:04 +0000 (15:09 -0800)
commit9554b4ccbf0783516ea1735a7c999919be33c280
tree3f73146fa0c5ed2bca7f01fb6e86a6d36b5a2e62
parented8553eea8b308ae0e686fdf96d3e209e83be28c
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