the user has to manually click the mail folder to refresh it

Bug #671586 reported by Allen Lowe
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Postler
Fix Released
High
Allen Lowe

Bug Description

the user has to manually click the mail folder to refresh the message list. Postler should refresh automatically as changes are pulled from the server

Allen Lowe (lallenlowe)
Changed in postler:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Allen Lowe (lallenlowe) wrote :

here is a patch that uses a gio FolderMonitor to auto refresh the message list of the currently selected folder.

Changed in postler:
assignee: nobody → Allen Lowe (lallenlowe)
status: Confirmed → In Progress
Cris Dywan (kalikiana)
Changed in postler:
status: In Progress → Fix Committed
Revision history for this message
Allen Lowe (lallenlowe) wrote :

I just realized that I setup the FileMonitor to monitor the "cur/" directory in any given mailbox, it also needs to monitor "new/" right now the inbox isn't refreshing properly because of that.

Changed in postler:
status: Fix Committed → In Progress
Revision history for this message
Allen Lowe (lallenlowe) wrote :

I just created a second patch that adds a FileMonitor for the "new/" directory also, here it is.

Revision history for this message
Cris Dywan (kalikiana) wrote :

So you basically wrote the same code once more. I would actually prefer a small function that takes a path name and a callback (delegate) to avoid the duplication and make the code clearer.
Also thinking of unified folders in the future, we could re-use the function to monitor any number of folders at once.

Revision history for this message
Allen Lowe (lallenlowe) wrote : Re: [Bug 671586] Re: the user has to manually click the mail folder to refresh it

Yeah, sorry about that. I would be happy to refactor that code in to a function. But I may not get to it until this weekend or possibly even later as Dexter has been taking up my time.
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

Revision history for this message
Allen Lowe (lallenlowe) wrote :

I am going to try to get to this tonight (Friday, November 12 2010), if all goes as planned.

Revision history for this message
Allen Lowe (lallenlowe) wrote :

ok, here's a patch that supersedes the second patch on this page (as that patch was never used). I have re-factored the file monitoring into a simple function, as requested.

Revision history for this message
Allen Lowe (lallenlowe) wrote :

ok, here's a patch that supersedes the second patch on this page (as that patch was never used). I have re-factored the file monitoring into a simple function, as requested.

Cris Dywan (kalikiana)
Changed in postler:
status: In Progress → Fix Committed
Cris Dywan (kalikiana)
Changed in postler:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.