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

EC-CUBEイベント実装漏れ #3784

Closed
sw-takaya-murata-zz opened this issue Sep 7, 2018 · 4 comments
Closed

EC-CUBEイベント実装漏れ #3784

sw-takaya-murata-zz opened this issue Sep 7, 2018 · 4 comments
Milestone

Comments

@sw-takaya-murata-zz
Copy link

概要(Overview)

コントローラーにイベントが実装されていない。

実装されていないイベント(一部)
FRONT_SHOPPING_INDEX_INITIALIZE
FRONT_SHOPPING_CONFIRM_INITIALIZE
FRONT_SHOPPING_CONFIRM_COMPLETE

期待する内容(Expect) or 要望 (Requirement)

リリース前にコントローラーへのイベントの実装する。

環境 (environment)

  • EC-CUBE: 4.0

関連情報 (Ref)

#3700

@chihiro-adachi
Copy link
Contributor

フォームへの項目追加であれば、FormExtensionで拡張していただくのが推奨されます。
もしフックポイントがないことで実装できないユースケースがあれば教えていただければありがたいです。

@sw-takaya-murata-zz
Copy link
Author

@chihiro-adachi
まず廃止なのかバグなのか教えていただけないですか?

3系と実装方法を変える場合、サポートや開発等の見通しが変わります。

@chihiro-adachi
Copy link
Contributor

@sw-takaya-murata
すみません、このあたり整理が追いついていないですが、基本的にはフックポイントは非推奨の扱いです。フォーム項目の追加であれば、FormExtensionでの拡張をお願いします。

現在定義されているものを削除するということはありませんが、単純なフォーム拡張のためにフックポイントを定義する、ということは無いかと思います。

@okazy okazy added this to the 4.0 milestone Sep 10, 2018
@okazy
Copy link
Contributor

okazy commented Sep 12, 2018

どうしてもフックポイントが必要なユースケースがあればreopenしてください。

@okazy okazy closed this as completed Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants