odoo-addon-pms 16.0.0.29.0__py3-none-any.whl → 16.0.1.1.0__py3-none-any.whl

odoo/addons/pms/models/pms_checkin_partner.py

@@ -145,51 +145,47 @@ class PmsCheckinPartner(models.Model):
         comodel_name="res.country",
         inverse=lambda r: r._inverse_partner_fields("nationality_id", "nationality_id"),
     )
-    residence_street = fields.Char(
-        string="Street",
-        help="Street of the guest's residence",
+    street = fields.Char(
+        help="Street of the guest",
         readonly=False,
         store=True,
-        compute="_compute_residence_street",
+        compute="_compute_street",
     )
-    residence_street2 = fields.Char(
-        string="Street2",
-        help="Second street of the guest's residence",
+    street2 = fields.Char(
+        help="Second street of the guest",
         readonly=False,
         store=True,
-        compute="_compute_residence_street2",
+        compute="_compute_street2",
     )
-    residence_zip = fields.Char(
-        string="Zip",
-        help="Zip of the guest's residence",
+    zip = fields.Char(
+        help="Zip of the guest",
         readonly=False,
         store=True,
-        compute="_compute_residence_zip",
+        compute="_compute_zip",
         change_default=True,
     )
-    residence_city = fields.Char(
-        string="City",
-        help="City of the guest's residence",
+    city = fields.Char(
+        help="City of the guest",
         readonly=False,
         store=True,
-        compute="_compute_residence_city",
+        compute="_compute_city",
     )
-    residence_country_id = fields.Many2one(
+    country_id = fields.Many2one(
         string="Country of residence",
-        help="Country of the guest's residence",
+        help="Country of the guest",
         readonly=False,
         store=True,
         index=True,
-        compute="_compute_residence_country_id",
+        compute="_compute_country_id",
         comodel_name="res.country",
     )
-    residence_state_id = fields.Many2one(
+    state_id = fields.Many2one(
         string="State of residence",
-        help="State of the guest's residence",
+        help="State of the guest",
         readonly=False,
         store=True,
         index=True,
-        compute="_compute_residence_state_id",
+        compute="_compute_state_id",
         comodel_name="res.country.state",
     )
 
@@ -272,7 +268,6 @@ class PmsCheckinPartner(models.Model):
         string="Possible existing customer",
         compute="_compute_possible_existing_customer_ids",
         comodel_name="res.partner",
-        inverse_name="checkin_partner_possible_customer_id",
     )
 
     partner_relationship = fields.Char(help="Family relationship between travelers")
@@ -376,53 +371,52 @@ class PmsCheckinPartner(models.Model):
                 record.nationality_id = False
 
     @api.depends("partner_id")
-    def _compute_residence_street(self):
+    def _compute_street(self):
         for record in self:
-            if not record.residence_street and record.partner_id.residence_street:
-                record.residence_street = record.partner_id.residence_street
-            elif not record.residence_street:
-                record.residence_street = False
+            if not record.street and record.partner_id.street:
+                record.street = record.partner_id.street
+            elif not record.street:
+                record.street = False
 
     @api.depends("partner_id")
-    def _compute_residence_street2(self):
+    def _compute_street2(self):
         for record in self:
-            if not record.residence_street2 and record.partner_id.residence_street2:
-                record.residence_street2 = record.partner_id.residence_street2
-            elif not record.residence_street2:
-                record.residence_street2 = False
+            if not record.street2 and record.partner_id.street2:
+                record.street2 = record.partner_id.street2
+            elif not record.street2:
+                record.street2 = False
 
     @api.depends("partner_id")
-    def _compute_residence_zip(self):
+    def _compute_zip(self):
         for record in self:
-            if not record.residence_zip and record.partner_id.residence_zip:
-                record.residence_zip = record.partner_id.residence_zip
-            elif not record.residence_zip:
-                record.residence_zip = False
+            if not record.zip and record.partner_id.zip:
+                record.zip = record.partner_id.zip
+            elif not record.zip:
+                record.zip = False
 
     @api.depends("partner_id")
-    def _compute_residence_city(self):
+    def _compute_city(self):
         for record in self:
-            if not record.residence_city and record.partner_id.residence_city:
-                record.residence_city = record.partner_id.residence_city
-            elif not record.residence_city:
-                record.residence_city = False
+            if not record.city and record.partner_id.city:
+                record.city = record.partner_id.city
+            elif not record.city:
+                record.city = False
 
     @api.depends("partner_id", "nationality_id")
-    def _compute_residence_country_id(self):
+    def _compute_country_id(self):
         for record in self:
-            if (
-                not record.residence_country_id
-                and record.partner_id.residence_country_id
-            ):
-                record.residence_country_id = record.partner_id.residence_country_id
+            if not record.country_id and record.partner_id.country_id:
+                record.country_id = record.partner_id.country_id
+            elif not record.state_id:
+                record.country_id = False
 
     @api.depends("partner_id")
-    def _compute_residence_state_id(self):
+    def _compute_state_id(self):
         for record in self:
-            if not record.residence_state_id and record.partner_id.residence_state_id:
-                record.residence_state_id = record.partner_id.residence_state_id
-            elif not record.residence_state_id:
-                record.residence_state_id = False
+            if not record.state_id and record.partner_id.state_id:
+                record.state_id = record.partner_id.state_id
+            elif not record.state_id:
+                record.state_id = False
 
     @api.depends(lambda self: self._checkin_manual_fields())
     def _compute_state(self):
@@ -440,7 +434,7 @@ class PmsCheckinPartner(models.Model):
                 elif any(
                     not getattr(record, field)
                     for field in record._checkin_mandatory_fields(
-                        residence_country=record.residence_country_id,
+                        country=record.country_id,
                         document_type=record.document_type,
                         birthdate_date=record.birthdate_date,
                     )
@@ -735,19 +729,38 @@ class PmsCheckinPartner(models.Model):
                         _("Document type and country of document do not match")
                     )
 
-    @api.constrains("residence_state_id", "residence_country_id")
-    def _check_residence_state_id_residence_country_id_consistence(self):
+    @api.constrains("state_id", "country_id")
+    def _check_state_id_country_id_consistence(self):
         for record in self:
-            if record.residence_state_id and record.residence_country_id:
+            if record.state_id and record.country_id:
                 if (
-                    record.residence_state_id.country_id
-                    and record.residence_country_id
-                    not in record.residence_state_id.country_id
+                    record.state_id.country_id
+                    and record.country_id not in record.state_id.country_id
                 ):
                     raise ValidationError(
                         _("State and country of residence do not match")
                     )
 
+    def set_partner_address(self):
+        """
+        Only sets the checkin.partner address in the associated partner if
+        the partner don't have any address field filled.
+        """
+        for record in self:
+            if record.partner_id:
+                residence_vals = {
+                    "street": record.street,
+                    "street2": record.street2,
+                    "zip": record.zip,
+                    "city": record.city,
+                    "country_id": record.country_id.id,
+                    "state_id": record.state_id.id,
+                }
+                if any(residence_vals.values()):
+                    address_fields = residence_vals.keys()
+                    if not any(record.partner_id[field] for field in address_fields):
+                        record.partner_id.write(residence_vals)
+
     @api.model_create_multi
     def create(self, vals_list):
         records = self.env["pms.checkin.partner"]
@@ -782,6 +795,7 @@ class PmsCheckinPartner(models.Model):
                         "check-in in this reservation"
                     )
                 )
+        records.set_partner_address()
         return records
 
     def write(self, vals):
@@ -791,6 +805,7 @@ class PmsCheckinPartner(models.Model):
             tourist_tax_services_cmds = reservation._compute_tourist_tax_lines()
             if tourist_tax_services_cmds:
                 reservation.write({"service_ids": tourist_tax_services_cmds})
+        self.set_partner_address()
         return res
 
     def unlink(self):
@@ -814,12 +829,12 @@ class PmsCheckinPartner(models.Model):
             "document_number",
             "document_expedition_date",
             "nationality_id",
-            "residence_street",
-            "residence_street2",
-            "residence_zip",
-            "residence_city",
-            "residence_country_id",
-            "residence_state_id",
+            "street",
+            "street2",
+            "zip",
+            "city",
+            "country_id",
+            "state_id",
             "document_country_id",
             "document_type",
         ]
@@ -833,7 +848,7 @@ class PmsCheckinPartner(models.Model):
 
     @api.model
     def _checkin_mandatory_fields(
-        self, residence_country=False, document_type=False, birthdate_date=False
+        self, country=False, document_type=False, birthdate_date=False
     ):
         mandatory_fields = []
         return mandatory_fields
@@ -983,12 +998,10 @@ class PmsCheckinPartner(models.Model):
         )
         if values.get("document_type"):
             values.update({"document_type": int(values.get("document_type"))})
-        if values.get("residence_state_id"):
-            values.update({"residence_state_id": int(values.get("residence_state_id"))})
-        if values.get("residence_country_id"):
-            values.update(
-                {"residence_country_id": int(values.get("residence_country_id"))}
-            )
+        if values.get("state_id"):
+            values.update({"state_id": int(values.get("state_id"))})
+        if values.get("country_id"):
+            values.update({"country_id": int(values.get("country_id"))})
 
         if values.get("document_expedition_date"):
             values.update(

odoo/addons/pms/models/pms_folio.py

@@ -527,7 +527,6 @@ class PmsFolio(models.Model):
         string="Possible existing customer",
         compute="_compute_possible_existing_customer_ids",
         comodel_name="res.partner",
-        inverse_name="folio_possible_customer_id",
     )
     first_checkin = fields.Date(
         string="First Folio Checkin",

odoo/addons/pms/models/pms_reservation.py

@@ -644,7 +644,6 @@ class PmsReservation(models.Model):
         string="Possible existing customer",
         compute="_compute_possible_existing_customer_ids",
         comodel_name="res.partner",
-        inverse_name="reservation_possible_customer_id",
     )
 
     avoid_mails = fields.Boolean(

odoo/addons/pms/models/res_partner.py

@@ -88,51 +88,9 @@ class ResPartner(models.Model):
         inverse_name="partner_id",
     )
 
-    country_id = fields.Many2one(
-        readonly=False,
-        store=True,
-        index=True,
-        compute="_compute_country_id",
-    )
-    state_id = fields.Many2one(
-        readonly=False,
-        store=True,
-        index=True,
-        compute="_compute_state_id",
-    )
-    city = fields.Char(
-        readonly=False,
-        store=True,
-        compute="_compute_city",
-    )
-    street = fields.Char(
-        readonly=False,
-        store=True,
-        compute="_compute_street",
-    )
-    street2 = fields.Char(
-        readonly=False,
-        store=True,
-        compute="_compute_street2",
-    )
-    zip = fields.Char(
-        readonly=False,
-        store=True,
-        compute="_compute_zip",
-    )
     comment = fields.Text(
         tracking=True,
     )
-    reservation_possible_customer_id = fields.Many2one(
-        string="Possible Customer In Reservation", comodel_name="pms.reservation"
-    )
-    folio_possible_customer_id = fields.Many2one(
-        string="Possible Customer In Folio", comodel_name="pms.folio"
-    )
-    checkin_partner_possible_customer_id = fields.Many2one(
-        string="Possible Customer In Checkin Partner",
-        comodel_name="pms.checkin.partner",
-    )
     invoicing_policy = fields.Selection(
         help="""The invoicing policy of the partner,
          set Property to user the policy configured in the Property""",
@@ -151,208 +109,6 @@ class ResPartner(models.Model):
         string="Days from Checkout",
         help="Days from Checkout to generate the invoice",
     )
-    residence_street = fields.Char(
-        string="Street of residence",
-        help="Street of the guest's residence",
-        readonly=False,
-        store=True,
-        compute="_compute_residence_street",
-    )
-    residence_street2 = fields.Char(
-        string="Second street of residence",
-        help="Second street of the guest's residence",
-        readonly=False,
-        store=True,
-        compute="_compute_residence_street2",
-    )
-    residence_zip = fields.Char(
-        string="Zip of residence",
-        help="Zip of the guest's residence",
-        readonly=False,
-        store=True,
-        compute="_compute_residence_zip",
-        change_default=True,
-    )
-    residence_city = fields.Char(
-        string="city of residence",
-        help="City of the guest's residence",
-        readonly=False,
-        store=True,
-        compute="_compute_residence_city",
-    )
-    residence_country_id = fields.Many2one(
-        string="Country of residence",
-        help="Partner country of residence",
-        readonly=False,
-        store=True,
-        index=True,
-        compute="_compute_residence_country_id",
-        comodel_name="res.country",
-    )
-    residence_state_id = fields.Many2one(
-        string="State of residence",
-        help="Partner state of residence",
-        readonly=False,
-        store=True,
-        index=True,
-        compute="_compute_residence_state_id",
-        comodel_name="res.country.state",
-    )
-
-    # pylint: disable=W8110
-    @api.depends("pms_checkin_partner_ids", "pms_checkin_partner_ids.residence_street")
-    def _compute_residence_street(self):
-        if hasattr(super(), "_compute_residence_street"):
-            super()._compute_residence_street()
-        for record in self:
-            if record.pms_checkin_partner_ids:
-                last_update_street = record.pms_checkin_partner_ids.filtered(
-                    lambda x, r=record: x.write_date
-                    == max(r.pms_checkin_partner_ids.mapped("write_date"))
-                )
-                if last_update_street and last_update_street[0].residence_street:
-                    record.residence_street = last_update_street[0].residence_street
-
-    # pylint: disable=W8110
-    @api.depends("pms_checkin_partner_ids", "pms_checkin_partner_ids.residence_street2")
-    def _compute_residence_street2(self):
-        if hasattr(super(), "_compute_residence_street2"):
-            super()._compute_residence_street2()
-        for record in self:
-            if record.pms_checkin_partner_ids:
-                last_update_street2 = record.pms_checkin_partner_ids.filtered(
-                    lambda x, r=record: x.write_date
-                    == max(r.pms_checkin_partner_ids.mapped("write_date"))
-                )
-                if last_update_street2 and last_update_street2[0].residence_street2:
-                    record.residence_street2 = last_update_street2[0].residence_street2
-
-    # pylint: disable=W8110
-    @api.depends("pms_checkin_partner_ids", "pms_checkin_partner_ids.residence_zip")
-    def _compute_residence_zip(self):
-        if hasattr(super(), "_compute_residence_zip"):
-            super()._compute_residence_zip()
-        for record in self:
-            if record.pms_checkin_partner_ids:
-                last_update_zip = record.pms_checkin_partner_ids.filtered(
-                    lambda x, r=record: x.write_date
-                    == max(r.pms_checkin_partner_ids.mapped("write_date"))
-                )
-                if last_update_zip and last_update_zip[0].residence_zip:
-                    record.residence_zip = last_update_zip[0].residence_zip
-
-    # pylint: disable=W8110
-    @api.depends("pms_checkin_partner_ids", "pms_checkin_partner_ids.residence_city")
-    def _compute_residence_city(self):
-        if hasattr(super(), "_compute_residence_city"):
-            super()._compute_residence_city()
-        for record in self:
-            if record.pms_checkin_partner_ids:
-                last_update_city = record.pms_checkin_partner_ids.filtered(
-                    lambda x, r=record: x.write_date
-                    == max(r.pms_checkin_partner_ids.mapped("write_date"))
-                )
-                if last_update_city and last_update_city[0].residence_city:
-                    record.residence_city = last_update_city[0].residence_city
-
-    # pylint: disable=W8110
-    @api.depends(
-        "pms_checkin_partner_ids",
-        "pms_checkin_partner_ids.residence_country_id",
-        "nationality_id",
-    )
-    def _compute_residence_country_id(self):
-        if hasattr(super(), "_compute_residence_country_id"):
-            super()._compute_residence_country_id()
-        for record in self:
-            if record.pms_checkin_partner_ids:
-                last_update_country = record.pms_checkin_partner_ids.filtered(
-                    lambda x, r=record: x.write_date
-                    == max(r.pms_checkin_partner_ids.mapped("write_date"))
-                )
-                if last_update_country and last_update_country[0].residence_country_id:
-                    record.residence_country_id = last_update_country[
-                        0
-                    ].residence_country_id
-
-    # pylint: disable=W8110
-    @api.depends(
-        "pms_checkin_partner_ids", "pms_checkin_partner_ids.residence_state_id"
-    )
-    def _compute_residence_state_id(self):
-        if hasattr(super(), "_compute_residence_state_id"):
-            super()._compute_residence_state_id()
-        for record in self:
-            if record.pms_checkin_partner_ids:
-                last_update_state = record.pms_checkin_partner_ids.filtered(
-                    lambda x, r=record: x.write_date
-                    == max(r.pms_checkin_partner_ids.mapped("write_date"))
-                )
-                if last_update_state and last_update_state[0].residence_state_id:
-                    record.residence_state_id = last_update_state[0].residence_state_id
-
-    # pylint: disable=W8110
-    @api.depends("id_numbers")
-    def _compute_country_id(self):
-        if hasattr(super(), "_compute_country_id"):
-            super()._compute_country_id()
-        for record in self:
-            if (
-                not record.parent_id
-                and not record.country_id
-                and record.id_numbers
-                and record.id_numbers.country_id
-            ):
-                record.country_id = record.id_numbers[0].country_id
-
-    # pylint: disable=W8110
-    @api.depends("residence_state_id")
-    def _compute_state_id(self):
-        if hasattr(super(), "_compute_state_id"):
-            super()._compute_state_id()
-        for record in self:
-            if (
-                not record.parent_id
-                and not record.state_id
-                and record.residence_state_id
-            ):
-                record.state_id = record.residence_state_id
-
-    # pylint: disable=W8110
-    @api.depends("residence_city")
-    def _compute_city(self):
-        if hasattr(super(), "_compute_city"):
-            super()._compute_city()
-        for record in self:
-            if not record.parent_id and not record.city and record.residence_city:
-                record.city = record.residence_city
-
-    # pylint: disable=W8110
-    @api.depends("residence_street")
-    def _compute_street(self):
-        if hasattr(super(), "_compute_street"):
-            super()._compute_street()
-        for record in self:
-            if not record.parent_id and not record.street and record.residence_street:
-                record.street = record.residence_street
-
-    # pylint: disable=W8110
-    @api.depends("residence_street2")
-    def _compute_street2(self):
-        if hasattr(super(), "_compute_street2"):
-            super()._compute_street2()
-        for record in self:
-            if not record.parent_id and not record.street2 and record.residence_street2:
-                record.street2 = record.residence_street2
-
-    # pylint: disable=W8110
-    @api.depends("residence_zip")
-    def _compute_zip(self):
-        if hasattr(super(), "_compute_zip"):
-            super()._compute_zip()
-        for record in self:
-            if not record.parent_id and not record.zip and record.residence_zip:
-                record.zip = record.residence_zip
 
     def _compute_reservations_count(self):
         # Return reservation with partner included in reservation and/or checkin
@@ -494,46 +250,6 @@ class ResPartner(models.Model):
             if not record.is_agency and record.sale_channel_id:
                 record.sale_channel_id = None
 
-    # REVIEW: problems with odoo demo data
-    # @api.constrains("mobile", "email")
-    # def _check_duplicated(self):
-    #     for record in self:
-    #         partner, field = record._search_duplicated()
-    #         if partner:
-    #             raise models.ValidationError(
-    #                 _(
-    #                     "Partner %s found with same %s (%s)",
-    #                     partner.name,
-    #                     partner._fields[field].string,
-    #                     getattr(record, field),
-    #                 )
-    #             )
-
-    @api.constrains("residence_state_id", "residence_country_id")
-    def _check_residence_state_id_residence_country_id_consistence(self):
-        for record in self:
-            if record.residence_state_id and record.residence_country_id:
-                if (
-                    record.residence_state_id.country_id
-                    and record.residence_country_id
-                    not in record.residence_state_id.country_id
-                ):
-                    raise ValidationError(
-                        _("State and country of residence do not match")
-                    )
-
-    def _search_duplicated(self):
-        self.ensure_one()
-        partner = False
-        for field in self._get_key_fields():
-            if getattr(self, field):
-                partner = self.search(
-                    [(field, "=", getattr(self, field)), ("id", "!=", self.id)]
-                )
-                if partner:
-                    field = field
-        return partner, field
-
     @api.model
     def _get_key_fields(self):
         key_fields = super()._get_key_fields()
@@ -545,15 +261,6 @@ class ResPartner(models.Model):
         # Template to be inherited by localization modules
         return True
 
-    def unlink(self):
-        various_partner_id = self.env.ref("pms.various_pms_partner").id
-        if various_partner_id in self.ids:
-            various_partner = self.browse(various_partner_id)
-            raise ValidationError(
-                _("The partner %s cannot be deleted"), various_partner.name
-            )
-        return super().unlink()
-
     @api.model_create_multi
     def create(self, vals_list):
         for vals in vals_list:

odoo/addons/pms/static/description/index.html

@@ -372,7 +372,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:2965337f00a257145df7b6ebdffe06a52cd958e822a49a7345c2cd10662d0718
+!! source digest: sha256:7a9133a071292495a65eeaa21c559b55701ed046c6b661f3728055fedda6f778
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/pms/tree/16.0/pms"><img alt="OCA/pms" src="https://img.shields.io/badge/github-OCA%2Fpms-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/pms-16-0/pms-16-0-pms"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/pms&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module is an all-in-one property management system (PMS) focused on medium-sized properties

odoo/addons/pms/tests/test_pms_checkin_partner.py

@@ -1535,7 +1535,7 @@ class TestPmsCheckinPartner(TestPms):
             "email": "serafin@example.com",
             "gender": "male",
             "nationality_id": 1,
-            "residence_state_id": 1,
+            "state_id": 1,
         }
         checkin_partner._save_data_from_portal(checkin_partner_vals)
         checkin_partner_vals.update(
@@ -1543,7 +1543,7 @@ class TestPmsCheckinPartner(TestPms):
                 "birthdate_date": datetime.date(1983, 10, 5),
                 "document_expedition_date": datetime.date(2010, 10, 7),
                 "nationality_id": self.env["res.country"].search([("id", "=", 1)]),
-                "residence_state_id": self.env["res.country.state"].browse(1),
+                "state_id": self.env["res.country.state"].browse(1),
                 "document_type": self.id_category,
             }
         )
@@ -1597,12 +1597,12 @@ class TestPmsCheckinPartner(TestPms):
             "mobile": "666555444",
             "phone": "123456789",
             "nationality_id": nationality.id,
-            "residence_street": "Calle 123",
-            "residence_street2": "Avda. Constitución 123",
-            "residence_zip": "15700",
-            "residence_city": "City Residence",
-            "residence_country_id": nationality.id,
-            "residence_state_id": state.id,
+            "street": "Calle 123",
+            "street2": "Avda. Constitución 123",
+            "zip": "15700",
+            "city": "City Residence",
+            "country_id": nationality.id,
+            "state_id": state.id,
             # "pms_checkin_partner_ids": checkin_partner_id,
         }
         self.partner_id = self.env["res.partner"].create(partner_vals)
@@ -1610,8 +1610,8 @@ class TestPmsCheckinPartner(TestPms):
         partner_vals.update(
             {
                 "nationality_id": nationality,
-                "residence_country_id": nationality,
-                "residence_state_id": state,
+                "country_id": nationality,
+                "state_id": state,
             }
         )