Closed
Bug 833213
Opened 11 years ago
Closed 11 years ago
[Email] Search breaks on empty subject; visible in "subject" and "all" search filters
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Tracking
(blocking-b2g:tef+, b2g18 fixed, b2g18-v1.0.0 fixed)
VERIFIED
FIXED
blocking-b2g | tef+ |
People
(Reporter: leo.bugzilla.gaia, Assigned: asuth)
References
Details
(Whiteboard: testrun 3)
Attachments
(2 files)
1. Title : Email list Search results for ALL and SUBJECT fields are empty 2. Precondition : Email account has to be configured and some emails should be present with subject. 3. Tester's Action : Email - Open Inbox list - Select search field - enter some text which is present in the existing mail (Subject)- click on subject and All fields. 4. Detailed Symptom (ENG.) :Not listing the mails which matches the search string in Subject and All fields 5. Expected :It should populate the list of returned results matching the search in ALL and Subject fields also. 6.Reproducibility: Y 1)Frequency Rate : 100% 7.Gaia Revision: a03f7b532e9998e646d55f93a0fc03a04d7ca7d9
Assignee | ||
Comment 1•11 years ago
|
||
Please provide logcat and account type info: https://wiki.mozilla.org/Gaia/Email/RequiredBugInfo Additionally, please describe the actual text string that was searched for and provide an example of the text that it was expected to match against.
Status: NEW → UNCONFIRMED
Ever confirmed: false
Flags: needinfo?(leo.bugzilla.gaia)
Issue Not reproduced in Gaia Revision: 7b8dea7e7ec377a2143151bbe5e9998d87f7b36d.
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Flags: needinfo?(leo.bugzilla.gaia)
Resolution: --- → WORKSFORME
Verified reproduceable in Unagi build 20130125070201 3/3 Dec 5th Kernel Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/94a2d6fcdfde Gaia: 6369dbf33b622faf4b4d176fed30b77c5c319dfc
Whiteboard: testrun 3
Assignee | ||
Comment 4•11 years ago
|
||
(In reply to Andrew Sutherland (:asuth) from comment #1) > Please provide logcat and account type info: > https://wiki.mozilla.org/Gaia/Email/RequiredBugInfo > > Additionally, please describe the actual text string that was searched for > and provide an example of the text that it was expected to match against.
Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: WORKSFORME → ---
Comment 5•11 years ago
|
||
Comment 6•11 years ago
|
||
I attempted to search using the term Twitter. Searching by To returned results, as did searching by Body. Searching by Subject returned a blank page even though there are valid results to be had from the search. This is from the same device version as mlevin's.
Comment 7•11 years ago
|
||
Oops, mlevin -> mdaniloff
Assignee | ||
Comment 8•11 years ago
|
||
Inferred from the log that we must be throwing an exception that sadly is not caught and logged and it had to do with the subject. Then I remembered we normalize subjects to be null!
Assignee: nobody → bugmail
Status: REOPENED → ASSIGNED
Summary: [Email] Email list Search results for ALL and SUBJECT fields are empty → [Email] Search breaks on empty subject; visible in "subject" and "all" search filters
Assignee | ||
Comment 9•11 years ago
|
||
Very straightforward patch. I add a test sub-case that fails without the fix. I also added a try block on the aggregating MessageFilterer that I verified logs a nice console.error message without the fix.
Attachment #707992 -
Flags: review?(squibblyflabbetydoo)
Updated•11 years ago
|
Attachment #707992 -
Flags: review?(squibblyflabbetydoo) → review+
Assignee | ||
Comment 10•11 years ago
|
||
landed on gaia-email-libs-and-more/master: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/118 landed on gaia/master: https://github.com/mozilla-b2g/gaia/pull/7910
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•11 years ago
|
||
blocking=tef? request: Bug 821447 improved our subject display logic for missing subjects, but either increased or introduced the problem where a message without a subject would break subject search. That bug is present on v1.0.0. We already have a fix that is now landed on master, and would like to land that. If tef+ can't be granted, we would like leo+ or approval-v1. Note that bug 835411 which is a dupe of this bug that I am going to dupe here is marked UX-P1, TEF_REQ, and PRODUCT-FIXFEATURE in the whiteboard.
blocking-b2g: --- → tef?
Depends on: 821447
Comment 13•11 years ago
|
||
Breaking email search is pretty bad, since this is a straightforward fix we'll block and get uplift to mozilla-b2g18 tip as well as mozilla-b2g18_v1_0_0
Comment 14•11 years ago
|
||
v1-train: 5f97bb66a70e0e61b1604f2572dc88cbbc140a10 v1.0.0: c4589884c89f724ba0f72107968d4111d4a9f1b2
Comment 15•11 years ago
|
||
Build ID: 201302014070203 Kernel: Dec 5 Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/d1288313218e Gaia: 6544fdb8dddc56f1aefe94482402488c89eeec49 Verified on "Unagi" This bug no longer reproduces.
Comment 16•11 years ago
|
||
Verified fixed on Unagi: Unagi Build ID: 20130322070203 Kernel Date: Dec 5 Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/4931ec89ebbe Gaia: 85fd164691bb852f1cfaf82405df4380629ced6e
Status: RESOLVED → VERIFIED
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•