Review Request 112705: set presence should only be done on user action

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112705/#review40426
-----------------------------------------------------------


This review has been submitted with commit 45e7e542d33eb04b56e3bd78a16850899f342731 by Weng Xuetian to branch master.

- Commit Hook


On Sept. 12, 2013, 7:52 p.m., Xuetian Weng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112705/
> -----------------------------------------------------------
> 
> (Updated Sept. 12, 2013, 7:52 p.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> add one account with wrong password, lock screen to trigger auto away, dbus messages shows that contact list set the requestedpresence to all account, caused by setPresence in onCurrentIndexChanged instead of activated.
> 
> 
> Diffs
> -----
> 
>   global-presence-chooser.cpp 55a30b9 
> 
> Diff: http://git.reviewboard.kde.org/r/112705/diff/
> 
> 
> Testing
> -------
> 
> now lock screen doesn't bring bad account online.
> 
> 
> Thanks,
> 
> Xuetian Weng
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130921/c4cb6f9b/attachment.html>