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

ProductClassControllerクラスへのフックポイント実装 #3700

Closed
naoki-shimo opened this issue Aug 29, 2018 · 6 comments

Comments

Projects
None yet
4 participants
@naoki-shimo
Copy link

commented Aug 29, 2018

概要(Overview)

商品登録(規格設定)画面の処理に対してフックポイントが全く設置されていない

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

EccubeEventsクラスには定義のみされている状態なので、これらを実装してもらいたい

環境 (environment)

  • EC-CUBE: 4.0beta

@ryo-endo ryo-endo added this to the 4.0 milestone Aug 30, 2018

@ryo-endo

This comment has been minimized.

Copy link
Contributor

commented Aug 30, 2018

@naoki-shimo ありがとうございます。検討させていただきます。

@naoki-shimo

This comment has been minimized.

Copy link
Author

commented Aug 30, 2018

ご確認頂きありがとうございます。

現在プラグインを作成する中で規格ごとに項目を追加したいのですが
Entityの拡張では対応できない項目ですので少なくとも
ADMIN_PRODUCT_PRODUCT_CLASS_EDIT_COMPLETE
のフックポイントがないと処理できない状況です。

ProductClassContorollerクラスに

use Eccube\Event\EccubeEvents;
use Eccube\Event\EventArgs;

を追加した上で130行目付近と160行目付近の
saveProductClassesの処理の後に

            $event = new EventArgs(
                [
                    'form' => $form,
                    'Product' => $Product,
                ],
                $request
            );
            $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_PRODUCT_CLASS_EDIT_COMPLETE, $event);

を追加して頂く事で使用出来る事は確認しております。
(もしくはsaveProductClassesの中に設置してもよいかもしれません)

すいませんが、リリースまでにご対応頂く事をご検討いただけないでしょうか。

@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Sep 6, 2018

Entityの拡張では対応できない項目ですので

こちら詳細お伺いさせていただくことはできますか?
具体的なコードでこれができなかった、とか教えていただければありがたいです。

@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Sep 6, 2018

@naoki-shimo

This comment has been minimized.

Copy link
Author

commented Sep 7, 2018

ご確認頂きありがとうございます。
別の方法で対応出来そうなので上記フックポイントについては実装頂かなくても大丈夫そうです。
お手数をおかけしました。

@okazy

This comment has been minimized.

Copy link
Contributor

commented Sep 12, 2018

解決済みのためクローズします。

@okazy okazy closed this Sep 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.