Greg Korba
Greg Korba
[As I found out](https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/6621#issuecomment-1237500506) in #6621 **DNF usage in class property causes fatal error**. At first I thought it was caused by complex intersections, but no - `A|(B&C&D)` works OK...
Yes, the code provided in `tests/Fixtures/Integration/misc/PHP8_2.test` work properly with `token_get_all()`: ```php
It also return 289 tokens, so the problem is not in tokenizing. I don't know internals enough, but [fail points to](https://github.com/FriendsOfPHP/PHP-CS-Fixer/runs/8196636666?check_suite_focus=true#step:11:562) `PhpCsFixer\Linter\ProcessLintingResult->check()`. I'll try to look at it more.
After #6621 gets merged, we need to create another MR that would add and fix such fixtures: SHOW DIFF ```diff Index: tests/Fixtures/Integration/misc/PHP8_2.test IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP UTF-8 =================================================================== diff...
@SpacePossum could you add checklist to the issue's description, like [here](https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/6605#issue-1359429795)? This will allow us track required work.
@SpacePossum checklist in the issue description, with link to [my comment](#issuecomment-1241958302), like in 6605 or 6613: It's hard to track what's left from the comments, so it's better when it's...
@Girgias yeah, that's why [I proposed](#issuecomment-1241958302) more complex, "real world" combinations without reusing the same intersection 🙂 Since integration test is merged (6621), these fixtures can be re-added to the...
@SpacePossum we should wait for #6621 to be merged, then rebase this one and change fixture in integration test, so keywords are in wrong order (now only casing is verified,...
@SpacePossum why did you close this PR? 🤔
@SpacePossum I guess it does bring something new. Example from RFC: ```php enum A: string { case B = 'B'; // This is currently not permitted const C = [self::B->value...