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:27 | Date Closed: | 2008-01-15 14:59:55.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ||
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) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=3257 |