Merge lp:~camptocamp/openerp-connector-magento/7.0-no-updated_at-1301417 into lp:~openerp-connector-core-editors/openerp-connector-magento/7.0

Proposed by Guewen Baconnier @ Camptocamp
Status: Merged
Approved by: Guewen Baconnier @ Camptocamp
Approved revision: 965
Merge reported by: Guewen Baconnier @ Camptocamp
Merged at revision: not available
Proposed branch: lp:~camptocamp/openerp-connector-magento/7.0-no-updated_at-1301417
Merge into: lp:~openerp-connector-core-editors/openerp-connector-magento/7.0
Diff against target: 14 lines (+3/-1)
1 file modified
magentoerpconnect/unit/export_synchronizer.py (+3/-1)
To merge this branch: bzr merge lp:~camptocamp/openerp-connector-magento/7.0-no-updated_at-1301417
Reviewer Review Type Date Requested Status
Leonardo Pistone (community) code review Approve
OpenERP Connector Core Editors Pending
Review via email: mp+213860@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Leonardo Pistone (lepistone) :
review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'magentoerpconnect/unit/export_synchronizer.py'
--- magentoerpconnect/unit/export_synchronizer.py 2013-11-12 08:43:40 +0000
+++ magentoerpconnect/unit/export_synchronizer.py 2014-04-02 14:33:10 +0000
@@ -83,7 +83,9 @@
83 return True83 return True
84 record = self.backend_adapter.read(self.magento_id,84 record = self.backend_adapter.read(self.magento_id,
85 attributes=['updated_at'])85 attributes=['updated_at'])
8686 if not record['updated_at']:
87 # in rare case it can be empty, in doubt, import it
88 return False
87 fmt = DEFAULT_SERVER_DATETIME_FORMAT89 fmt = DEFAULT_SERVER_DATETIME_FORMAT
88 sync_date = datetime.strptime(sync, fmt)90 sync_date = datetime.strptime(sync, fmt)
89 magento_date = datetime.strptime(record['updated_at'], fmt)91 magento_date = datetime.strptime(record['updated_at'], fmt)