prettier / prettier/plugin-php

Parens incorrectly removed when cloning the result of an expression

Abierto
#2,097 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug
Lenguaje dominante
PHP
Estrellas
1.9k
Forks
139
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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.

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza reproduciendo la entrada PHP indicada y comparando Prettier 2.7.1 con la salida esperada. Sigue el tratamiento que hace el formateador de los paréntesis alrededor de la clone expression; se considera terminado cuando los paréntesis exteriores se conservan, de modo que se clone la null-coalesce expression, con una prueba de regresión que cubra este caso.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
php
Área
tooling
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.