Sorry if this has already been mentioned, but for plugins and custom code - you need to add a lot more events. Working a bit with other systems they all have hundreds of different events you can hook into so you don't have to change core code or override entire methods. It seems like you may have removed events instead of adding more. It's not always a bad thing to have to override, but it can be problematic. For example, if Plugin A wants to add a few lines of code right after Login, right now Plugin A has to override the entire method and now takes control of the method. If Plugin B comes along and also wants to add a few lines - it has to know Plugin A has control now and it most likely will not, so it will override and now take control - Plugin A functionality will now break.

Easy way to get around this is a LoggedIn/LoggedOut plugins can hook into. There are other ways too like adding a Param in your classes that take an interface collection that our Plugins can add to and then you call in your classes so they inject their code...etc but at least more events would be very helpful

Edited by user Friday, October 30, 2020 10:09:21 AM(UTC)  | Reason: Not specified

Excellent. A couple of areas I've run into - Login/Logout

Also CheckoutController

_CreditCardPaymentForm

if (checkOut) {

... We usually have multiple extensions that need to add code right here -- We need an event we can all hook into