prettier / prettier/plugin-php

[BUG] Please remove Unnecessary Parenthesis like Foo;

Offen
#2,454 3 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
PHP
Sterne
1.9k
Forks
139
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

**Prettier 3.8.1 (lastest) **
PHP Plugin 0.24.0

# Options (if any):
--php-version 8.3
--single-quote

Input:

// TRUE
echo (match ('option1') {
    'option1' => fn() => 'First',
    'option2' => fn() => 'Second',
})();


// code snippet
$x = new Foo; // True style
$y = new Foo(1); // True style
$z = (new Foo)->value; // True style

Output:

// ERROR
echo match ('option1') {
    'option1' => fn() => 'First',
    'option2' => fn() => 'Second',
}();


$x = new Foo(); // False sytle: I don't want add ()
$y = new Foo(1); // True style
$z = (new Foo())->value; // False style: I don't want add ()

Expected behavior:

I dont want add unnecessary parenthesis

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die PHP-Snippets mit Prettier 3.8.1, PHP Plugin 0.24.0, PHP 8.3 und den angegebenen Optionen zu reproduzieren. Vergleiche die Formatierung des match-Ausdrucks und jedes neuen Foo-Beispiels; abgeschlossen ist die Aufgabe, wenn keine unnötigen Konstruktor- und Zugriffs-Klammern hinzugefügt werden, während die übrige Formatierung korrekt bleibt, mit Regressionsabdeckung für diese Fälle.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.