Handles 180 SIP responses arriving after a 183 by not starting a local audio notification since the 183 means the server is already sending one.

1 files changed, 8 insertions, 1 deletions

diff --git a/src/net/java/sip/communicator/impl/gui/main/call/CallPeerAdapter.java b/src/net/java/sip/communicator/impl/gui/main/call/CallPeerAdapter.java
index ad6b803..60c83e6 100644
--- a/src/net/java/sip/communicator/impl/gui/main/call/CallPeerAdapter.java
+++ b/src/net/java/sip/communicator/impl/gui/main/call/CallPeerAdapter.java
@@ -81,8 +81,13 @@ public class CallPeerAdapter
             NotificationManager.stopSound(NotificationManager.DIALING);
         }
 
-        if (newState == CallPeerState.ALERTING_REMOTE_SIDE)
+        if (newState == CallPeerState.ALERTING_REMOTE_SIDE
+            //if we were already in state CONNECTING_WITH_EARLY_MEDIA the server
+            //is already taking care of playing the notifications so we don't
+            //need to fire a notification here.
+            && oldState != CallPeerState.CONNECTING_WITH_EARLY_MEDIA)
         {
+            //
             NotificationManager
                 .fireNotification(NotificationManager.OUTGOING_CALL);
         }
@@ -108,6 +113,8 @@ public class CallPeerAdapter
         }
         else if (newState == CallPeerState.CONNECTING_WITH_EARLY_MEDIA)
         {
+            //this means a call with early media. make sure that we are not
+            //playing local notifications any more.
             NotificationManager
                 .stopSound(NotificationManager.OUTGOING_CALL);
         }