Merge lp:~hanane-s/prestashoperpconnect/prestashoperpconnect-map-country-in-addresses into lp:prestashoperpconnect

[hanane]

we map the country in addresses when we get the new addresses from prestashop

[Sébastien BEAU - http://www.akretion.com (sebastien.beau)]

Hi hanane, thanks for your work
I did not see you merge before and I had propose similary to your work here
https://code.launchpad.net/~sebastien.beau/prestashoperpconnect/prestashoperpconnect-fix/+merge/179949

But I think the good solution is to add a generic powerfull binding and use a direct mapping.
I will talk about it tomorrow with Guewen

Have a good day

[Sébastien BEAU - http://www.akretion.com (sebastien.beau)]

Hi hanane I merge a fix, I do not use the method get_openerp_id as I think this method should be remove (using unwrap in binding is better)
The merge done is here :
https://code.launchpad.net/~sebastien.beau/prestashoperpconnect/prestashoperpconnect-fix/+merge/179949

Thanks for your work

Unmerged revisions

243. By hanane on 2013-07-29

[IMP] mapping country with customer addresses

=== modified file 'prestashoperpconnect/unit/mapper.py'
--- prestashoperpconnect/unit/mapper.py	2013-07-17 07:03:37 +0000
+++ prestashoperpconnect/unit/mapper.py	2013-07-29 09:12:32 +0000
@@ -231,6 +231,15 @@
     @mapping
     def customer(self, record):
         return {'customer': True}
+    
+    @mapping
+    def country_id(self, record):
+        if record.get('id_country'):
+            return {'country_id': self.get_openerp_id(
+                'prestashop.res.country',
+                record['id_country']
+            )}
+        return {}
 
 
 @prestashop