prettier / prettier/plugin-php

`{` and `}` for variable property

Ouverte
#790 1 commentaire 14 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

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

Description

Prettier 1.14.3

PHP Plugin 0.8.0

Input:

$var = $this->{$key};
$var = $this->value->{$method}();

Output:

$var = $this->$key;
$var = $this->value->$method();

Expected behavior:

Need vote

👍 for $this->{$key}
❤️ for $this->$key

I found what laravel, symfony and a lot of other projects use { and }, looks it is popular.

One note - we should respect { and } in source code, because it can be break php5 code http://php.net/manual/en/migration70.incompatible.php#migration70.incompatible.variable-handling.indirect.

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

Reproduisez l’entrée signalée avec Prettier 1.14.3 et PHP Plugin 0.8.0, puis examinez comment les propriétés des variables sont formatées. L’issue ne nomme aucun fichier ni test et ne définit aucune sortie attendue arrêtée ; confirmez la décision du projet concernant la conservation des accolades avant de définir l’achèvement.

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é
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

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