[asterisk-bugs] [Asterisk 0016646]: [patch] Its not possible to pass more than one agrument in custom features.

Wed Apr 14 18:00:00 CDT 2010

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16646 
====================================================================== 
Reported By:                pinga-fogo
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   16646
Category:                   Resources/res_features
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Target Version:             1.4.32
Asterisk Version:           SVN 
JIRA:                       SWP-758 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-01-19 04:46 CST
Last Modified:              2010-04-14 18:00 CDT
====================================================================== 
Summary:                    [patch] Its not possible to pass more than one
agrument in custom features.
Description: 
Is not possible to pass mor than one argument in a custon features, if i
use the pipe | to separate the arguments, this downt work in dialplan, and
if i use "," to separate the arguments the feature accept the first
argument only, 
example:
testfeature => *25,peer/callee,goto,tstfax|s|1
(in execution the dialplan not recognise the pipe separator (deprecated))

testfeature => *25,peer/callee,goto,tstfax,s,1
(the parameters are parsed incorrect, MOH_Class = s)

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
duplicate of        0016984 impossible to pass more than one argume...
====================================================================== 

---------------------------------------------------------------------- 
 (0120432) svnbot (reporter) - 2010-04-14 18:00
 https://issues.asterisk.org/view.php?id=16646#c120432 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 257263

_U  branches/1.6.0/
U   branches/1.6.0/configs/features.conf.sample
U   branches/1.6.0/main/features.c

------------------------------------------------------------------------
r257263 | tilghman | 2010-04-14 17:59:59 -0500 (Wed, 14 Apr 2010) | 22
lines

Merged revisions 257262 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r257262 | tilghman | 2010-04-14 17:57:35 -0500 (Wed, 14 Apr 2010) | 15
lines

  Yet another issue where the conversion of the application delimiter to
comma caused an issue.

  Application arguments within the feature map could possibly contain a
comma,
  which conflicts with the syntax of the features.conf configuration file.
 This
  patch allows the argument to be wrapped in parentheses or quoted, to
allow the
  application arguments to be interpreted as a single configuration
parameter.

  (closes issue https://issues.asterisk.org/view.php?id=16646)
   Reported by: pinga-fogo
   Patches: 
         20100414__issue16646.diff.txt uploaded by tilghman (license 14)
   Tested by: tilghman

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

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=257263 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-04-14 18:00 svnbot         Checkin                                      
2010-04-14 18:00 svnbot         Note Added: 0120432                          
======================================================================