← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 670856] Re: Delivery carrier mapping correction

 

Hello,

It seems that we have a misunderstanding, maybe I miss something, so I'm
sorry about that.

But if I put the shipping_method as carrier name, it won't be the same code as the carrier_code.
Delivery carriers eg.
Carrier 1:
  name : laposteChronopost_Chrono13
  magento_code : laposteChronopost
Carrier 2:
  name : laposteChronopost_Chrono10
  magento_code : laposteChronopost
Carrier 3:
  name : laposteChronopost_ChronoExpressInternational
  magento_code : laposteChronopost
etc.

Otherwise I proposed something like that that I prefer for a better display and comprehension :
Carrier 1
  name : Chronopost 13
  magento_shipping_method : laposteChronopost_Chrono13
  magento_code : laposteChronopost
Carrier 2:
  name : Chronopost 10
  magento_shipping_method : laposteChronopost_Chrono10
  magento_code : laposteChronopost
Carrier 3:
  name : Chronopost Express International
  magento_shipping_method : laposteChronopost_ChronoExpressInternational
  magento_code : laposteChronopost

When you import an order from magento, you don't have the "carrier_code" information but the "shipping_method" (like laposteChronopost_Chrono13), so you have to search the carrier from the magento_shipping_method and not from the magento_code.
The following mapping
  carrier_ids = self.pool.get('delivery.carrier').search(cr, uid, [('magento_code', '=', ifield)])
won't work because ifield=='laposteChronopost_Chrono13' and magento_code=='laposteChronopost'

In fact :
The magento_code is used to add a tracking.
The magento_shipping_method is used to retrieve the good carrier from the shipping method when we import magento orders.

Thanks
Regards,
Guewen

-- 
Delivery carrier mapping correction
https://bugs.launchpad.net/bugs/670856
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.

Status in Magento Open ERP Connector: Confirmed

Bug description:
Hello,

I just tested the delivery carrier mapping from Magento to OpenERP with the revision http://bazaar.launchpad.net/~openlabs-akretion-consortium/magentoerpconnect/magentoerpconnect/revision/346

The condition to map the carrier is inverted, it will never be updated in OpenERP ;-)

Here is a patch with the correction.

Thanks
Guewen





References