操作
Backport(バックポート) #3666
完了コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない
開始日:
2014-07-15
期日:
進捗率:
0%
予定工数:
説明
改行¶
コーディング規約チェックのルールで制御構造のブロックで空白が考慮されていない.
data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace/ControlSignatureSniff.php
19 protected function getPatterns() 20 { 21 return array( 22 'tryEOL{EOL...}EOLcatch (...)EOL{EOL', 23 'doEOL{EOL...}EOLwhile (...);EOL', 24 'while (...)EOL{EOL', 25 'for (...)EOL{EOL', 26 'if (...)EOL{EOL', 27 'foreach (...)EOL{EOL', 28 '}EOLelseif (...)EOL{EOL', 29 '}EOLelseEOL{EOL', 30 ); 31 }
修正案¶
下記のように修正.
diff --git a/data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace/ControlSignatureSniff.php b/data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace/ControlSignatureSniff.php index d89fd62..b05ca6a 100644 --- a/data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace/ControlSignatureSniff.php +++ b/data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace/ControlSignatureSniff.php @@ -19,14 +19,14 @@ class OpenPNE3_Sniffs_WhiteSpace_ControlSignatureSniff extends PHP_CodeSniffer_S protected function getPatterns() { return array( - 'tryEOL{EOL...}EOLcatch (...)EOL{EOL', - 'doEOL{EOL...}EOLwhile (...);EOL', - 'while (...)EOL{EOL', - 'for (...)EOL{EOL', - 'if (...)EOL{EOL', - 'foreach (...)EOL{EOL', - '}EOLelseif (...)EOL{EOL', - '}EOLelseEOL{EOL', + 'tryEOL *{EOL...}EOL *catch (...)EOL *{EOL', + 'doEOL *{EOL...}EOL *while (...);EOL', + 'while (...)EOL *{EOL', + 'for (...)EOL *{EOL', + 'if (...)EOL *{EOL', + 'foreach (...)EOL *{EOL', + '}EOL *elseif (...)EOL *{EOL', + '}EOL *elseEOL *{EOL', ); } }
isao sano さんが10年以上前に更新
- コピー元 Bug(バグ) #2503: コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない を追加
誠二 天重 さんが10年以上前に更新
どういう理由かは不明ですが、stable-3.6.x ブランチに data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace/ControlSignatureSniff.php というファイルが存在していません( https://github.com/openpne/OpenPNE3/tree/stable-3.6.x/data/CodingStandard/OpenPNE3/Sniffs/WhiteSpace )。
ちなみに、3.4系、3.8系およびmasterには同ファイルは存在しています。
調べてみたところ、 release-3.6beta10 から release-3.6beta11 の間になぜかしらファイルがなくなっており、それ以降3.6系には含まれていないようです。
チケットをどのように扱って良いのか不明のため、一旦保留とします。
誠二 天重 さんが10年以上前に更新
- 関連している Enhancement(機能追加・改善) #1744: コーディング規約違反のチェックが継続的におこなえるように体制を整える を追加
操作