Review Request 117791: Fix for a bug that caused the contacts list to not expand some groups on opening and filtering

Renato Atilio renatoat at gmail.com
Sat Apr 26 18:02:38 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117791/
-----------------------------------------------------------

(Updated April 26, 2014, 6:02 p.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Bugs: 325506
    http://bugs.kde.org/show_bug.cgi?id=325506


Repository: ktp-contact-list


Description
-------

I've fixed the 325506 bug by creating a decision upon the validity of the inserted item's parent. If it is valid, it works as before, considering it is a top-level element we want to collapse/expand. If it is invalid, the inserted item itself is probably the top-level node (hence the invalid parent - there's none), so we get it in another way.

I don't think the second validity check is strictly needed, but I thought it was safer to keep it anyway.


Diffs
-----

  contact-list-widget.h c5b5328 
  contact-list-widget.cpp ae1b9c9 

Diff: https://git.reviewboard.kde.org/r/117791/diff/


Testing
-------


Thanks,

Renato Atilio

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140426/62a5d1b5/attachment-0001.html>


More information about the KDE-Telepathy mailing list