[Bug 304085] Contact list crashed after unlocking screen

[Martin Klapetek]

https://bugs.kde.org/show_bug.cgi?id=304085

Martin Klapetek <martin.klapetek at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
   Version Fixed In|                            |0.5
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/tele
                   |                            |pathy-contact-list/67d235c4
                   |                            |6e492db89de25fc185201128d5b
                   |                            |e7bdc

--- Comment #2 from Martin Klapetek <martin.klapetek at gmail.com> ---
Git commit 67d235c46e492db89de25fc185201128d5be7bdc by Martin Klapetek.
Committed on 31/07/2012 at 15:22.
Pushed by mklapetek into branch 'master'.

Set the new presence first and only then remove the temp presence

Removing the temp presence first causes setting new selected index (as the
currently selected one was just deleted) which is set to the next in the list,
which in case of the temp presence is the 'Now playing...' one, which prompts
the user for enabling it if it is disabled.
So now we first change the presence (and current index) and only then remove
the temp presence, then it won't cause another unwanted index re-setting.

Reviewed-by: David Edmundson
FIXED-IN: 0.5

M  +3    -2    global-presence-chooser.cpp

http://commits.kde.org/telepathy-contact-list/67d235c46e492db89de25fc185201128d5be7bdc

-- 
You are receiving this mail because:
You are the assignee for the bug.