Remove MODIFIER_ prefix from node dumps

These constants are now called Modifiers::PUBLIC rather than
Class_::MODIFIER_PUBLIC etc, so update the dumped name as well.

Support empty list insertion for attributes

Add test for StaticVar rename

Also run these in separate processes, they're not really meaningful
if the classes are already loaded.

Rename Stmt\StaticVar to StaticVar

This is part of a statement, not a statement by itself.

Rename Expr\ArrayItem to ArrayItem

Array items are not expressions by themselves.

Make sure Array nodes can not contain null

Now that destructuring is always represented using List nodes,
make sure that Array nodes can no longer contain null elements,
so well-type

Make sure Array nodes can not contain null

Now that destructuring is always represented using List nodes,
make sure that Array nodes can no longer contain null elements,
so well-typed code doesn't have to deal with them unnecessarily.

If an array does contain empty elements, these are now result in
an error and are represented as a ArrayItem with Error value if
error recovery is used.

The implementation is a bit tricky because at the time the Array
node is created, we cannot tell whether it will be used in a
creation or destructuring context. For this reason the error
reporting is delayed parsing has finished.

Closes #876.

show more ...

Update grammar/README.md (#877)

Add some test coverage for Token class

Add array/callable to BUILTIN_TYPE_VERSIONS

Listing these is not strictly necessary, in that array/callable
are keywords, and as such don't use the relevant code path. We
can still i

Add array/callable to BUILTIN_TYPE_VERSIONS

Listing these is not strictly necessary, in that array/callable
are keywords, and as such don't use the relevant code path. We
can still include them for the sake of completeness.

Closes #872.

show more ...

Add CONTRIBUTING.md

For now just a mention of the non-standard coding style.

Also format the grammar directory

Format tests as well

The unnecessary parentheses for "new" are a bit annoying, but I
can live with it...

Require strict_types in php-cs-fixer config

Add php-cs-fixer config and reformat

The formatting in this project has become something of a mess,
because it changed over time. Add a CS fixer config and reformat
to the desired st

Add php-cs-fixer config and reformat

The formatting in this project has become something of a mess,
because it changed over time. Add a CS fixer config and reformat
to the desired style, which is PSR-12, but with sane brace placement.

show more ...

Introduce separate Modifiers class

Use PhpParser\Modifiers::PUBLIC instead of
PhpParser\Node\Stmt\Class_::MODIFIER_PUBLIC, etc. Old constants
of course remain available.

Fix

Introduce separate Modifiers class

Use PhpParser\Modifiers::PUBLIC instead of
PhpParser\Node\Stmt\Class_::MODIFIER_PUBLIC, etc. Old constants
of course remain available.

Fixes #476.

show more ...

Always use List_ node for array destructuring

Fixes #471.

Fix length bounds check in Name::slice()

The length check did not take into account that there may be a
non-zero offset at this point.

Fixes #875.

Fix typo

[docs] Add generic types to NodeFinder to allow returning exact type in static analysis (#869)

Add support for DNF types (#862)

Print trailing comma in param list if supported

Always use pMaybeMultiline() for function parameters

Closes GH-861.

Mark NodeVisitorAbstract as abstract class

It doesn't actually have any abstract methods, but doesn't do
anything by itself and is intended for extension only.

Fixes #865.

Don't trim in Comment::getReformattedText()

In the past, single-line comments were stored together with the
trailing newline. Later we switched to the PHP8 comment
representation, wh

Don't trim in Comment::getReformattedText()

In the past, single-line comments were stored together with the
trailing newline. Later we switched to the PHP8 comment
representation, where the trailing newline is not part of the
comment anymore. As such, there is also no need to trim here.

This is split out from GH-867.

show more ...

Add json and ctype as required extensions in composer.json (#864)