prettier / prettier/plugin-php

Invoked import/require callable removes necessary parentheses

Offen
#2,333 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

An imported or required file returning a callable value cannot be immediately invoked as Prettier is stripping the surrounding parentheses.

(The below example assumes file.php contains e.g. <?php return function() { echo 'Hello world'; };)

@prettier/plugin-php v0.22.2
Playground link

Input:

<?php
(include 'file.php')();
(require 'file.php')();

Output:

<?php
include "file.php"();
require "file.php"();

Expected behavior:

<?php
(include 'file.php')();
(require 'file.php')();

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

Das Issue stellt eine minimale PHP-Eingabe, eine falsch formatierte Ausgabe und die erwartete Ausgabe bereit, nennt jedoch weder eine Quelldatei noch einen Testpfad. Beginne damit, die Formatter-Logik für eingeklammerte include/require-Ausdrücke gefolgt von einem Aufruf zu finden, und füge anschließend eine Regressionstestabdeckung hinzu, die die Klammern beibehält und die gezeigte erwartete Ausgabe verifiziert.

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
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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