prettier / prettier/plugin-php
Prettier adds incorrect semicolon with nested foreach, for, try/catch
オープン
まだ誰も着手していません。
bug
- 主要言語
- PHP
- スター
- 1.9k
- フォーク
- 139
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
@prettier/plugin-php v0.18.8
Playground link
Input:
<?php
foreach( $x as $y)
foreach ($a as $b) {
$a = $x;
}
foreach ($x as $y)
for ($a = 1; $a < 10; a++) {
$a = $x;
};
if( $foo)
foreach($b as $a)
{
$foo = bar;
}
if($foo)
try {
} catch (Exception $e){
}
Output:
<?php
foreach ($x as $y) {
foreach ($a as $b) {
$a = $x;
};
}
foreach ($x as $y) {
for ($a = 1; $a < 10; a++) {
$a = $x;
};
}
if ($foo) {
foreach ($b as $a) {
$foo = bar;
};
}
if ($foo) {
try {
} catch (Exception $e) {
};
}
I think this shows the issue. The expected output should be the same as the output, without the extra semicolons.
If you run prettier on the same file a second time, it will removed the extra semicolons. I know this affects foreach, for, try/catch, but I'm not sure that is a comprehensive list.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
リンクされた Playground の例から始め、最初のフォーマット処理と 2 回目の処理を比較します。ネストされた foreach、for、try/catch、および関連するケースを再現し、フォーマットによって余分なセミコロンが生成されず、もう一度実行しても変更されないことを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- tooling
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100