808490 - [EMAIL] When you send a email to yourself, you get two same emails in your Inbox

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[Walter Chen[:ypwalter][:wachen]]

*Environment:
Phone - Otoro
Gaia - 16fda982b02d7279abd1c15b66af25e692d502f3
Gecko - e070197f4c89

*How to reproduce:
1. Unlock the phone
2. Go to email
3. Choose others and enter a gmail account
4. Send a email to yourself

*Expected Result:
   Received one email in Inbox

*Actual Result:
   Received two same emails in Inbox

Comment 1

[Walter Chen[:ypwalter][:wachen]]

*Note: This is 11/04 PST Gecko Aurora Build

Comment 2

[John Shih]

Hi, I can't reproduce this
build info
2012-11-06
gaia master : a67126c5cf366cc2d635f7f5433a8363a3a93df4
mozilla-aurora (hg) : 66bada6ea6b2

can you check again?

Comment 3

[Walter Chen[:ypwalter][:wachen]]

Hi, 
I can't reproduce this bug. This bug is not highly reproductive. But, it happens.

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

It's possible two click events were being generated and we processed both.  This should happen less frequently now that we have the modal sending dialog which should reduce the time window during which the click can be dispatched and result in an additional send.

Since we want the send button to be more than one-shot since we may need to retry to send, we should probably just set a boolean flag when we are nominally sending the message and bail if we receive an additional click while the send is active.

Comment 6

[Andrew Sutherland [:asuth] (he/him)]

I believe I have identified the problem.  Until a new patch lands, we try and always put the sent message in the sent folder of the IMAP account.  This is right on normal SMTP/IMAP servers, but wrong on gmail.  Since the gmail SMTP server automatically puts outgoing messages in the sent folder, we get a duplicate.  The patch for this is currently a ride-along on the fix for bug 804977.  I'll fix this bug when I fix that one assuming the patch gets authorized.

Comment 7

[Walter Chen[:ypwalter][:wachen]]

What is the status of this one?

Comment 8

[Andrew Sutherland [:asuth] (he/him)]

Bug 804977 got fixed but I forgot to mark this bug fixed.  The problem should be resolved, please reopen and attach the relevant "adb logcat | grep Gecko" output from the start of the composition process until after the send process completes plus another 30 seconds or so.