バグ報告 > その他 > 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして |
その他
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
MPV |
投稿日時: 2012/6/2 10:49
対応状況: −−−
|
仙人 登録日: 2011/3/27 居住地: 投稿: 309 |
【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして 稼働環境
[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も同様でした。 |
tanaca |
投稿日時: 2012/6/2 10:57
対応状況: −−−
|
仙人 登録日: 2010/11/28 居住地: 福岡県 投稿: 332 |
Re: 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして MPV様
私の勘違いでしたら恐縮ですが、phpの最後の”?>”はてっきり不要なものだと思っていました。 ”?>”で閉じていた方にとってはしっくりこないと思いますが http://itpro.nikkeibp.co.jp/article/COLUMN/20070307/264114/
|
MPV |
投稿日時: 2012/6/2 11:08
対応状況: −−−
|
仙人 登録日: 2011/3/27 居住地: 投稿: 309 |
Re: 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして tanaca様
早々のご返信ありがとうございました。 お教え頂きました内容を拝見いたしました。 これだと確かに不要ですね。 パッと見たところ、新しい方は 無しで揃えてあるのですね。 当方、勉強不足で大変恐縮です。 そうすると、data/mtb_constants_init.phpは むしろ無しの方がいいのでしょうか? 当方も、今回を機会に揃えようと思います。 |
tanaca |
投稿日時: 2012/6/2 11:14
対応状況: −−−
|
仙人 登録日: 2010/11/28 居住地: 福岡県 投稿: 332 |
Re: 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして たしかに統一されていないとスッキリしないですよね。
>今後は統一されるのでしょうか? 株式会社ロックオンさんやコミッターさんの意見次第だと思いますが、いずれにしても統一してほしいですね。
|
MPV |
投稿日時: 2012/6/2 11:33
対応状況: 解決済
|
仙人 登録日: 2011/3/27 居住地: 投稿: 309 |
Re: 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして tanaca様
ご丁寧に回答頂きましてありがとうございます。 統一の点についてはまだ残りそうですが、 疑問点は解決しましたので、スレッドは 解決済みにします。 重ねまして、ご回答ありがとうございました。 |
shutta |
投稿日時: 2012/6/2 11:58
対応状況: −−−
|
仙人 登録日: 2010/2/4 居住地: 関西 投稿: 835 |
Re: 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして 統一の点に関しては、
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 以上、ご報告までに。
|
MPV |
投稿日時: 2012/6/2 12:50
対応状況: −−−
|
仙人 登録日: 2011/3/27 居住地: 投稿: 309 |
Re: 【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして shutta様
ご丁寧なコメント並びにご対応頂きまして ありがとうございました。<(_ _)> 削除して取り込みします。 |
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |