<?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>309146</bug_id>
          
          <creation_ts>2012-10-28 13:59:27 +0000</creation_ts>
          <short_desc>&apos;Audio Call&apos; button opens call-ui although the call-ui-window is already open</short_desc>
          <delta_ts>2014-05-15 15:10:04 +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>call-ui</component>
          <version>0.5.0</version>
          <rep_platform>Gentoo Packages</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>Future</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Lukas Schneiderbauer">lukas.schneiderbauer</reporter>
          <assigned_to name="Telepathy Bugs">kde-telepathy-bugs-null</assigned_to>
          <cc>heri+kde</cc>
    
    <cc>kde</cc>
    
    <cc>mayank25080562</cc>
          
          <cf_commitlink>http://commits.kde.org/telepathy-call-ui/70c319c79aee4c4829587cfa7a53a3e1518378f6</cf_commitlink>
          <cf_versionfixedin>0.8.1</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1310215</commentid>
    <comment_count>0</comment_count>
    <who name="Lukas Schneiderbauer">lukas.schneiderbauer</who>
    <bug_when>2012-10-28 13:59:27 +0000</bug_when>
    <thetext>The &apos;Audio Call&apos; button opens call-ui despite the fact that it is already openend. This results in weird behaviour. For example if an active call is going on and you press the &apos;Audio Call&apos;-button again (for whatever reasons) a second window opens and one is tempted to close it again because it shows the exact same information the other call-ui window does. But with closing the window the active call-connection gets closed as well. This is likely not what the user intended to do.

Reproducible: Always

Steps to Reproduce:
1. Open chat-ui 
2. Click &apos;Audio Call&apos;
3. Click &apos;Audio Call&apos; again
Actual Results:  
A second call-ui window appears.

Expected Results:  
Maybe a focus-switch to the existing window would be a solution. (?)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1333637</commentid>
    <comment_count>1</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-01-20 13:50:00 +0000</bug_when>
    <thetext>Confirmed bug in Call-UI. 

It is always up to a handler when told to handleChanenls to check if it is already handling the channel.

Current code does not.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357910</commentid>
    <comment_count>2</comment_count>
    <who name="Dennis Schridde">heri+kde</who>
    <bug_when>2013-04-06 10:18:08 +0000</bug_when>
    <thetext>Does this persist in 0.6.0?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1357913</commentid>
    <comment_count>3</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2013-04-06 10:20:04 +0000</bug_when>
    <thetext>Yes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1433215</commentid>
    <comment_count>4</comment_count>
    <who name="mayank">mayank25080562</who>
    <bug_when>2014-02-28 18:31:48 +0000</bug_when>
    <thetext>This also happens with the &quot;Video Call&quot; button. Can this be solved, by disabling the button soon the button is clicked, and then re-enabling it once the call-ui is closed ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1433216</commentid>
    <comment_count>5</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2014-02-28 18:33:12 +0000</bug_when>
    <thetext>&gt;Can this be solved, by disabling the button soon the button is clicked, a

What would happen then if you opened a video call from the contact list, and then opened a text chat?

&gt;and then re-enabling it once the call-ui is closed ?

how will you know when this happens?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1433217</commentid>
    <comment_count>6</comment_count>
    <who name="mayank">mayank25080562</who>
    <bug_when>2014-02-28 18:41:13 +0000</bug_when>
    <thetext>We could make use of chat logger, logging in a message once you start a video call, and logging again when you hangup, thus knowing when to disable and when to re-enable again ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1433219</commentid>
    <comment_count>7</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2014-02-28 18:43:54 +0000</bug_when>
    <thetext>No that&apos;s a massive hack. 

The simplest approach is for the call-ui to see if it&apos;s already handling a call when it&apos;s told to handle the second one. It&apos;s all in the same process.
See #1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1433220</commentid>
    <comment_count>8</comment_count>
    <who name="mayank">mayank25080562</who>
    <bug_when>2014-02-28 18:52:30 +0000</bug_when>
    <thetext>Yeah right! I was inspired to do this, as in gmail chat, you get to see when you started a call, and when it ended. But to check whether it is handling a call, one would be needing a global parameter to check for, right ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1434031</commentid>
    <comment_count>9</comment_count>
    <who name="mayank">mayank25080562</who>
    <bug_when>2014-03-05 22:46:59 +0000</bug_when>
    <thetext>https://git.reviewboard.kde.org/r/116623/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1436257</commentid>
    <comment_count>10</comment_count>
    <who name="Martin Klapetek">mklapetek</who>
    <bug_when>2014-03-18 15:23:27 +0000</bug_when>
    <thetext>Git commit b4626ae5b866464d9abfa2762b979224dbb4b589 by Martin Klapetek, on behalf of mayank jha.
Committed on 18/03/2014 at 15:22.
Pushed by mklapetek into branch &apos;kde-telepathy-0.8&apos;.

Removes duplication of call-ui window for a single channel
REVIEW: 116623
FIXED-IN: 0.8.1

M  +5    -3    src/call-handler.cpp
M  +2    -0    src/call-handler.h

http://commits.kde.org/telepathy-call-ui/b4626ae5b866464d9abfa2762b979224dbb4b589</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1448604</commentid>
    <comment_count>11</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2014-05-15 15:10:04 +0000</bug_when>
    <thetext>Git commit 70c319c79aee4c4829587cfa7a53a3e1518378f6 by David Edmundson, on behalf of Vadim A. Misbakh-Soloviov.
Committed on 15/05/2014 at 15:10.
Pushed by davidedmundson into branch &apos;kde-telepathy-0.8&apos;.

Properly removes duplication of call-ui window for a single channel.

Fixes issue, made by b4626ae5b866464d9abfa2762b979224dbb4b589,
where call-ui dont make call at all, due to the typo in condition.

REVIEW: 118151

M  +1    -1    src/call-handler.cpp

http://commits.kde.org/telepathy-call-ui/70c319c79aee4c4829587cfa7a53a3e1518378f6</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>