Closed
Bug 1084633
Opened 10 years ago
Closed 10 years ago
Sender name shown for sessionstart messages as well as messages without a sender name in search results.
Categories
(Thunderbird :: Instant Messaging, defect)
Thunderbird
Instant Messaging
Tracking
(thunderbird36 fixed)
RESOLVED
FIXED
Thunderbird 36.0
Tracking | Status | |
---|---|---|
thunderbird36 | --- | fixed |
People
(Reporter: nhnt11, Assigned: nhnt11)
References
Details
Attachments
(2 files, 1 obsolete file)
1.74 KB,
patch
|
nhnt11
:
review+
|
Details | Diff | Splinter Review |
1.39 KB,
patch
|
aleth
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #8507217 -
Flags: review?(aleth)
Comment 1•10 years ago
|
||
Comment on attachment 8507217 [details] [diff] [review] Patch Review of attachment 8507217 [details] [diff] [review]: ----------------------------------------------------------------- Thanks! ::: mail/components/im/modules/index_im.js @@ +445,5 @@ > if (!aLogConv) > return Gloda.kWorkDone; > > let content = aLogConv.getMessages() > + .filter(m => !m.noLog) Please add a comment here to avoid future confusion as to why Log.getMessages can return noLog messages before checking it in.
Attachment #8507217 -
Flags: review?(aleth) → review+
Assignee | ||
Comment 2•10 years ago
|
||
Added the requested comment, carrying forward the r+.
Assignee: nobody → nhnt11
Attachment #8507217 -
Attachment is obsolete: true
Attachment #8507359 -
Flags: review+
Updated•10 years ago
|
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 36.0
Assignee | ||
Comment 4•10 years ago
|
||
Remove extra parenthesis
Attachment #8516158 -
Flags: review?(aleth)
Comment 5•10 years ago
|
||
Comment on attachment 8516158 [details] [diff] [review] Bustage fix Review of attachment 8516158 [details] [diff] [review]: ----------------------------------------------------------------- Thanks! Please land this asap, a=bustage-fix.
Attachment #8516158 -
Flags: review?(aleth) → review+
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/comm-central/rev/e52b7e18a9b5
Updated•9 years ago
|
status-thunderbird36:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•