Closed Bug 1085411 Opened 11 years ago Closed 8 years ago

[Messages] We save Drafts too many times when it's not necessary

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: julienw, Unassigned)

Details

(Keywords: regression)

+++ This bug was initially created as a clone of Bug #1079824 +++ STR: 1. Open Contacts app and choose any contact with phone number; 2. Tap on "Send message" button; 3. Once Messages app is opened, tap on "X" to close app; 4. Choose save draft; From Bug 1079828 comment 7: So, I can confirm that after patch for bug 994190 duplicated drafts appeared. But something bad also happens on our side, I see that we call Drafts.store 3(!) times on activity close: 1. when user confirms saving a draft; 2. onVisibilityChange -> ThreadUI.saveDraft (there is implicit Drafts.store call inside) call; 3. onVisibilityChange -> explicit Drafts.store call.
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.