[asterisk-commits] jrose: trunk r395215 - in /trunk: funcs/ include/asterisk/ main/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Jul 23 16:32:37 CDT 2013


Author: jrose
Date: Tue Jul 23 16:32:33 2013
New Revision: 395215

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=395215
Log:
func_channel: dtmf_features setting

Allows reading andsetting dtmf features via a channel function
CHANNEL(dtmf_features)

(closes issue ASTERISK-21876)
Reported by: Matt Jordan
Review: https://reviewboard.asterisk.org/r/2648/

Modified:
    trunk/funcs/func_channel.c
    trunk/include/asterisk/bridging_basic.h
    trunk/main/bridging_basic.c

Modified: trunk/funcs/func_channel.c
URL: http://svnview.digium.com/svn/asterisk/trunk/funcs/func_channel.c?view=diff&rev=395215&r1=395214&r2=395215
==============================================================================
--- trunk/funcs/func_channel.c (original)
+++ trunk/funcs/func_channel.c Tue Jul 23 16:32:33 2013
@@ -44,6 +44,7 @@
 #include "asterisk/indications.h"
 #include "asterisk/stringfields.h"
 #include "asterisk/global_datastores.h"
+#include "asterisk/bridging_basic.h"
 
 /*** DOCUMENTATION
 	<function name="CHANNELS" language="en_US">
@@ -101,6 +102,12 @@
 					<enum name="audiowriteformat">
 						<para>R/O format currently being written.</para>
 					</enum>
+					<enum name="dtmf_features">
+						<para>R/W The channel's DTMF bridge features.
+						May include one or more of 'T' 'K' 'H' 'W' and 'X' in a similar manner to options
+						in the <literal>Dial</literal> application. When setting it, the features string
+						must be all upper case.</para>
+					</enum>
 					<enum name="callgroup">
 						<para>R/W numeric call pickup groups that this channel is a member.</para>
 					</enum>
@@ -436,9 +443,13 @@
 		ast_copy_string(buf, ast_channel_trace_is_enabled(chan) ? "1" : "0", len);
 		ast_channel_unlock(chan);
 #endif
-	} else if (!strcasecmp(data, "tonezone") && ast_channel_zone(chan))
+	} else if (!strcasecmp(data, "tonezone") && ast_channel_zone(chan)) {
 		locked_copy_string(chan, buf, ast_channel_zone(chan)->country, len);
-	else if (!strcasecmp(data, "language"))
+	} else if (!strcasecmp(data, "dtmf_features")) {
+		if (ast_bridge_features_ds_get_string(chan, buf, len)) {
+			buf[0] = '\0';
+		}
+	} else if (!strcasecmp(data, "language"))
 		locked_copy_string(chan, buf, ast_channel_language(chan), len);
 	else if (!strcasecmp(data, "musicclass"))
 		locked_copy_string(chan, buf, ast_channel_musicclass(chan), len);
@@ -615,6 +626,8 @@
 			ast_channel_unlock(chan);
 			new_zone = ast_tone_zone_unref(new_zone);
 		}
+	} else if (!strcasecmp(data, "dtmf_features")) {
+		ret = ast_bridge_features_ds_set_string(chan, value);
 	} else if (!strcasecmp(data, "callgroup")) {
 		ast_channel_lock(chan);
 		ast_channel_callgroup_set(chan, ast_get_group(value));

Modified: trunk/include/asterisk/bridging_basic.h
URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/bridging_basic.h?view=diff&rev=395215&r1=395214&r2=395215
==============================================================================
--- trunk/include/asterisk/bridging_basic.h (original)
+++ trunk/include/asterisk/bridging_basic.h Tue Jul 23 16:32:33 2013
@@ -36,15 +36,41 @@
 /* ------------------------------------------------------------------- */
 
 /*!
+ * \brief Sets the features a channel will use upon being bridged.
+ * \since 12.0.0
+ *
+ * \param chan Which channel to set features for
+ * \param features Which feature codes to set for the channel
+ *
+ * \retval 0 on success
+ * \retval -1 on failure
+ */
+int ast_bridge_features_ds_set_string(struct ast_channel *chan, const char *features);
+
+/*!
+ * \brief writes a channel's DTMF features to a buffer string
+ * \since 12.0.0
+ *
+ * \param chan channel whose feature flags should be checked
+ * \param buffer pointer string buffer where the output should be stored
+ * \param buf_size size of the provided buffer (ideally enough for all features, 6+)
+ *
+ * \retval 0 on successful write
+ * \retval -1 on failure
+ */
+int ast_bridge_features_ds_get_string(struct ast_channel *chan, char *buffer, size_t buf_size);
+
+/*!
  * \brief Get DTMF feature flags from the channel.
  * \since 12.0.0
  *
  * \param chan Channel to get DTMF features datastore.
  *
  * \note The channel should be locked before calling this function.
+ * \note The channel must remain locked until the flags returned have been consumed.
  *
  * \retval flags on success.
- * \retval NULL on error.
+ * \retval NULL if the datastore does not exist.
  */
 struct ast_flags *ast_bridge_features_ds_get(struct ast_channel *chan);
 

