Properly support prefix operator precedence printing

The pretty printer is supposed to produce a minimal-parentheses
printing for expressions. However, for prefix operators, we were

Properly support prefix operator precedence printing

The pretty printer is supposed to produce a minimal-parentheses
printing for expressions. However, for prefix operators, we were
failing to do so in ways that are practically meaningful, e.g.
$a = yield from $b produced redundant parentheses around the
yield from.

For prefix operators, the precedence of the direct parent operator
is not actually relevant: What matters is the precedence of any
infix operator whose LHS it appears on. For example, $a . include $b
does not require parentheses, but (include $a) . $b does.

To handle this, keep separate track of the parent operator
precedence, and a special LHS precedence which is used for unary
operator printing only. Because the LHS precedence may not come
from the direct parent, we have to pass it down the call stack.
And if we do that, we may as well do the same for the parent
precedence.

This also fixes an integration test failure with php-src, because
arrow function precedence was previously not respected (and would
be ugly to respect without this change).

show more ...

Support fixup for dynamic class const name

This is new in PHP 8.3.

Properly handle static deref LHS

The rules for static and array/object deref are slightly different:
The former does not allow constants.

Support new variables in fixup

Properly handle new/instanceof operand restrictions

Fixes #912.

Respect precedence during clone pretty printing

Clone is the prefix operator with the highest operator precedence,
but was not treated as an operator at all.

Bail out on PHP tags in removed code

If dropping a node would drop PHP tags, bail out of formatting
preservation. This will lose formatting, but at least produce
legal code.

Bail out on PHP tags in removed code

If dropping a node would drop PHP tags, bail out of formatting
preservation. This will lose formatting, but at least produce
legal code.

Closes GH-884.

show more ...

Add support for perserving formatting for static modifier change

Closes GH-891.

Bail out on list insertion of non-Node

PhpStan correctly detected that this is not supported. We could
add support for it, but for now just make sure it doesn't crash.

Specify more types

Add missing return types

Use PHPStan level 5

Add phpstan baseline

Fix empty list insertion of multiple attributes

Fix formatting preservation for match

Switch list insertion maps to use class name

Also highlights that the list insertion entry for Expr\Match
was not used.

Switch modifierChangeMap to use class name

For better refactoring support, prefer class name over node type.

Fix ArrayItem entries in pretty printer maps

The tests were written in such a way that the regression was not
caught.

Rename Stmt\UseUse to UseItem

Rename Stmt\PropertyProperty to PropertyItem

Rename Scalar\Encapsed to Scalar\InterpolatedString

Rename Scalar\EncapsedStringPart to InterpolatedStringPart

It is no longer an expression node, which unfortunately does
require a more awkward type for the Encaps node.

Support empty list insertion for attributes

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 ...