<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>317850</bug_id>
          
          <creation_ts>2013-04-04 22:37:22 +0000</creation_ts>
          <short_desc>text-ui window opened from quick chat plasmoid button doesn&apos;t send message</short_desc>
          <delta_ts>2013-04-16 14:04:58 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>10</classification_id>
          <classification>Unmaintained</classification>
          <product>telepathy</product>
          <component>chat-plasmoid</component>
          <version>0.6.0</version>
          <rep_platform>Arch Linux</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>0.6.1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ezio Vergine">virgolus</reporter>
          <assigned_to name="Telepathy Bugs">kde-telepathy-bugs-null</assigned_to>
          <cc>heri+kde</cc>
    
    <cc>kde</cc>
    
    <cc>m4yer+zeug</cc>
    
    <cc>roland.leissa</cc>
          
          <cf_commitlink>http://commits.kde.org/telepathy-common-internals/f3a30eb36e174a38ca95fd1b7d2ea6a6c17fa281</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1357591</commentid>
    <comment_count>0</comment_count>
    <who name="Ezio Vergine">virgolus</who>
    <bug_when>2013-04-04 22:37:22 +0000</bug_when>
    <thetext>text-ui window opened from quick chat plasmoid button doesn&apos;t send message and does&apos;t refresh if new message incoming. If I open new text-ui from contact list I&apos;havent problems. 

Reproducible: Always

Steps to Reproduce:
1. open text-ui from chat plasmoid
2. write a message
Actual Results:  
Message isn&apos;t send and new incoming messages isn&apos;t visualized

Expected Results:  
message was sent correctly</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357907</commentid>
    <comment_count>1</comment_count>
    <who name="Dennis Schridde">heri+kde</who>
    <bug_when>2013-04-06 10:15:23 +0000</bug_when>
    <thetext>I confirm this behaviour.

The bug appears in a variety of flavours:
* Sometimes the input line in text-ui is greyed out and I cannot enter text at all.
* Sometimes I can enter text, but after pressing enter no message is send to my peer and neither is anything displayed in the chat view above.
* Incomming messages often do not appear in the text-ui.

I also use the quickchat, which works throughout the whole time:
* Incomming messages are always displayed there.
* I can send messages from there.
* After opening the text-ui using the blue button, the quickchat vanishes as intended, but it will always reappear later.
* I do not close my quickchat manually using the X button, because it will reappear on the next incomming message anyway.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357908</commentid>
    <comment_count>2</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-04-06 10:16:46 +0000</bug_when>
    <thetext>Can we get a log from the mission-control tab of ktp-debugger when this is hapenning</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357919</commentid>
    <comment_count>3</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-04-06 10:31:35 +0000</bug_when>
    <thetext>This is probably caused by:

Conversation::~Conversation()
{
    kDebug();
    requestClose();
    delete d;
}

We shouldn&apos;t be doing a requestClose if we&apos;re not handling the channel, which goes back to the problem here: https://bugs.kde.org/show_bug.cgi?id=293637 which is very difficult to fix 
without fixing TpQt</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357958</commentid>
    <comment_count>4</comment_count>
    <who name="">m4yer+zeug</who>
    <bug_when>2013-04-06 12:30:34 +0000</bug_when>
    <thetext>mission-control tab of ktp-debugger: http://paste.kde.org/717248/

after opening the ktp-chat-ui from quickchat, the debugger shows nothing, if i send a new message.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357959</commentid>
    <comment_count>5</comment_count>
    <who name="">m4yer+zeug</who>
    <bug_when>2013-04-06 12:31:10 +0000</bug_when>
    <thetext>[14:29:39] &lt;d_ed&gt; could you add in the comment &quot;d_ed says to look at line 43 and 47 and 87&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357968</commentid>
    <comment_count>6</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-04-06 13:08:41 +0000</bug_when>
    <thetext>Proposed plan of action, technically a slightly bodge but we don&apos;t have any other option:

 - Add method to Conversation  delegateChannel();
 - Move all calls to ChannelDelegator::delegateChannel() to here (there&apos;s one in ConversationsModel)
 - Add private member to Conversation bool isDelegated;  this should be set to default to false
 - Conversation::delegateChannel() will set this to true 
 - Conversation::requestClose() should check &quot;isDelegated == false&quot; before calling channel-&gt;requestClose();

I also want something added to our ReleaseTesting wiki page so that we won&apos;t be broken in 0.7.0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1359119</commentid>
    <comment_count>7</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-04-10 12:20:49 +0000</bug_when>
    <thetext>Git commit f3a30eb36e174a38ca95fd1b7d2ea6a6c17fa281 by David Edmundson.
Committed on 09/04/2013 at 14:40.
Pushed by davidedmundson into branch &apos;kde-telepathy-0.6&apos;.

Make sure ConversationModel never close a channels it is not handling
Related: bug 293637
REVIEW: 109925

M  +10   -1    KTp/Declarative/conversation.cpp
M  +1    -2    KTp/Declarative/conversations-model.cpp

http://commits.kde.org/telepathy-common-internals/f3a30eb36e174a38ca95fd1b7d2ea6a6c17fa281</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1361290</commentid>
    <comment_count>8</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-04-16 14:04:58 +0000</bug_when>
    <thetext>*** Bug 318447 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>