- Issue created by @mxh
- Assigned to jurgenhaas
- Merge request !407Issue #3380771 by mxh, jurgenhaas: Cleanup usage of Event classes → (Merged) created by jurgenhaas
- Issue was unassigned.
- Status changed to Needs review
9 months ago 5:08pm 22 February 2024 - 🇩🇪Germany jurgenhaas Gottmadingen
This is now all cleaned up.
Not sure about this one:
Careful on early invokations (such as listener methods and processor's execution method), where only object arguments are being used.
Tests seem to be OK.
- Status changed to RTBC
9 months ago 2:06pm 23 February 2024 - 🇩🇪Germany danielspeicher Steisslingen
That is a lot but straightforward. The code is more type safe. I searched for tests and the specific events are well covered.
-
jurgenhaas →
committed 708168c2 on 2.0.x
Issue #3380771 by jurgenhaas, mxh: Cleanup usage of Event classes
-
jurgenhaas →
committed 708168c2 on 2.0.x
- Status changed to Fixed
9 months ago 2:25pm 23 February 2024 - 🇩🇪Germany jurgenhaas Gottmadingen
Thanks for reviewing all this. Yes, it's a lot of extra code that we could now get rid of, fortunately ;-)
Automatically closed - issue fixed for 2 weeks with no activity.