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

Fix/reindex on create product outside admin scope #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ticaje
Copy link

@ticaje ticaje commented Jun 21, 2017

This fixes an issue i've come across regarding importing product's stock outside admin interface.
It turns out that when importing product's stock abroad from an interface other thatn admin back-office, a file based import process for instance, the method multiLocationInventoryProductSave is not being dispatched since it's tied to adminhtml node in the config file. 
So, this fix consists of creating a new event under global node and defining an argument to make sure inside attached method that it's only being called in this context. Then in the observer's method defined i perform the proper index table persistence, based on the product's stock related data.
An array must be defined which will be passed out trough request object, this is responsibility of the importer. Such an array must containg at least the same keys than the one obtained by:
Mage::app()->getRequest()->getPost('multilocationinventory');

Valid to say that this fix could include way much refactoring.

Hector Luis Barrientos Margolles added 2 commits June 21, 2017 17:43
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.

1 participant