miau's blog?

PHP_CodeSniffer バグ報告 その 2

また PHP_CodeSniffer のバグ見つけたので報告とかやってました。今日も 2 件。

PEAR :: Bug #14250 :: ArrayDeclarationSniff emit warnings at malformed array

「"=>" の前後には必ずスペースを入れる」ってのをルール化したくて、Squiz/Sniffs/Arrays/ArrayDeclarationSniff.php を取り込んで使ってたんだけど、配列の書き方が変だとエラーが出ちゃう。普通に使うぶんには問題ないんだけど、phpUnderControl 上で実行したときはこのエラーメッセージのせいで XML のパースに失敗して、レポートが出なくなってしまうから結構困る。

・・・だったんだけど、この ArrayDeclarationSniff.php はチェック厳しすぎたので、結局は半分くらいのチェックをコメントアウトして独自ルールとして取り込んでたり。そしたらエラーが出なくなったから、まあ一応報告しておきました、という程度。

PEAR :: Bug #14251 :: --extensions option doesn't work

どちらかというとこっちのほうが困る。CakePHP 使ってるからテンプレートファイル名が .ctp なんだけど、--extensions オプションが利かないから、サーバ上で一括チェックできるのはデフォルトの対応拡張子である .inc と .php だけという状態。

暫定対応としては CodeSniffer.php の $allowedFileExtensions に .ctp を追加してやればいいだけなんだけど、Standards 配下じゃないところに手を入れるのはちょっと抵抗あるなぁ。find コマンドかまして個別にファイル指定って手もあるけど・・・うーん。

・・・とまあ今日も本筋と関係ないところに結構時間とられてしまった。最近は裏方の仕事ばっかやってて自分の担当分が全然進んでない。この流れは早めに変えないと。


posted at 05:28:11 on 2008-06-28 by miau - Category: PHP No Trackbacks - Permalink

TrackBack

このエントリにトラックバックはありません
現在トラックバックは受け付けていません。

Comments

No comments yet

Add Comments

現在コメントは受け付けていません。
お手数ですが、 こちら のコメント欄にでも記載していただければと思います。