Skip to content

[PHP 8.6] Partial Function Application (PFA) #229

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lisachenko merged 5 commits into from
Sep 23, 2026
Merged

[PHP 8.6] Partial Function Application (PFA) #229

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b056da2
test: document PFA parse behavior while blocked on PHP-Parser
claude Aug 25, 2026
890cee1
chore: merge master to pick up CI pipeline naming
claude Aug 25, 2026
0891589
chore: merge master
claude Sep 15, 2026
33d0d4c
feat: full PHP 8.6 partial function application support via php-parse…
claude Sep 15, 2026
b2ef543
build: require rector ^2.6.7 so its preloaded php-parser bundle is >=…
claude Sep 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,13 @@
},
"require": {
"php": ">=8.5",
"nikic/php-parser": "^5.4"
"nikic/php-parser": "^5.9"
},
"require-dev": {
"phpunit/phpunit": "^13.3.1",
"phpstan/phpstan": "^2.0",
"tracy/tracy": "^2.10",
"rector/rector": "^2.0"
"rector/rector": "^2.6.7"
},
"extra": {
"branch-alias": {
Expand Down
4 changes: 2 additions & 2 deletions src/Resolver/NodeExpressionResolver.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ protected function resolveExprFuncCall(Expr\FuncCall $node): mixed
$resolvedArgs = [];
foreach ($node->args as $argumentNode) {
if (!$argumentNode instanceof Node\Arg) {
throw new ReflectionException('Cannot statically resolve a variadic placeholder argument in a function call');
throw new ReflectionException('Cannot statically resolve a placeholder argument in a function call');
}
$value = $this->resolve($argumentNode->value);
// if function uses named arguments, then unpack argument name first
Expand Down Expand Up @@ -385,7 +385,7 @@ protected function resolveExprNew(Expr\New_ $node): object
$resolvedArgs = [];
foreach ($node->args as $argumentNode) {
if (!$argumentNode instanceof Node\Arg) {
throw new ReflectionException('Cannot statically resolve a variadic placeholder argument in a constructor call');
throw new ReflectionException('Cannot statically resolve a placeholder argument in a constructor call');
}
$value = $this->resolve($argumentNode->value);
// if constructor uses named arguments, then unpack argument name first
Expand Down
Loading