[svn-commits] mjordan: branch 13 r431622 - in /branches/13: res/ari/ rest-api/api-docs/

[SVN commits to the Digium repositories]

Author: mjordan
Date: Sun Feb  8 21:10:59 2015
New Revision: 431622

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=431622
Log:
res/ari/resource_channels: Add missing 'no_answer' reason to DELETE /channels

One of the canonical reasons for hanging up a channel is because the far end
failed to answer - or because someone else answered, and we want to get rid of
this channel. This patch adds the missing value to the 'reason' query parameter
for the DELETE /channels operation.

Review: https://reviewboard.asterisk.org/r/4400

ASTERISK-24745 #close
Reported by: Ben Merrills
patches:
  add_no_answer_ari_hangup_cause.diff uploaded by Ben Merrills (License 6678)

Modified:
    branches/13/res/ari/resource_channels.c
    branches/13/rest-api/api-docs/channels.json

Modified: branches/13/res/ari/resource_channels.c
URL: http://svnview.digium.com/svn/asterisk/branches/13/res/ari/resource_channels.c?view=diff&rev=431622&r1=431621&r2=431622
==============================================================================
--- branches/13/res/ari/resource_channels.c (original)
+++ branches/13/res/ari/resource_channels.c Sun Feb  8 21:10:59 2015
@@ -704,6 +704,8 @@
 		cause = AST_CAUSE_BUSY;
 	} else if (!strcmp(args->reason, "congestion")) {
 		cause = AST_CAUSE_CONGESTION;
+	} else if (!strcmp(args->reason, "no_answer")) {
+		cause = AST_CAUSE_NOANSWER;
 	} else {
 		ast_ari_response_error(
 			response, 400, "Invalid Reason",

Modified: branches/13/rest-api/api-docs/channels.json
URL: http://svnview.digium.com/svn/asterisk/branches/13/rest-api/api-docs/channels.json?view=diff&rev=431622&r1=431621&r2=431622
==============================================================================
--- branches/13/rest-api/api-docs/channels.json (original)
+++ branches/13/rest-api/api-docs/channels.json Sun Feb  8 21:10:59 2015
@@ -313,7 +313,8 @@
 								"values": [
 									"normal",
 									"busy",
-									"congestion"
+									"congestion",
+									"no_answer"
 								]
 							}
 						}