稼働環境

[EC-CUBE]　2.11.5
[レンタルサーバ]　WADAX
[OS]　Red Hat Linux
[PHP]　PHP 5.1.6
[データベース]　MySQL 5.0.95
[WEBサーバ]　Apache
[ブラウザ]　Firefox 12.0　Google Chrome　19.0.1084.52　IE9

お世話になります。

当方、2.12を取り込むための準備作業をしているのですが
diffを取得した際に気づきましたので報告します。

eccube-2.11.5_2.12.0-update.zip　内の
data/app_initial.php

phpの最後の”?>”が抜けておりました。

大変細かい話で恐縮ですが、
ご対応をよろしくお願いいたします。<(_ _)>

追伸）その後確認したら、eccube-2.12.0.zipも同様でした。

MPV様

私の勘違いでしたら恐縮ですが、phpの最後の”?>”はてっきり不要なものだと思っていました。

”?>”で閉じていた方にとってはしっくりこないと思いますが

http://itpro.nikkeibp.co.jp/article/COLUMN/20070307/264114/

----------------
他社システムからEC-CUBEへの移行などお気軽にご相談ください。
福岡のホームページ制作会社 ECカンパニー
担当：田中利宏

tanaca様

早々のご返信ありがとうございました。

お教え頂きました内容を拝見いたしました。
これだと確かに不要ですね。

パッと見たところ、新しい方は
無しで揃えてあるのですね。
当方、勉強不足で大変恐縮です。

そうすると、data/mtb_constants_init.phpは
むしろ無しの方がいいのでしょうか？

当方も、今回を機会に揃えようと思います。

たしかに統一されていないとスッキリしないですよね。

＞今後は統一されるのでしょうか？
株式会社ロックオンさんやコミッターさんの意見次第だと思いますが、いずれにしても統一してほしいですね。

----------------
他社システムからEC-CUBEへの移行などお気軽にご相談ください。
福岡のホームページ制作会社 ECカンパニー
担当：田中利宏

tanaca様

ご丁寧に回答頂きましてありがとうございます。

統一の点についてはまだ残りそうですが、
疑問点は解決しましたので、スレッドは
解決済みにします。

重ねまして、ご回答ありがとうございました。

統一の点に関しては、

EC-CUBE標準規約
http://svn.ec-cube.net/open_trac/wiki/EC-CUBE標準規約

によって定められて(Zend Framework PHP 標準コーディング規約に準拠)いますので、
終了タグ(?>)に関しては、「PHPコードのみからなるファイル」に対しては除去しないといけません。

2.12.0開発の際に、

チェンジセット 21420
http://svn.ec-cube.net/open_trac/changeset/21420

のあたりで除去されていたようですが、一部漏れていたようです。

下記、チケットを発行し、漏れていた分をコミットしておきました。

チケット#1841 コーディング規約への準拠(PHPコードの終了タグ)
http://svn.ec-cube.net/open_trac/ticket/1841

チェンジセット 21881
http://svn.ec-cube.net/open_trac/changeset/21881

以上、ご報告までに。

----------------

リゾート会員権(エクシブ等)の売買・仲介

shutta様

ご丁寧なコメント並びにご対応頂きまして
ありがとうございました。<(_ _)>

削除して取り込みします。