Searching for allowed carriers incorrectly uses magento_picking_id instead of magento_order_id
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenERP Connector - Magento |
Fix Released
|
High
|
Guewen Baconnier @ Camptocamp |
Bug Description
Lines 83-92 in stock_tracking.py:
magento_picking_id = picking.magento_id
if magento_picking_id is None:
raise NoExternalId("No value found for the picking ID on "
self._validate(
self._check_
tracking_args = self._get_
self.backend_
The call to _check_
Related branches
- Allison (community): Approve
- Brendan Clune (Logic Supply) (community): Approve
- OpenERP Connector Core Editors: Pending requested
-
Diff: 24 lines (+7/-1)1 file modifiedmagentoerpconnect/stock_tracking.py (+7/-1)
description: | updated |
description: | updated |
Changed in openerp-connector: | |
status: | New → Confirmed |
importance: | Undecided → High |
information type: | Embargoed → Public |
affects: | openerp-connector → openerp-connector-magento |
Changed in openerp-connector-magento: | |
status: | Confirmed → Fix Committed |
For clarification, here is the Magento API call:
public function getCarriers( $orderIncrement Id) 'sales/ order') ->loadByIncreme ntId($orderIncr ementId) ;
{
$order = Mage::getModel(
...