Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

テンプレートのディレクトリ構造の整理 #128

Open
okazy opened this issue May 8, 2020 · 1 comment
Open

テンプレートのディレクトリ構造の整理 #128

okazy opened this issue May 8, 2020 · 1 comment

Comments

@okazy
Copy link
Contributor

okazy commented May 8, 2020

4系のプラグインにおいて、テンプレートのディレクトリ構造と不具合の状況について調査した。

インストール時にファイルが配置される。

app/template/default/Coupon4/coupon_shopping_item.twig
app/template/default/Coupon4/coupon_shopping_item_confirm.twig
app/template/default/Coupon4/mypage_history_coupon.twig

以下のようにスニペットとして利用される。

$event->addSnippet('@Coupon4/default/coupon_shopping_item.twig');

無効化時に削除される。
作成されたファイルを編集してもフロントには反映されず、以下のファイルが優先される。
app/Plugin/Coupon4/Resource/template/default/coupon_shopping_item.twig

管理画面のページ管理から編集時に作成される

app/template/default/Coupon4/Resource/template/default/shopping_coupon.twig

以下のように利用されている。

/**
 * @Template("@Coupon4/default/shopping_coupon.twig")
*/

管理画面で編集はできるがフロントには反映されない、以下のファイルが優先される。(★)
app/Plugin/Coupon4/Resource/template/default/shopping_coupon.twig

★の問題に対して、以下のプルリクをいただいている。
#126

/**
 * @Template("@Coupon4/default/shopping_coupon.twig")
*//**
 * @Template("Coupon4/Resource/template/default/shopping_coupon.twig")
*/

こちらの修正で以下のファイルが読み込まれるようになる。
app/template/Coupon4/Resource/template/default/shopping_coupon.twig

本当なら根本対応したいところであるが、稼働中のサイトへ影響があるので慎重に行う必要がある。

4.0.4では app/template/plugin/[Plugin Code] 配下に配置することで優先して読み込まれるようになる見込みである。
EC-CUBE/ec-cube#4527
なので以下に配置するのが適切かと思われる。

app/template/plugin/Coupon4/coupon_shopping_item.twig
app/template/plugin/Coupon4/coupon_shopping_item_confirm.twig
app/template/plugin/Coupon4/mypage_history_coupon.twig
app/template/plugin/Coupon4/shopping_coupon.twig

ファイルの配置を変える必要があるのでマイナーバージョンアップでは変更しにくい。
現在も不具合があるので、実際には影響は限定的だと思われる。

4.0.3では上記に配置しても読み込まれない問題は残る。

どのように対応すべきか判断が難しいので、ご意見をいただけると嬉しいです。

@okazy okazy added this to the 4.0 milestone May 8, 2020
@okazy
Copy link
Contributor Author

okazy commented May 8, 2020

不具合の修正はできるため、 #126 の修正でいく。
今後マイナーバージョンアップではディレクトリ構造に関する修正が難しくなる。
不適切なお作法の状態となってしまうが、不具合があるよりはよいと判断。

@okazy okazy modified the milestones: 4.0, 4.x May 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant