Symfony Expression: Parse without context #50105
Labels
Comments
6 tasks
|
For the record, this is my function to be able to parse without giving names context. This can be extended to create a simple "variable extractor" that can fetch variable values from another source: use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
use Symfony\Component\ExpressionLanguage\ParsedExpression;
use Symfony\Component\ExpressionLanguage\SyntaxError;
use function preg_match;
private function parseExpressionWithoutContext(string $expression): ParsedExpression
{
$expressionLanguage = new ExpressionLanguage();
$names = [];
while (true) {
try {
return $expressionLanguage->parse($expression, $names);
} catch (SyntaxError $e) {
$matches = [];
if (! preg_match('/^Variable "(?<variable>[^"]+)" is not valid/', $e->getMessage(), $matches)) {
throw $e;
}
$names[] = $matches['variable'];
}
}
} |
|
Fixed by #53806 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Current
parsefunction requires full context to complete successfully. It needs to know about declaredfunctions andnames.I would like to be able to parse the nodes without those details. I have 2 use case for this:
is_granted()receives 2 argumentsFrom my understanding, the parser need to be updated to avoid throwing on the following lines:
https://github.com/symfony/expression-language/blob/83e1fee4c018aa60bcbbecd585a2c54af6aca905/Parser.php#L228-L249
Alternatively it could be possible to code my own parser based on the existing one, but it cannot be overridden:
https://github.com/symfony/expression-language/blob/83e1fee4c018aa60bcbbecd585a2c54af6aca905/ExpressionLanguage.php#L150
Implementation
About the implementation, I guess that a
parseWithoutContext(TokenStream $stream)function could be the easier, it would just update an internal flag such as the currentlintflag and will use this flag duringfunction/nameresolution.Example
The following code would then be valid and its results could be used to construct what I want to achieve.
The text was updated successfully, but these errors were encountered: