If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[email] quick tap on message in list after first load sometimes does not work

VERIFIED FIXED in 2.1 S1 (1aug)

Status

Firefox OS
Gaia::E-Mail
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: jrburke, Assigned: jrburke)

Tracking

unspecified
2.1 S1 (1aug)
x86
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
With the cookie cache, it is possible to tap on an email message to start navigating to the message_reader card even before the full data model is loaded up.

With the outbox changes (only on master, not on 2.0), it introduced a failure in message_list's onClickMessage where it expects this.curFolder to already be set.

The error only occurs if you tap fast enough, and it is non-fatal. If you try again a bit later everything still works. The only evidence of the error is watching the logcat, and this will show up:

ERR: onerror reporting: TypeError: this.curFolder is null @ app://email.gaiamobile.org/js/mail_app.js : 2458

It is a simple fix to only do that check if there is a curFolder, prepping a fix now.
(Assignee)

Comment 1

3 years ago
Created attachment 8464332 [details] [review]
GitHub pull request

Just a test for this.curFolder for a fix. Asking :mcav since he has outbox context.
Attachment #8464332 - Flags: review?(m)
Comment on attachment 8464332 [details] [review]
GitHub pull request

Looks good, thanks for catching that.
Attachment #8464332 - Flags: review?(m) → review+
(Assignee)

Comment 3

3 years ago
Merged in master:
https://github.com/mozilla-b2g/gaia/commit/c6a6ac54703970ad4668d37161f599d438e37681

from pull request:
https://github.com/mozilla-b2g/gaia/pull/22281
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Comment 4

3 years ago
[Environment]
Gaia      3584b2723412ed3299c6761f465885d80651c87e
Gecko     https://hg.mozilla.org/mozilla-central/rev/e7806c9c83f3
BuildID   20140820160201
Version   34.0a1
ro.build.version.incremental=94
ro.build.date=Tue May 20 09:29:20 CST 2014


[Result]
PASS
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.