[Asterisk-code-review] manager.c: Update AMI Status event documentation (asterisk[15])

Jenkins2 asteriskteam at digium.com
Thu Dec 28 13:39:13 CST 2017


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/7748 )

Change subject: manager.c: Update AMI Status event documentation
......................................................................

manager.c: Update AMI Status event documentation

The AMI Status event had linkedid listed twice and was missing the
effective connected line name and number headers.

NOTE: The linkedid and other standard channel snapshot fields in the XML
documentation are part of the <channel_snapshot/> XML template defined in
doc/appdocsxml.xslt.

Change-Id: I004c4c4f9e7b40ef55035c831702721bec82496c
---
M main/manager.c
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  George Joseph: Looks good to me, but someone else must approve
  Sean Bright: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/main/manager.c b/main/manager.c
index 71d7432..576978c 100644
--- a/main/manager.c
+++ b/main/manager.c
@@ -253,13 +253,15 @@
 				<parameter name="DNID">
 					<para>Dialed number identifier</para>
 				</parameter>
+				<parameter name="EffectiveConnectedLineNum">
+				</parameter>
+				<parameter name="EffectiveConnectedLineName">
+				</parameter>
 				<parameter name="TimeToHangup">
 					<para>Absolute lifetime of the channel</para>
 				</parameter>
 				<parameter name="BridgeID">
 					<para>Identifier of the bridge the channel is in, may be empty if not in one</para>
-				</parameter>
-				<parameter name="Linkedid">
 				</parameter>
 				<parameter name="Application">
 					<para>Application currently executing on the channel</para>
@@ -4614,7 +4616,6 @@
 		"EffectiveConnectedLineName: %s\r\n"
 		"TimeToHangup: %ld\r\n"
 		"BridgeID: %s\r\n"
-		"Linkedid: %s\r\n"
 		"Application: %s\r\n"
 		"Data: %s\r\n"
 		"Nativeformats: %s\r\n"
@@ -4635,7 +4636,6 @@
 		S_COR(effective_id.name.valid, effective_id.name.str, "<unknown>"),
 		(long)ast_channel_whentohangup(chan)->tv_sec,
 		bridge ? bridge->uniqueid : "",
-		ast_channel_linkedid(chan),
 		ast_channel_appl(chan),
 		ast_channel_data(chan),
 		ast_format_cap_get_names(ast_channel_nativeformats(chan), &codec_buf),

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

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: merged
Gerrit-Change-Id: I004c4c4f9e7b40ef55035c831702721bec82496c
Gerrit-Change-Number: 7748
Gerrit-PatchSet: 2
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett 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/20171228/df9856db/attachment.html>


More information about the asterisk-code-review mailing list