vscode-phpfmt icon indicating copy to clipboard operation
vscode-phpfmt copied to clipboard

PSR2ModifierVisibilityStaticOrder mangles nullable type declarations in function parameters

Open metanomial opened this issue 6 months ago • 3 comments

e.g.,

PSR2ModifierVisibilityStaticOrder disabled:

class Foo {
    static public function bar(?string $baz) {
        noop();
    }
}

function foo(?string $bar): void {
    noop();
}

PSR2ModifierVisibilityStaticOrder enabled:

class Foo {
    public static function bar(?string $baz) {
        noop();
    }
}

function foo(string ? $bar): void {
    noop();
}

metanomial avatar Jun 12 '25 07:06 metanomial

Ups, this one is good, thank you.

driade avatar Jun 12 '25 07:06 driade

Hi! This should be fixed in the latest release, may you confirm? Thanks in advance.

driade avatar Jun 15 '25 14:06 driade

Working as expected!

metanomial avatar Jun 15 '25 18:06 metanomial