prettier / prettier/plugin-php

Wrapping of static method calls

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

Nadie ha tomado este issue todavía.

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

Descripción

The plugin is currently wrapping variable static method calls before the double-colon ::. This seems wrong to me, but at least warrants some further discussion. It doesn't appear to be mentioned in PER-CS 2.0 at all, so it's not technically wrong. We do not do this wrapping direct static method calls from a class name, only from a variable.

@prettier/plugin-php v0.22.4
Playground link

Input:

<?php

$thisIsAReallyLongVariableNameUsedToTriggerTheBehaviour::staticMethod()
    ->anotherMethod();

Output:

<?php

$thisIsAReallyLongVariableNameUsedToTriggerTheBehaviour
    ::staticMethod()
    ->anotherMethod();

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

Comienza con la reproducción vinculada en Playground e inspecciona la ruta del formatter responsable de los saltos de línea alrededor de las llamadas a métodos estáticos. Reproduce el caso de una llamada estática mediante una variable, compáralo con las llamadas directas usando el nombre de la clase y considera que el issue está resuelto cuando el comportamiento de ajuste previsto esté establecido y cubierto por una prueba de regresión.

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
Bastante claro
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.