Ticket #529 (closed バグ指摘: 修正済)

Opened 12 years ago

Last modified 10 years ago

pdf 関連モジュールの配置誤り

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.11.0
Component: フロント Version: 2.4.1
Keywords: Cc:
修正済み:

Description

/data/pdf に配置しているが、/data/module/pdf に配置すべき。

Change History

comment:1 Changed 12 years ago by AMUAMU

EC-CUBE自体が提供するpdfファイルとかも/data/pdf/ディレクトリ直下に入っているので、ここらへんの扱い整理及びディレクトリの整理も必要かと思います。

/data/module/以下は、純粋に他者製造モジュール(主にPEARモジュール)に対してのrequire変更で構成されているという点で分かりやすいのでアップデート等も分かりやすいという利点があります。

comment:2 Changed 11 years ago by kajiwara

Seasoft様、AMUAMU様 本件ご報告とご提案ありがとうございます。

r18394、r18395にて一旦対応いたしました。
まずは、pdfモジュールを/data/module/pdf/以下に配置させていただきました。

その他、PDFのテンプレートファイルはそのまま /data/pdf/ に残しております。
現状使用されていないものも多数ありましたが、便利に使われている可能性があると思い、ファイル自体の削除は行っておりません。
また、PDFテンプレートも、Smartyのtemplete内にあっても良いかと思いましたが、今回はディレクトリの変更はなしとしております。 (やはりデザインテンプレートなどの関係上です。申し訳ございません。)

追々、カスタマイズしやすい、分かりやすいディレクトリに配置など検討したいと思います。

またご意見等ございましたら、よろしくお願いします。

comment:3 Changed 11 years ago by kajiwara

  • Milestone changed from EC-CUBE2.4.2 to EC-CUBE2.5.0

管理画面のテンプレートに関して、引き続き検討を行うため、一旦マイルストーンを2.5に変更させていただきたいと思います。

申し訳ございませんが、よろしくお願いいたします。

comment:4 Changed 10 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • Status changed from new to assigned

comment:5 Changed 10 years ago by Seasoft

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.0

PDF とロゴは、フロント配下でも良い気もして迷いましたが、一旦は「TEMPLATE_ADMIN_DIR . 'pdf/'」配下に移動しました。

comment:6 Changed 10 years ago by Seasoft

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.