Modified: trunk/main/bridging_basic.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/bridging_basic.c?view=diff&rev=395215&r1=395214&r2=395215
==============================================================================
--- trunk/main/bridging_basic.c (original)
+++ trunk/main/bridging_basic.c Tue Jul 23 16:32:33 2013
@@ -81,6 +81,143 @@
 	.type = "bridge-dtmf-features",
 	.destroy = ast_free_ptr,
 };
+
+/*!
+ * \internal
+ * \since 12.0.0
+ * \brief read a feature code character and set it on for the give feature_flags struct
+ *
+ * \param feature_flags flags being modifed
+ * \param feature feature code provided - should be an uppercase letter
+ *
+ * \retval 0 if the feature was set successfully
+ * \retval -1 failure because the requested feature code isn't handled by this function
+ */
+static int set_feature_flag_from_char(struct ast_flags *feature_flags, char feature)
+{
+	switch (feature) {
+	case 'T':
+		ast_set_flag(feature_flags, AST_FEATURE_REDIRECT);
+		return 0;
+	case 'K':
+		ast_set_flag(feature_flags, AST_FEATURE_PARKCALL);
+		return 0;
+	case 'H':
+		ast_set_flag(feature_flags, AST_FEATURE_DISCONNECT);
+		return 0;
+	case 'W':
+		ast_set_flag(feature_flags, AST_FEATURE_AUTOMON);
+		return 0;
+	case 'X':
+		ast_set_flag(feature_flags, AST_FEATURE_AUTOMIXMON);
+		return 0;
+	default:
+		return -1;
+	}
+}
+
+/*!
+ * \internal
+ * \since 12.0.0
+ * \brief Write a features string to a string buffer based on the feature flags provided
+ *
+ * \param feature_flags pointer to the feature flags to write from.
+ * \param buffer pointer to a string buffer to write the features
+ * \param buffer_size size of the buffer provided (should be able to fit all feature codes)
+ *
+ * \retval 0 on successful write
+ * \retval -1 failure due to running out of buffer space
+ */
+static int dtmf_features_flags_to_string(struct ast_flags *feature_flags, char *buffer, size_t buffer_size)
+{
+	size_t buffer_expended = 0;
+	unsigned int cur_feature;
+	static const struct {
+		char letter;
+		unsigned int flag;
+	} associations[] = {
+		{ 'T', AST_FEATURE_REDIRECT },
+		{ 'K', AST_FEATURE_PARKCALL },
+		{ 'H', AST_FEATURE_DISCONNECT },
+		{ 'W', AST_FEATURE_AUTOMON },
+		{ 'X', AST_FEATURE_AUTOMIXMON },
+	};
+
+	for (cur_feature = 0; cur_feature < ARRAY_LEN(associations); cur_feature++) {
+		if (ast_test_flag(feature_flags, associations[cur_feature].flag)) {
+			if (buffer_expended == buffer_size - 1) {
+				buffer[buffer_expended] = '\0';
+				return -1;
+			}
+			buffer[buffer_expended++] = associations[cur_feature].letter;
+		}
+	}
+
+	buffer[buffer_expended] = '\0';
+	return 0;
+}
+
+static int build_dtmf_features(struct ast_flags *flags, const char *features)
+{
+	const char *feature;
+
+	char missing_features[strlen(features) + 1];
+	size_t number_of_missing_features = 0;
+
+	for (feature = features; *feature; feature++) {
+		if (!isupper(*feature)) {
+			ast_log(LOG_ERROR, "Features string '%s' rejected because it contains non-uppercase feature.\n", features);
+			return -1;
+		}
+
+		if (set_feature_flag_from_char(flags, *feature)) {
+			missing_features[number_of_missing_features++] = *feature;
+		}
+	}
+
+	missing_features[number_of_missing_features] = '\0';
+
+	if (number_of_missing_features) {
+		ast_log(LOG_WARNING, "Features '%s' from features string '%s' can not be applied.\n", missing_features, features);
+	}
+
+	return 0;
+}
+
+int ast_bridge_features_ds_set_string(struct ast_channel *chan, const char *features)
+{
+	struct ast_flags flags = {0};
+
+	if (build_dtmf_features(&flags, features)) {
+		return -1;
+	}
+
+	ast_channel_lock(chan);
+	if (ast_bridge_features_ds_set(chan, &flags)) {
+		ast_channel_unlock(chan);
+		ast_log(LOG_ERROR, "Failed to apply features datastore for '%s' to channel '%s'\n", features, ast_channel_name(chan));
+		return -1;
+	}
+	ast_channel_unlock(chan);
+
+	return 0;
+}
+
+int ast_bridge_features_ds_get_string(struct ast_channel *chan, char *buffer, size_t buf_size)
+{
+	struct ast_flags *channel_flags;
+	struct ast_flags held_copy;
+
+	ast_channel_lock(chan);
+	if (!(channel_flags = ast_bridge_features_ds_get(chan))) {
+		ast_channel_unlock(chan);
+		return -1;
+	}
+	held_copy = *channel_flags;
+	ast_channel_unlock(chan);
+
+	return dtmf_features_flags_to_string(&held_copy, buffer, buf_size);
+}
 
 int ast_bridge_features_ds_set(struct ast_channel *chan, struct ast_flags *flags)
 {




More information about the asterisk-commits mailing list