Ticket #1841 (closed 改善提案: 修正済)

Opened 12 years ago

Last modified 12 years ago

コーディング規約への準拠(PHPコードの終了タグ)

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.12.1
Component: その他 Version: 2.12.0
Keywords: Cc:
修正済み: yes

Description

開発コミュニティより

【EC-CUBE 2.12】提供ファイルの(細かいですが・・・)誤りにつきまして
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=10670&forum=1

PHPコードの終了タグ(?>)の扱いが統一されていないとの報告。

Change History

comment:1 follow-up: ↓ 2 Changed 12 years ago by shutta

  • Owner changed from somebody to shutta
  • 修正済み set
  • Status changed from new to assigned

r21420(#1613)あたりで、コーディング規約への準拠を高めるために終了タグの除去がなされていたが、一部漏れているファイルがあった。

漏れていた分を対応してコミット。(r21881)

comment:2 in reply to: ↑ 1 Changed 12 years ago by Seasoft

mtb_constants_init.php は自動生成されるキャッシュファイル mtb_constants.php から保守しているので、mtb_constants.php の生成元も直しておく必要がある予感が。

comment:3 follow-up: ↓ 4 Changed 12 years ago by shutta

私の理解が間違っているかもしれませんが、
/data/cache/mtb_constants.php は、存在していなければ、 /data/mtb_constants_init.php からコピーされて生成され、 マスターデータ管理でDBの更新と同時に /data/cache/mtb_constants.php も更新される挙動だと理解しています。
なので、 /data/cache/mtb_constants.php の生成元 = /data/mtb_constants_init.php なので問題ないような気がしますが、如何でしょうか?

comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 12 years ago by Seasoft

実装を未確認で恐縮ですが、/data/cache/mtb_constants.php の生成ロジック (コピーでなくて、DB からの生成の方ね) を修正済みという事でしたら、問題ないと思います。

comment:5 in reply to: ↑ 4 Changed 12 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済

Seasoft さまへの返信

実装を未確認で恐縮ですが、/data/cache/mtb_constants.php の生成ロジック (コピーでなくて、DB からの生成の方ね) を修正済みという事でしたら、問題ないと思います。

r21946 にて対応させて頂きました。

閉じさせて頂きます。あありがとうございました。

Note: See TracTickets for help on using tickets.