[Tickets #11362] Re: Mobile view (smart phone): Delete not working

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/11362
------------------------------------------------------------------------------
  Ticket             | 11362
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Mobile view (smart phone): Delete not working
  Queue              | IMP
  Version            | Git develop
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2012-08-23 15:51) wrote:

>>> I just noticed the "Delete" button is not working in the smartphone
>>> view: It jumps back to the overview but the message stays in the
>>> folder. "Copy" and "Move" work fine.
>>
>> Works for me.  There can be a delay if the response from the server
>> is slow (or no update if the response from the server is either never
>> received or is broken), so not sure if this is what you have run into.
>
> To me it looks like it doesn't handle the \Delete flag on parse.  
> Here's a tcpdump of a message delete + a refresh of the INBOX.
>
> *** Delete command ***
> 3 UID FETCH 10 (ENVELOPE)
> * 1 FETCH (UID 10 ENVELOPE ("Mon, 30 Jul 2012 09:02:50 +0200 (CEST)"  
> "Postmaster Warning: Delayed Mail" (("Mail Delivery System" NIL  
> "MAILER-DAEMON" "intranator.net.lan")) (("Mail Delivery System" NIL  
> "MAILER-DAEMON" "intranator.net.lan")) (("Mail Delivery System" NIL  
> "MAILER-DAEMON" "intranator.net.lan")) ((NIL NIL "postmaster"  
> "intranator.net.lan")) NIL NIL NIL  
> "<20120730070250.9DB4A17769 at intranator.net.lan>"))
> 3 OK Completed (0.000 sec)
> 4 UID STORE 10 +FLAGS.SILENT \deleted
> 4 OK Completed
>
> *** Refresh of INBOX (logout + login) ***
> ...
> 5 UID FETCH 28,26,18,17,16,15,14,13,12,11,10,27 (ENVELOPE FLAGS  
> RFC822.SIZE BODY.PEEK[HEADER.FIELDS (IMPORTANCE LIST-POST  
> X-PRIORITY)])
> ...
> * 10 FETCH (FLAGS (\Deleted \Seen) UID 26 RFC822.SIZE 1305 ENVELOPE  
> ("Sat, 18 Aug 2012 18:23:27 +0200" "" (("Thomas Jarosch" NIL  
> "thomas.jarosch" "intra2net.com")) (("Thomas Jarosch" NIL  
> "thomas.jarosch" "intra2net.com")) (("Thomas Jarosch" NIL  
> "thomas.jarosch" "intra2net.com")) ((NIL NIL "admin"  
> "intranator.net.lan")) NIL NIL NIL  
> "<20120818182327.Horde.VnBhMQhKP71QL8F-w5gRAFA at intranator.net.lan>")  
> BODY[HEADER.FIELDS (IMPORTANCE LIST-POST X-PRIORITY)] {2}
>
>
> Should it mark deleted messages in a special way?
> I would expect them to be hidden since there is no way to expunge them, too.

No.  Messages in the smartmobile mode are not distinguished by any  
flag except for seen/unseen.  So deleted messages, if trash mailbox is  
not used, will continue to remain in the INBOX.

And these messages *shouldn't* be hidden since this is not the default  
behavior in the other views.  We are not going to add a 'hide deleted'  
option in smartmobile either since that interface needs to be as  
simple as possible.