Project

General

Profile

Enhancement(機能追加・改善) #4004

PHP_CodeSniffer 最新版で動作するようにする

Added by kaoru n about 3 years ago. Updated over 2 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Target version:
Start date:
2016-09-02
Due date:
% Done:

50%


Description

Overview (概要)

現在 OpenPNE3 の data/CodingStandard/OpenPNE3 以下にあるファイルは PHP_CodeSniffer 1.2.x でないと動作しない。
PHP_CodeSniffer 最新版で動作するようにする

Spec (仕様)

PHP_CodeSniffer 最新版で動作するようにする


Related issues

Related to OpenPNE 3 - Bug(バグ) #2503: コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていない Invalid(無効) 2011-10-18
Related to OpenPNE 3 - Backport(バックポート) #4037: PHP_CodeSniffer 最新版で動作するようにする Won't fix(対応せず) 2016-11-14

History

#1 Updated by kaoru n about 3 years ago

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

#2 Updated by Youichi Kimura about 3 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to Youichi Kimura
  • % Done changed from 0 to 50

下記 Pull Request にて対応しました (#4002, #4004 に対する修正を含んでいます)
https://github.com/openpne/OpenPNE3/pull/369

#3 Updated by Youichi Kimura about 3 years ago

#4004-2 について、下記の手順でチェックが行えます:

$ curl -OL https://squizlabs.github.io/PHP_CodeSniffer/phpcs.phar
$ php phpcs.phar --version
PHP_CodeSniffer version 2.7.0 (stable) by Squiz (http://www.squiz.net)
$ php phpcs.phar --standard=data/CodingStandard/OpenPNE3/ --report=summary .

PHP CODE SNIFFER REPORT SUMMARY
----------------------------------------------------------------------
FILE                                                  ERRORS  WARNINGS
----------------------------------------------------------------------
....
----------------------------------------------------------------------
A TOTAL OF 1592 ERRORS AND 2393 WARNINGS WERE FOUND IN 486 FILES
----------------------------------------------------------------------
PHPCBF CAN FIX 290 OF THESE SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 29.88 secs; Memory: 82.5Mb

#4 Updated by isao sano almost 3 years ago

#5 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

Also available in: Atom PDF