baserproject / basercms

baserCMS : Based Website Development Project
http://basercms.net
Other
173 stars 131 forks source link

afterSendEmailのデータをエンティティに変更 #3992

Closed seto1 closed 3 weeks ago

seto1 commented 3 weeks ago

beforeSendEmailだとエンティティがイベントで渡されるのですが、afterSendEmailだとリクエストデータなのでafterSendEmailの方もエンティティを渡すように変更しました。

エンティティのほうがidが取得できるので嬉しいです。 もしリクエストデータが必要な際も、いざとなったらリクエストデータはEventListenerからでも取得できるので困らないと思います。

ご確認お願いします。

ryuring commented 3 weeks ago

@seto1 この仕様だと仕様変更になってしますので、両方定義した方がいいですね

seto1 commented 3 weeks ago

@ryuring 両方定義というと、beforeSendEmailとafterSendEmail、どちらも以下のように変更するイメージでしょうか?

[
    'data' => $this->request->getData(),
    'entity' => $entity,
]
ryuring commented 3 weeks ago

@seto1 そうですね。その方が後方互換的に安全かと。