[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kopete-devel
Subject:    Re: [kopete-devel] How to unset an account mood in Kopete 0.70.90
From:       Romain <romainguinot () gmail ! com>
Date:       2009-10-04 15:28:13
Message-ID: ae44b51f0910040828v1fd65f3apc8d2db5fbf6ab821 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Proposed patch added as an attachment to this bug report :
https://bugs.kde.org/show_bug.cgi?id=111329#c3

On Thu, Oct 1, 2009 at 11:20 AM, Romain <romainguinot@gmail.com> wrote:

> I've added a comment on this existing bug regarding jabber mood :
> https://bugs.kde.org/show_bug.cgi?id=111329#c3
> I think i see the fix as you'll see in my comment but do not have the
> possibility to test it right now, i'll probably test if for myself in the
> next few days if you haven't fixed it yet :)
>
> BTW, i'm using KDE 4.3.1 on Fedora 11.
> Best regards,
> Romain
>
>
>
>
> On Thu, Oct 1, 2009 at 4:07 AM, Matt Rogers <mattr@kde.org> wrote:
>
>> On Wednesday 30 September 2009 07:17:57 Romain wrote:
>> > Hi,
>> >
>> > I've set a mood for myself on a jabber account, but now would like to
>> >  remove it.
>> > It seems clicking "None" does not work, at least not immediately since
>> >  other colleagues still see the mood with different clients (Gajim,
>> >  Pidgin..; all linux laptops).
>> >
>> > Should the change be immediate ?
>> >
>> > I've seen in protocols/jabber/jabberaccount.cpp (1109-1122) this snippet
>> of
>> > code :
>> >
>> > void JabberAccount::slotSetMood()
>> > {
>> >     KAction *action = (KAction *)sender();
>> >     Mood::Type type = (Mood::Type)action->data().toInt();
>> >     if(type == Mood::None)
>> >     {
>> >     }
>> >     else
>> >     {
>> >         PubSubItem psi("current",
>> > Mood(type).toXml(*client()->client()->rootTask()->doc()));
>> >         JT_PubSubPublish *task = new
>> > JT_PubSubPublish(client()->client()->rootTask(), QString("
>> > http://jabber.org/protocol/mood"), psi);
>> >         task->go(true);
>> >     }
>> > }
>> >
>> >
>> > If nothing is done when Mood is None, how is one supposed to reset a
>> mood
>> > back to None once one has been chosen ?
>> >
>> > Many thanks,
>> > Romain.
>> >
>>
>> I think you've found the answer. :) Could you provide a patch to fix this,
>> or
>> at the very least file a bug (if there isn't one already?)
>> --
>> Matt
>>
>
>
>
> --
> Romain.
>



-- 
Romain.

[Attachment #5 (text/html)]

Proposed patch added as an attachment to this bug report : <a \
href="https://bugs.kde.org/show_bug.cgi?id=111329#c3" \
target="_blank">https://bugs.kde.org/show_bug.cgi?id=111329#c3</a><br><br><div \
class="gmail_quote">On Thu, Oct 1, 2009 at 11:20 AM, Romain <span dir="ltr">&lt;<a \
href="mailto:romainguinot@gmail.com">romainguinot@gmail.com</a>&gt;</span> wrote:<br> \
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); \
margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I&#39;ve added a comment on this \
existing bug regarding jabber mood : <a \
href="https://bugs.kde.org/show_bug.cgi?id=111329#c3" \
target="_blank">https://bugs.kde.org/show_bug.cgi?id=111329#c3</a><br> I think i see \
the fix as you&#39;ll see in my comment but do not have the possibility to test it \
right now, i&#39;ll probably test if for myself in the next few days if you \
haven&#39;t fixed it yet :)<br> <br>BTW, i&#39;m using KDE 4.3.1 on Fedora 11. \
<br>Best regards, <br>Romain<div><div></div><div class="h5"><br><br><br><br><div \
class="gmail_quote">On Thu, Oct 1, 2009 at 4:07 AM, Matt Rogers <span \
dir="ltr">&lt;<a href="mailto:mattr@kde.org" \
target="_blank">mattr@kde.org</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); \
margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div>On Wednesday 30 \
September 2009 07:17:57 Romain wrote:<br> &gt; Hi,<br>
&gt;<br>
&gt; I&#39;ve set a mood for myself on a jabber account, but now would like to<br>
&gt;  remove it.<br>
&gt; It seems clicking &quot;None&quot; does not work, at least not immediately \
since<br> &gt;  other colleagues still see the mood with different clients \
(Gajim,<br> &gt;  Pidgin..; all linux laptops).<br>
&gt;<br>
&gt; Should the change be immediate ?<br>
&gt;<br>
&gt; I&#39;ve seen in protocols/jabber/jabberaccount.cpp (1109-1122) this snippet \
of<br> &gt; code :<br>
&gt;<br>
&gt; void JabberAccount::slotSetMood()<br>
&gt; {<br>
&gt;     KAction *action = (KAction *)sender();<br>
&gt;     Mood::Type type = (Mood::Type)action-&gt;data().toInt();<br>
&gt;     if(type == Mood::None)<br>
&gt;     {<br>
&gt;     }<br>
&gt;     else<br>
&gt;     {<br>
&gt;         PubSubItem psi(&quot;current&quot;,<br>
&gt; Mood(type).toXml(*client()-&gt;client()-&gt;rootTask()-&gt;doc()));<br>
&gt;         JT_PubSubPublish *task = new<br>
&gt; JT_PubSubPublish(client()-&gt;client()-&gt;rootTask(), QString(&quot;<br>
&gt; <a href="http://jabber.org/protocol/mood" \
target="_blank">http://jabber.org/protocol/mood</a>&quot;), psi);<br> &gt;         \
task-&gt;go(true);<br> &gt;     }<br>
&gt; }<br>
&gt;<br>
&gt;<br>
&gt; If nothing is done when Mood is None, how is one supposed to reset a mood<br>
&gt; back to None once one has been chosen ?<br>
&gt;<br>
&gt; Many thanks,<br>
&gt; Romain.<br>
&gt;<br>
<br>
</div></div>I think you&#39;ve found the answer. :) Could you provide a patch to fix \
this, or<br> at the very least file a bug (if there isn&#39;t one already?)<br>
--<br>
<font color="#888888">Matt<br>
</font></blockquote></div><br><br clear="all"><br></div></div>-- <br><font \
color="#888888">Romain.<br> </font></blockquote></div><br><br clear="all"><br>-- \
<br>Romain.<br>



_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic