thaond / magento-openerp-smile-synchro

Automatically exported from code.google.com/p/magento-openerp-smile-synchro
0 stars 0 forks source link

CODE DESIGN: refactor+test sale order push feature #33

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The contributed code by Charles Galpin to provide sale order push to
Magento has not been tested enough and, moreover, it's not properly done.

Mostly, we don't want Magento to do himself all the work of pushing orders
back to OpenERP. Instead, what we want is Magento to CALL OpenERP to tell
him to grab the given (with id) sale order from Magento using the usual
sale order import method. This will maximize code reuse and avoid doing
complex things on the Magento side.

Consenquently, I believe the magento_utils.py should be removed totally
after refactoring. Of course the PHP part should take this into account.

Finally, the feature should deal about possible network/OpenERP errors.
Meaning that if Magento don't manage to have a sale order imported by
OpenERP then he should remember it somewhere for later re-import.

Original issue reported on code.google.com by rva...@gmail.com on 2 Mar 2009 at 12:49