Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added: dispatchEvent to deleteAction(). Only existed in massDeleteAction(). #9

Closed
wants to merge 1 commit into from

Conversation

Bogardo
Copy link

@Bogardo Bogardo commented Apr 2, 2012

Mage::dispatchEvent('catalog_controller_product_delete', array('product' => $product));
To the deleteAction().
Currently only exists in massDeleteAction()

  Mage::dispatchEvent('catalog_controller_product_delete', array('product' => $product));
To the deleteAction().
Currently only exists in massDeleteAction()
@magento-team
Copy link
Contributor

Thank you for the request.
'catalog_controller_product_delete' event was used for refreshing Tags related to products. This logic was replaced by adding Tag indexer and as a result, 'catalog_controller_product_delete' event is not needed anymore and was removed (see nearest GitHub update).
Actually Bogardo, you are right, this event should be used both for delete() and massDelete() actions.
If you need some event on delete product action, you can use 'catalog_product_delete_before' event, which is dispatched in Mage_Core_Model_Abstract::_beforeDelete(). In this case you can be sure, that you catch product delete event not depending on a way the product is deleted.

@WouterSteen WouterSteen mentioned this pull request Oct 3, 2014
@askwhyweb askwhyweb mentioned this pull request Dec 18, 2014
@sivajik34 sivajik34 mentioned this pull request Jan 2, 2015
This was referenced Jan 24, 2015
@makkoff makkoff mentioned this pull request May 30, 2023
5 tasks
@nabeelfocus nabeelfocus mentioned this pull request Jan 3, 2024
5 tasks
@FabXav FabXav mentioned this pull request Oct 11, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants