logo
Welcome to our new AbleCommerce forums. As a guest, you may view the information here. To post to this forum, you must have a registered account with us, either as a new user evaluating AbleCommerce or an existing user of the application. For all questions related to the older version of Gold and earlier, please go to forums.ablecommerce.com. Please use your AbleCommerce username and password to Login. Forums Registration.

Notification

Icon
Error

Options
Go to last post Go to first unread
jstrzempa16590316  
#1 Posted : Friday, October 30, 2020 10:04:34 AM(UTC)
jstrzempa16590316

Rank: Newbie

Groups: Authorized User, Developers, Registered
Joined: 1/31/2019(UTC)
Posts: 4

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

Wanna join the discussion?! Login to your AbleCommerce Forums forum account. Forums Registration.

mazhar  
#2 Posted : Thursday, November 5, 2020 8:42:30 AM(UTC)
mazhar

Rank: Administration

Groups: Admin, Administrators, HelpDesk, System, Authorized User, Developers, Registered
Joined: 10/5/2018(UTC)
Posts: 169

Thanks: 7 times
Was thanked: 13 time(s) in 12 post(s)
Hi,

Thanks for the feedback, Yes having more event coverage will certainly help with plugin development. We are hoping to gradually improve the plugin framework and bring more events into the application.

Regards
jstrzempa16590316  
#3 Posted : Friday, December 4, 2020 10:39:42 AM(UTC)
jstrzempa16590316

Rank: Newbie

Groups: Authorized User, Developers, Registered
Joined: 1/31/2019(UTC)
Posts: 4

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
mazhar  
#4 Posted : Tuesday, April 6, 2021 4:50:11 AM(UTC)
mazhar

Rank: Administration

Groups: Admin, Administrators, HelpDesk, System, Authorized User, Developers, Registered
Joined: 10/5/2018(UTC)
Posts: 169

Thanks: 7 times
Was thanked: 13 time(s) in 12 post(s)
We added four new events in 9.0.4, those are Login, Logout, CheckingOut and CheckedOut.
Users browsing this topic
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.