Summary:ASTERISK-01847: Something changed-- double quotes are now removed before eval-- exprs affected
Reporter:Steve Murphy (murf)Labels:
Date Opened:2004-06-17 13:25:27Date Closed:2008-01-15 14:59:55.000-0600
Versions:Frequency of
Description:I updated from CVS a little while back, and now notice that
the expr upgrade I submitted earlier is now broken with respect to double quotes! They appear to be stripped now before eval, which destroys most of the usefulness of the features I added. I haven't investigated yet what was modded to make this happen... I'll investigate when I get a moment.
Comments:By: Mark Spencer (markster) 2004-06-17 14:33:55

Quotes are now used to help work around the processing of "," into "|".  You can escape your quotes with \.  Of course you could also escape a comma with \, so the new quote handling is not not really required, but it does, presumably, lead to more readable stuff.  For example:

exten => 1,1,Foo("this, that","and the other, for now")

As opposed to:

exten => 1,1,Foo(this\, that,and the other\, for now)

If you want to see more take a look at process_quotes_and_slashes in pbx/pbx_config.c

By: Mark Spencer (markster) 2004-06-18 09:58:40

Are you still able to do your example with quotes by escaping them?  Can we close this out?

By: Mark Spencer (markster) 2004-06-21 10:34:23

Should be fixed when CVS mirrors update.  I changed pbx_config to use ' instead of " as the quote symbol.

By: Digium Subversion (svnbot) 2008-01-15 14:59:55.000-0600

Repository: asterisk
Revision: 3257

U   trunk/pbx/pbx_config.c

r3257 | markster | 2008-01-15 14:59:54 -0600 (Tue, 15 Jan 2008) | 2 lines

Use ' instead of " for quoting in extensions.conf (bug ASTERISK-1847)