プロジェクト

全般

プロフィール

Backport(バックポート) #3666

コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない

isao sano9年以上前に追加. 7年以上前に更新.

ステータス:
Invalid(無効)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
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',
     );
   }
 }

関連するチケット

関連している OpenPNE 3 - Enhancement(機能追加・改善) #1744: コーディング規約違反のチェックが継続的におこなえるように体制を整える Fixed(完了) 2010-10-27 2011-06-24
コピー元 OpenPNE 3 - Bug(バグ) #2503: コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない Invalid(無効) 2011-10-18

履歴

#1 isao sano9年以上前に更新

  • コピー元 Bug(バグ) #2503: コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない を追加

#2 誠二 天重9年以上前に更新

どういう理由かは不明ですが、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系には含まれていないようです。
チケットをどのように扱って良いのか不明のため、一旦保留とします。

#3 誠二 天重9年以上前に更新

#4 誠二 天重9年以上前に更新

対象ファイルが削除されているのは #1744 のチケットにおいて(コミットハッシュ:6ca5dcd853b3594685a5274bf7d526d48bc9eaf8)ですが、対象ファイルが削除された理由は不明。

#5 Shinichi Urabe9年以上前に更新

  • 対象バージョンOpenPNE 3.6.16 から OpenPNE 3.6.x に変更

3.6 と 3.8 でコーディング規約が変わっていないため、どちらかのソースに合わせた方がよいと考える

master and stable-3.8.x に #1744 の修正を取り込むか、 #1744 の修正の妥当性を確認した上で、問題があれば、取消すか のいずれか

今回の OpenPNE 3.6.16 のリリースに含むには検証時間が足りないため、3.6.x ににバージョンを一旦変更

#6 kaoru n7年以上前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.24 に変更

#7 kaoru n7年以上前に更新

  • 担当者Youichi Kimura にセット

#8 kaoru n7年以上前に更新

  • 対象バージョンOpenPNE 3.6.24 から OpenPNE 3.6.x に変更

#9 kaoru n7年以上前に更新

  • ステータスNew(新規) から Invalid(無効) に変更

#2503-13 により無効とします

他の形式にエクスポート: Atom PDF