Closed Bug 1069845 Opened 6 years ago Closed 6 years ago

Viewing logs from faceted search results is broken


(Thunderbird :: Instant Messaging, defect)

Not set


(Not tracked)

Thunderbird 38.0


(Reporter: aleth, Assigned: nhnt11)




(1 file)

Search for some string that has hits in chat logs.
Click on a chat log.
The desired log does not open, instead a broken search result pane appears.

Timestamp: 19/09/2014 00:13:26
Error: A promise chain failed to handle a rejection. Did you forget to '.catch', or did you forget to 'return'?

Date: Fri Sep 19 2014 00:13:21 GMT+0200 (CEST)
Full Message: Error: Log was passed an invalid argument, expected a non-empty array or a string.
Source File: /dist/
Line: 449
Source Code:

I really hope this is not already broken in TB 31.
Probably a dupe?
Depends on: 1103647
While adding the unread ruler to TB, I noticed that this code looks like it hasn't been touched in a long time:

Probably a good place to start digging.
Attached patch PatchSplinter Review
This is a very unfortunate regression caused by bug 955292.
The index_im changes from that bug cause chat logs to be indexed against their full paths, instead of path relative to the logs directory.
This patch fixes that, but we now need a way to re-index chat log files since bug 955292 landed and remove the now-garbage entries. I'll file a follow-up for that.
Thankfully, release channel users won't be affected.
Assignee: nobody → nhnt11
Attachment #8567381 - Flags: review?(aleth)
Depends on: 1135291
When did this regression first land in Thunderbird?
Comment on attachment 8567381 [details] [diff] [review]

Review of attachment 8567381 [details] [diff] [review]:

Thanks for fixing this! Unfortunately tidying up the effects of the regression is likely going to be more complicated...

::: mail/components/im/content/chat-messenger-overlay.js
@@ +606,3 @@
>        imServices.logs.getLogFromFile(path, true).then(aLog => {
>          aLog.getConversation().then(aConv => {
>            this._showLog(aConv, path, item.searchTerm || undefined);

I was going to say this should be aLog.path not path, but in fact the path argument of _showLog is never used for anything anymore. Since I made the change that caused that, I'll file a followup to clean this up.
Attachment #8567381 - Flags: review?(aleth) → review+
(In reply to Kent James (:rkent) from comment #4)
> When did this regression first land in Thunderbird?

This July 4 push caused the regression, which puts it in TB 33. I have checked the ESR 31.4 is not affected.
As landing this will cause further breakage for users of TB 33 and up, let's hold off landing until the migration patch is ready if we can.
Blocks: 1135297
It's a day before the merge, so I'm pushing this.
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 38.0
You need to log in before you can comment on or make changes to this bug.