diff -Nru a/apps/app_dial.c b/apps/app_dial.c --- a/apps/app_dial.c 2005-01-08 09:34:13 -07:00 +++ b/apps/app_dial.c 2005-01-08 09:34:13 -07:00 @@ -488,9 +488,6 @@ char *newnum; char *l; char *url=NULL; /* JDG */ - struct ast_var_t *current; - struct varshead *headp, *newheadp; - struct ast_var_t *newvar; unsigned int calldurationlimit=0; char *cdl; time_t now; @@ -511,9 +508,7 @@ char toast[80]; int play_to_caller=0,play_to_callee=0; int playargs=0, sentringing=0, moh=0; - char *varname; char *mohclass = NULL; - int vartype; char *outbound_group = NULL; char *macro_result = NULL, *macro_transfer_dest = NULL; int digit = 0; @@ -853,40 +848,8 @@ } } - /* Contitionally copy channel variables to the newly created channel */ - headp = &chan->varshead; - AST_LIST_TRAVERSE(headp, current, entries) { - varname = ast_var_full_name(current); - vartype = 0; - if (varname) { - if (varname[0] == '_') { - vartype = 1; - if (varname[1] == '_') - vartype = 2; - } - } - if (vartype == 1) { - newvar = ast_var_assign((char*)&(varname[1]), - ast_var_value(current)); - newheadp = &tmp->chan->varshead; - AST_LIST_INSERT_HEAD(newheadp, newvar, entries); - if (option_debug) - ast_log(LOG_DEBUG, "Copying soft-transferable variable %s.\n", - ast_var_name(newvar)); - } else if (vartype == 2) { - newvar = ast_var_assign(ast_var_full_name(current), - ast_var_value(current)); - newheadp = &tmp->chan->varshead; - AST_LIST_INSERT_HEAD(newheadp, newvar, entries); - if (option_debug) - ast_log(LOG_DEBUG, "Copying hard-transferable variable %s.\n", - ast_var_name(newvar)); - } else { - if (option_debug) - ast_log(LOG_DEBUG, "Not copying variable %s.\n", - ast_var_name(current)); - } - } + /* Inherit specially named variables from parent channel */ + ast_channel_inherit_variables(chan, tmp->chan); tmp->chan->appl = "AppDial"; tmp->chan->data = "(Outgoing Line)"; diff -Nru a/apps/app_queue.c b/apps/app_queue.c --- a/apps/app_queue.c 2005-01-08 09:34:13 -07:00 +++ b/apps/app_queue.c 2005-01-08 09:34:13 -07:00 @@ -823,8 +823,13 @@ tmp->chan->cid.cid_name = strdup(qe->chan->cid.cid_name); if (qe->chan->cid.cid_ani) tmp->chan->cid.cid_ani = strdup(qe->chan->cid.cid_ani); + + /* Inherit specially named variables from parent channel */ + ast_channel_inherit_variables(qe->chan, tmp->chan); + /* Presense of ADSI CPE on outgoing channel follows ours */ tmp->chan->adsicpe = qe->chan->adsicpe; + /* Place the call, but don't wait on the answer */ res = ast_call(tmp->chan, location, 0); if (res) { diff -Nru a/channel.c b/channel.c --- a/channel.c 2005-01-08 09:34:13 -07:00 +++ b/channel.c 2005-01-08 09:34:14 -07:00 @@ -2243,6 +2243,49 @@ manager_event(EVENT_FLAG_CALL, "Rename", "Oldname: %s\r\nNewname: %s\r\nUniqueid: %s\r\n", tmp, chan->name, chan->uniqueid); } +void ast_channel_inherit_variables(const struct ast_channel *parent, struct ast_channel *child) +{ + struct ast_var_t *current, *newvar; + char *varname; + + AST_LIST_TRAVERSE(&parent->varshead, current, entries) { + int vartype = 0; + + varname = ast_var_full_name(current); + if (!varname) + continue; + + if (varname[0] == '_') { + vartype = 1; + if (varname[1] == '_') + vartype = 2; + } + + switch (vartype) { + case 1: + newvar = ast_var_assign(&varname[1], ast_var_value(current)); + if (newvar) { + AST_LIST_INSERT_HEAD(&child->varshead, newvar, entries); + if (option_debug) + ast_log(LOG_DEBUG, "Copying soft-transferable variable %s.\n", ast_var_name(newvar)); + } + break; + case 2: + newvar = ast_var_assign(ast_var_full_name(current), ast_var_value(current)); + if (newvar) { + AST_LIST_INSERT_HEAD(&child->varshead, newvar, entries); + if (option_debug) + ast_log(LOG_DEBUG, "Copying hard-transferable variable %s.\n", ast_var_name(newvar)); + } + break; + default: + if (option_debug) + ast_log(LOG_DEBUG, "Not copying variable %s.\n", ast_var_name(current)); + break; + } + } +} + /* Clone channel variables from 'clone' channel into 'original' channel All variables except those related to app_groupcount are cloned Variables are actually _removed_ from 'clone' channel, presumably diff -Nru a/include/asterisk/channel.h b/include/asterisk/channel.h --- a/include/asterisk/channel.h 2005-01-08 09:34:14 -07:00 +++ b/include/asterisk/channel.h 2005-01-08 09:34:14 -07:00 @@ -782,6 +782,20 @@ /* Find bridged channel */ struct ast_channel *ast_bridged_channel(struct ast_channel *chan); +/*! + \brief Inherits channel variable from parent to child channel + \param parent Parent channel + \param child Child channel + + Scans all channel variables in the parent channel, looking for those + that should be copied into the child channel. + Variables whose names begin with a single '_' are copied into the + child channel with the prefix removed. + Variables whose names begin with '__' are copied into the child + channel with their names unchanged. +*/ +void ast_channel_inherit_variables(const struct ast_channel *parent, struct ast_channel *child); + /* Misc. functions below */ /* Helper function for migrating select to poll */