prettier / prettier/plugin-php

Parens incorrectly removed when cloning the result of an expression

Ouverte
#2,097 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug
Langage dominant
PHP
Étoiles
1.9k
Forks
139
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Prettier 2.7.1

PHP Plugin 0.19.1

# Options (if any):
# our .prettierrc
{
    "phpVersion": "8.1",
    "printWidth": 80,
    "tabWidth": 4,
    "useTabs": false,
    "singleQuote": true,
    "trailingCommaPHP": true,
    "braceStyle": "psr-2",
    "requirePragma": false,
    "insertPragma": false
}

Input:

$questions = clone ($panel->primaryViewer()?->questions() ?? new Questions());

Output:

$questions = clone $panel->primaryViewer()?->questions() ?? new Questions();

Expected behavior:

The outermost parens are being removed by prettier, affecting behavior. Given

$questions = clone ($panel->primaryViewer()?->questions() ?? new Questions());

if $panel->primaryViewer() is null, because prettier drops the outermost parens, it ends up leaving PHP to evaluate clone null - something I'm actively guarding against with the null coalesce operator.

I expect the parens to remain in place, resulting in the ?? new Questions() to be cloned.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par reproduire l'entrée PHP signalée et comparer Prettier 2.7.1 avec la sortie attendue. Suivez la gestion par le formateur des parenthèses autour de la clone expression ; le travail est terminé lorsque les parenthèses extérieures sont conservées, afin que la null-coalesce expression soit clonée, avec un test de régression couvrant ce cas.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
tooling
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.