Extra quotes around strings containing "new"

[bcosca]

Extra quotes around strings containing "new"

[xfra35]

I forgot to mention that it concerns also e-mail addresses since the @ sign also belongs to the same check procedure. E.g:

F3::set('_str1','E-mail:');
F3::set('_str2','[email protected]');
echo F3::resolve('{{@_str1}} {{@_str2}}');

will output E-mail: '[email protected]'

[ikkez]

This is a known issue. See https://groups.google.com/forum/#!topic/f3-framework/0I4oth-eojY for more information on Google Group.

[xfra35]

Sorry I didn't see it. Anyway there's no mention in that post of the problem with the word 'new'.
Could you (or somebody else) explain the purpose of the following test in F3::resolve ?

!preg_match('/@|\bnew\s+/i',$out)

I'm probably wrong but here's how I see it :

• concerning the '@' test, it seems to be here to prevent evaluating expressions containing F3 variables, although the @-starting F3 variables are handled by the previous preg_replace_callback and therefore should never appear in $out.
• concerning the 'new' test, it looks like an attempt to prohibit object instantiation within templates in order to stick to MVC principles. If that's the case, the benefit of this control has to be balanced against the impossibility of using the word 'new' in the dictionary.

[bcosca]

Please let me know if the latest commit provides finality on this issue

[xfra35]

Yes, it's fixed now. Thanks !