[Asterisk-code-review] app queue: Add priority to AMI QueueStatus (asterisk[13])

George Joseph asteriskteam at digium.com
Tue Aug 1 15:39:15 CDT 2017


George Joseph has submitted this change and it was merged. ( https://gerrit.asterisk.org/5980 )

Change subject: app_queue: Add priority to AMI QueueStatus
......................................................................

app_queue: Add priority to AMI QueueStatus

Add priority to callers in AMI QueueStatus response

ASTERISK-27092 #close

Change-Id: I8d1f737a72c7c38f4cfe1a4ee3ecc0a4f85bd199
---
M CHANGES
M apps/app_queue.c
M include/asterisk/manager.h
3 files changed, 9 insertions(+), 2 deletions(-)

Approvals:
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, but someone else must approve
  Scott Griepentrog: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved
  George Joseph: Approved for Submit



diff --git a/CHANGES b/CHANGES
index cce7f33..8063214 100644
--- a/CHANGES
+++ b/CHANGES
@@ -33,9 +33,15 @@
    which sends signals to the application and its descendants directly, or
    "process" which sends signals only to the application itself.
 
+res_pjsip
+------------------
  * New dialplan function PJSIP_DTMF_MODE added to get or change the DTMF mode
    of a channel on a per-call basis.
 
+app_queue
+------------------
+ * Add priority to callers in AMI QueueStatus response.
+
 ------------------------------------------------------------------------------
 --- Functionality changes from Asterisk 13.16.0 to Asterisk 13.17.0 ----------
 ------------------------------------------------------------------------------
diff --git a/apps/app_queue.c b/apps/app_queue.c
index ac4738f..e17780a 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -9695,6 +9695,7 @@
 					"ConnectedLineNum: %s\r\n"
 					"ConnectedLineName: %s\r\n"
 					"Wait: %ld\r\n"
+					"Priority: %d\r\n"
 					"%s"
 					"\r\n",
 					q->name, pos++, ast_channel_name(qe->chan), ast_channel_uniqueid(qe->chan),
@@ -9702,7 +9703,7 @@
 					S_COR(ast_channel_caller(qe->chan)->id.name.valid, ast_channel_caller(qe->chan)->id.name.str, "unknown"),
 					S_COR(ast_channel_connected(qe->chan)->id.number.valid, ast_channel_connected(qe->chan)->id.number.str, "unknown"),
 					S_COR(ast_channel_connected(qe->chan)->id.name.valid, ast_channel_connected(qe->chan)->id.name.str, "unknown"),
-					(long) (now - qe->start), idText);
+					(long) (now - qe->start), qe->prio, idText);
 				++q_items;
 			}
 		}
diff --git a/include/asterisk/manager.h b/include/asterisk/manager.h
index afd9ca1..a9e960c 100644
--- a/include/asterisk/manager.h
+++ b/include/asterisk/manager.h
@@ -54,7 +54,7 @@
 - \ref manager.c Main manager code file
  */
 
-#define AMI_VERSION                     "2.10.0"
+#define AMI_VERSION                     "2.10.1"
 #define DEFAULT_MANAGER_PORT 5038	/* Default port for Asterisk management via TCP */
 #define DEFAULT_MANAGER_TLS_PORT 5039	/* Default port for Asterisk management via TCP */
 

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I8d1f737a72c7c38f4cfe1a4ee3ecc0a4f85bd199
Gerrit-Change-Number: 5980
Gerrit-PatchSet: 4
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Niklas Larsson <niklas at tese.se>
Gerrit-Reviewer: Scott Griepentrog <sgriepentrog at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170801/2a0b2ea5/attachment-0001.html>


More information about the asterisk-code-review mailing list