After all, this was just a workaround. Remove it.

The real fix was in OCA#171. I leave the tests.
yvaucher · Jan 18, 2016 · 398a1f5 · 398a1f5
1 parent a2c1be2
commit 398a1f5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/partner_contact_birthdate/models.py b/partner_contact_birthdate/models.py
@@ -40,8 +40,7 @@ class Partner(models.Model):
     @api.depends("birthdate_date")
     def _birthdate_compute(self):
         """Store a string of the new date in the old field."""
-        if self.exists():
-            self.birthdate = self.birthdate_date
+        self.birthdate = self.birthdate_date
 
     @api.one
     def _birthdate_inverse(self):

diff --git a/partner_firstname/__openerp__.py b/partner_firstname/__openerp__.py
@@ -21,7 +21,7 @@
 {
     'name': 'Partner first name and last name',
     'summary': "Split first name and last name for non company partners",
-    'version': '8.0.2.1.1',
+    'version': '8.0.2.1.0',
     'author': "Camptocamp, Grupo ESOC, Odoo Community Association (OCA)",
     "license": "AGPL-3",
     'maintainer': 'Camptocamp, Acsone',

diff --git a/partner_firstname/models.py b/partner_firstname/models.py
@@ -99,8 +99,7 @@ def _get_computed_name(self, lastname, firstname):
     @api.depends("firstname", "lastname")
     def _compute_name(self):
         """Write the 'name' field according to splitted data."""
-        if self.exists():
-            self.name = self._get_computed_name(self.lastname, self.firstname)
+        self.name = self._get_computed_name(self.lastname, self.firstname)
 
     @api.one
     def _inverse_name_after_cleaning_whitespace(self):