[Asterisk-code-review] res/res pjsip/presence xml.c: Add missing 2nd call presence ... (asterisk[13])

Matt Jordan asteriskteam at digium.com
Thu Jan 21 17:25:22 CST 2016


Matt Jordan has submitted this change and it was merged.

Change subject: res/res_pjsip/presence_xml.c: Add missing 2nd call presence state case.
......................................................................


res/res_pjsip/presence_xml.c: Add missing 2nd call presence state case.

ASTERISK-25712 #close
Reported by: Richard Mudgett

Change-Id: I70634df24f8c6c3a2c66c45af61d021e4999253f
---
M res/res_pjsip/presence_xml.c
1 file changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved



diff --git a/res/res_pjsip/presence_xml.c b/res/res_pjsip/presence_xml.c
index b98ea02..c991a0d 100644
--- a/res/res_pjsip/presence_xml.c
+++ b/res/res_pjsip/presence_xml.c
@@ -91,6 +91,12 @@
 		*pidfstate = "busy";
 		*pidfnote = "Ringing";
 		break;
+	case (AST_EXTENSION_INUSE | AST_EXTENSION_RINGING):
+		*statestring = "confirmed";
+		*local_state = NOTIFY_INUSE;
+		*pidfstate = "busy";
+		*pidfnote = "Ringing";
+		break;
 	case AST_EXTENSION_INUSE:
 		*statestring = "confirmed";
 		*local_state = NOTIFY_INUSE;
@@ -121,7 +127,7 @@
 		*statestring = "terminated";
 		*local_state = NOTIFY_OPEN;
 		*pidfstate = "--";
-		*pidfnote ="Ready";
+		*pidfnote = "Ready";
 		break;
 	}
 }

-- 
To view, visit https://gerrit.asterisk.org/2061
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I70634df24f8c6c3a2c66c45af61d021e4999253f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list