Bug 325957

Summary: Log viewer crash displaying conversations from multiple accounts on the same day
Product: [Unmaintained] telepathy Reporter: Leon Handreke <leonh>
Component: text-uiAssignee: Telepathy Bugs <kde-telepathy-bugs>
Status: RESOLVED FIXED    
Severity: crash    
Priority: NOR    
Version: git-latest   
Target Milestone: Future   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Leon Handreke 2013-10-13 10:40:06 UTC
When chatting with the same person with multiple accounts on the same day, the date row in the log viewer can be expanded to reveal the accounts from which conversations with this person were held on this day.

This situation can lead to crashes. Here are some examples:
1. The first row in the date list can be expanded to reveal the accounts. The first account has an expansion arrow next to it and if expanded, it crashes. The second account displays without such an arrow and clicking it reveals the conversation.
2. The first an only row in the date list can be expanded and should reveal the two accounts that conversations were held with on that date, but instead it crashes.

Reproducible: Always

Steps to Reproduce:
Case 2 can easily be reproduced by doing the following:

1. Add two test contacts (test1@test.com and test2@test.com)
2. Send a message to each of them
3. Try to view these conversations in the log viewer
Actual Results:  
Crash

Expected Results:  
I can view the conversation

After going through it with a debugger, I have a hunch that it may be related to rowCount not always returning 0 if the parent is an account but rather tries to return the rowCount for the corresponding row index at the date level (and either possibly wrongly displays expansion arrows or crashes because the date level has less rows than the accounts level). No idea how to fix it though.
Comment 1 Leon Handreke 2013-10-17 22:05:24 UTC
Git commit 2e9c295e71137a4ddd79edd27a700e2f289eeead by Leon Handreke.
Committed on 16/10/2013 at 20:03.
Pushed by leonh into branch 'kde-telepathy-0.7'.

Fix logviewer crash with multiple conversations per date

This situation commonly occurs when conversations from multiple accounts
were held on a single day.

REVIEW: 113289

M  +4    -3    logviewer/dates-model.cpp

http://commits.kde.org/telepathy-text-ui/2e9c295e71137a4ddd79edd27a700e2f289eeead