prettier / prettier/plugin-php
Non-associative arrays formatting is not consistent with associative arrays
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 1.9k
- Forks
- 139
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Prettier 2.6.2
PHP Plugin 0.18.5
Input:
class Test
{
protected $array1 = [
'published', 'title', 'description', 'position', 'nid'];
protected $array2 = [
0=>'published', 'title', 'description', 'position', 'nid'];
}
Output:
class Test
{
protected $array1 = ['published', 'title', 'description', 'position', 'nid'];
protected $array2 = [
0 => 'published',
'title',
'description',
'position',
'nid',
];
}
Expected behavior:
class Test
{
protected $array1 = [
'published',
'title',
'description',
'position',
'nid',
];
protected $array2 = [
0 => 'published',
'title',
'description',
'position',
'nid',
];
}
If we break any array (associative or not) into multiple lines it should format the array as multiple lines, as it does for associative arrays. Also, this comment states that the multi-line format should be preserved.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere die gemeldete Eingabe mit den im Issue aufgeführten Versionen des Prettier PHP plugin und vergleiche die Ausgabe mit der erwarteten Formatierung. Beginne damit, das Verhalten des Plugins bei der Array-Formatierung nachzuverfolgen, einschließlich des verlinkten Kommentars aus Issue 1401. Als erledigt gilt, wenn mehrzeilige nicht-assoziative Arrays ihre Zeilenumbrüche beibehalten und der gezeigten erwarteten Ausgabe entsprechen.
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
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100