Project

General

Profile

Actions

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

open

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

Added by kaoru n over 8 years ago. Updated almost 5 years ago.

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

50%

Estimated time:

Description

Overview (概要)

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

Spec (仕様)

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


Related issues 2 (0 open2 closed)

Related to OpenPNE 3 - Bug(バグ) #2503: コーディング規約チェックのルールにおいて制御構造のブロックで空白が考慮されていないInvalid(無効)Youichi Kimura2011-10-18

Actions
Related to OpenPNE 3 - Backport(バックポート) #4037: PHP_CodeSniffer 最新版で動作するようにするWon't fix(対応せず)isao sano2016-11-14

Actions
Actions #1

Updated by kaoru n over 8 years ago

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

Updated by Youichi Kimura over 8 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

Actions #3

Updated by Youichi Kimura over 8 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
Actions #4

Updated by isao sano about 8 years ago

Actions #5

Updated by kaoru n almost 8 years ago

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

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

Actions #6

Updated by kaoru n almost 5 years ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x
Actions

Also available in: Atom PDF