プロジェクト

全般

プロフィール

Backport(バックポート) #3642

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

誠二 天重ほぼ10年前に追加. 7年以上前に更新.

ステータス:
Invalid(無効)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-10-18
期日:
進捗率:

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 - Bug(バグ) #2503: コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない Invalid(無効) 2011-10-18

履歴

#1 誠二 天重ほぼ10年前に更新

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

#2 isao sano9年以上前に更新

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

#3 kaoru n7年以上前に更新

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

#4 Youichi Kimura7年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

下記 Pull Request を作成しました
https://github.com/openpne/OpenPNE3/pull/346

#5 kaoru n7年以上前に更新

  • 担当者Youichi Kimura にセット

#6 Youichi Kimura7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#2503-12 と同様に理由で差し戻します。

#7 kaoru n7年以上前に更新

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

#8 kaoru n7年以上前に更新

  • ステータスRejected(差し戻し) から Invalid(無効) に変更
  • 進捗率50 から 0 に変更

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

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