Fixed the hook listener for the hook actionProductAttributeDelete & Removed hooks actionProductOutOfStock & registerGDPRConsent as hook listeners are not defined

[Progi1984]

Questions	Answers
Description?	Fixed the hook listener for the hook actionProductAttributeDelete Removed hooks actionProductOutOfStock & registerGDPRConsent as hook listeners are not defined
Type?	bug fix
BC breaks?	no
Deprecations?	no
Fixed ticket?	Relative to PrestaShop/PrestaShop#19230
How to test?	Install this branch ps_emailalerts is not linked to hooks actionProductOutOfStock & registerGDPRConsent In upgrade, hooks actionProductOutOfStock & registerGDPRConsent are unlinked. When the hook actionProductAttributeDelete is triggered, the line is now correctly removed in the table mailalert_customer_oos

[jolelievre]

Thanks @Progi1984

[florine2623]

Hello @Progi1984 ,

Thanks for the PR !

Hooks actionProductOutOfStock & registerGDPRConsent are well removed ✅

Regarding the hook actionProductAttributeDelete, I don't have the same result as what you mentioned in the How to test section.

Here are the steps I followed :

• In a combination product, add a combination A with 0 stock
• Go to product page, add your email address in the email alert section
• See a new line in table ps_mailalert_customer_oos
• Go to product sheet and delete the combination A
• Refresh your database page, line is still present in ps_mailalert_customer_oos

Could you confirm this is the wanted behavior or not ?
Thanks!

[Progi1984]

@florine2623 This hook actionProductAttributeDelete is executed only when a product is removed.

[florine2623]

@Progi1984,
Thanks for the details.

Indeed, now when I delete my whole product and not just an attribute of a product, the line is deleted in the database ✅ the name of the hook is a little confusing ^^

Before I delete my product :

After I delete my product :

Thanks it is QA ✅

[Progi1984]

Thanks @florine2623