nettika/geary
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make App.ReseedOperation alway check the window for re-filling

Browse source 
When the remote re-opens, new messages may have become available that
could be added to the conversation set if the window is not yet full.
Thus always check whether a fill operation should be queued.
This commit is contained in:
Michael Gratton 2019-08-26 13:28:10 +10:00 • committed by Michael James Gratton
parent 52f0817116
commit 006b4c5ca5
1 changed files with 7 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/engine/app/conversation-monitor/app-reseed-operation.vala
View file

@ -22,10 +22,7 @@ private class Geary.App.ReseedOperation : ConversationOperation {
}
public override async void execute_async() throws Error {
// Clear the fill flag since more messages may have appeared
// after coming online.
this.monitor.fill_complete = false;
// Check for and load any newly appended messages
EmailIdentifier? earliest_id = this.monitor.window_lowest;
if (earliest_id != null) {
debug("Reseeding starting from Email ID %s on opened %s",
@ -38,11 +35,13 @@ private class Geary.App.ReseedOperation : ConversationOperation {
int.MAX,
Folder.ListFlags.OLDEST_TO_NEWEST | Folder.ListFlags.INCLUDING_ID
);
} else {
// No conversations are present, so do a check to get the
// side effect of queuing a fill operation.
this.monitor.check_window_count();
}
// Clear the fill flag since more messages may have appeared
// after coming online, and do a check to get them filled if
// needed.
this.monitor.fill_complete = false;
this.monitor.check_window_count();
}
}