<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/118045/">https://git.reviewboard.kde.org/r/118045/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 8th, 2014, 10:23 a.m. UTC, <b>James Smith</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">There was a problem IIRC with the signal form of onRequestedPresenceChanged() not the m_globalPresence->requestedPresence(). The onRequestedPresenceChanged() signal doesn't set the presence properly when accounts are disabled in the accounts kcm. Couldn't using m_globalPresence->requestedPresence() cause problems when the requestedPresence() isn't actually what the account is set to? For instance, now, when the currentPresence() is Away and the account presence is Away, the plugin won't activate. If the requestedPresence() was at Not Available and the account moved to its best presence Away, the plugin will activate. So, that's probably an improvement. Do you know why Away was filtered as an originating presence? This doesn't work for the status message plugins because the autoaway plugin doesn't overwrite the status message with the preset away message (if there is one set), and only doesn't overwrite the status message with the preset away message (if there is one set) for Away -only-, as the originating presence.</pre>
</blockquote>
<p>On May 8th, 2014, 8:38 p.m. UTC, <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">>onRequestedPresenceChanged() signal doesn't set the presence properly when accounts are disabled in the accounts kcm
onRequestedPresenceChanged is a slot. I'm confused what you mean.
If an accunt is disabled doing requestPresence won't do anything.
>Do you know why Away was filtered as an originating presence?
If you're already away there's no point setting yourself to away again. More importantly you don't want to leave being in away when you autoaway gets deactivated.</pre>
</blockquote>
<p>On May 8th, 2014, 8:39 p.m. UTC, <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">*account</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">currentPresenceChanged() signal. I think the contact list currently connects to it and every other place the presence is needed requestedPresence() is used. Using requestedPresenceChanged() has issues when disabling accounts where when reenabling and showing enabled in the kcm the status doesn't actually get set at what it's supposed to in the presence chooser and / or the presence app, instead staying offline.
>If you're already away there's no point setting yourself to away again
It should actually be required, to set the status message if there's a custom status message. Otherwise the set message isn't picked up, and the plugin doesn't go to away with message, or continue to not available. Even if left empty, the away / N/A messages don't get changed to the active status message plugin. So I think that's currently a fixme. Leaving AutoAway should restore the last presence, whether it was away or not. We have enough problems in the contact list with that signal triggering on status message plugins disabling, it also doubtlessly works to set the presence back to what it was prior to the plugin engaging whether it was also away or not.</pre>
<br />
<p>- James</p>
<br />
<p>On May 7th, 2014, 10:29 p.m. UTC, David Edmundson wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By David Edmundson.</div>
<p style="color: grey;"><i>Updated May 7, 2014, 10:29 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-kded-module
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Use requestedPresence throughtout
Mixing current + requested will also get you in a mess as things will
often be slightly out for a while.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>status-handler.cpp <span style="color: grey">(4b9c25a)</span></li>
<li>autoaway.cpp <span style="color: grey">(13b71a5)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118045/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>