yomoro様
株式会社ロックオンの梶原でございます。
今回、リファクタリングガイドラインが結構多岐にわたっており、そんな中で作業いただいており、本当にありがとうございます。
というところで、少し時間がかかってしまいましたが、ソースの確認をさせていただきました。
以下の点がリファクタリングガイドラインから外れておりますので、修正いただけるとありがたいです。
(1)http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン#ビジネスロジック
- 宣言を除き, 引数や返り値が無く, すべて内部のメンバ変数で処理するような関数は極力作成しない
こちらは分かりにくいと思いますので、申し訳ございませんが、以下チケットで同様の指摘をさせていただいておりますので、ご参照いただければ幸いです。
#979
▼その他注意事項
(1)関数の引数にオブジェクトのインスタンスを渡している場合, 明示的 に参照渡しをしていないので, PHP4 環境で予期せぬ不具合が発生する可能性があります.
(2)LC_Page_FrontParts_Bloc_Category.php に, 下記のようなコードが見られますが, SC_Display::detectDevice() は boolean を返しません.
if (SC_Display::detectDevice() === true) {
以上、姑みたいなチェックで申し訳ございません。
今回、「とりあえず動作する」⇒「振る舞いが綺麗なソースにする」 という観点で進めておりますので、ご了承いただきたく思います。
お忙しい中、ご協力いただいており恐縮ですが、ご確認のほどよろしくお願い致します。
また、分からないことがございましたら、他コミッター様への情報共有も兼ね、本チケットに返信いただけますと幸いです。
それでは、どうぞよろしくお願い致します。