Claus Due
Claus Due
@mbrodala I think this is ready - trimmed down to only adding test cases for chained math.
The sequencer (symbol-and-capture yielding, context-switching tokenisation) strategy I am writing as a replacement for the current regular expression based parsing will support this. The reason for the current lack of...
Not exactly - here's a more complete version: ``` {{foo: bar,} | v:h()} // supported, last comma ignored {{foo, bar,} | v:h()} // supported, last comma ignored (ECMA literal) {[foo,...
All the old syntax is still supported! I just didn't mention those because they're so well known already ;) ``` {v:h(arg: x, arg2: y, )} // also supported despite tailing...
`{variable | raw()}` would work the same (didn't test but the sequencer should see the same situation except this variant is slightly more expensive). Avoid the 'roid! ^^
Syntax like `` appears to be supported in 2.x - would you mind confirming, @mbrodala? If you can confirm I say we close this issue - and lean back and...
Perfect - happy to hear that :)
Do you have Fluid 2.6.8 on the system that reports this? (I did a test earlier today and it works on 2.6.8 at least on my system). Does it fail...
Agreed, but I think the right solution is to correct the return type annotation to `string|null` - it is nice to be able to tell if the template truly didn't...
I took a look at your failing test case - but VariableExtractor is no longer in use, so a failure to resolve the variable in that class shouldn't cause this...