linawolf / form_double_opt_in

TYPO3 Extension adding Double Opt-In to the TYPO3 CMS Form Framework
12 stars 18 forks source link

Avoid deprecated SignalSlot Dispatcher and use PSR-14 Events instead #6

Closed simonschaufi closed 3 years ago

simonschaufi commented 3 years ago

See:

danielkestler commented 3 years ago

Closed by https://github.com/medienreaktor/form_double_opt_in/commit/a667843e9146cc945a8f395f1166ea54b506e9ed

simonschaufi commented 3 years ago

Thank you for the quick implementation. You can actually leave out the injectEventDispatcher method and eventDispatcher property as they exist in the ActionController already.

danielkestler commented 3 years ago

Ah, thanks, didn't know that. But that is only true for the ActionController, right? I need the injection in the form finisher?

simonschaufi commented 3 years ago

Yes, only the controller. If you use phpstorm you should see that the method and prop is overwritten