Skip to content

Commit

Permalink
[FIX] website_portal_contact: don't set parent_id with the same partn…
Browse files Browse the repository at this point in the history
…er_id
  • Loading branch information
mathben committed Mar 8, 2020
1 parent ce1a1d9 commit d268b6e
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions website_portal_contact/controllers/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,13 +118,14 @@ def _contacts_fields_check(self, received):
_("Fields not available: %s") % ", ".join(disallowed)
)

def _contacts_clean_values(self, values):
def _contacts_clean_values(self, values, contact=False):
"""Set values to a write-compatible format"""
result = {k: v or False for k, v in values.items()}
result.setdefault("type", "contact")
result.setdefault(
"parent_id", request.env.user.commercial_partner_id.id
)
if not contact or contact.id != request.env.user.commercial_partner_id.id:
result.setdefault(
"parent_id", request.env.user.commercial_partner_id.id
)
return result

@route(
Expand Down Expand Up @@ -189,7 +190,7 @@ def portal_my_contacts_update(
):
"""Update a contact."""
self._contacts_fields_check(kwargs.keys())
values = self._contacts_clean_values(kwargs)
values = self._contacts_clean_values(kwargs, contact=contact)
_logger.debug("Updating %r with: %s", contact, values)
contact.write(values)
return local_redirect(redirect.format(contact.id))
Expand Down

0 comments on commit d268b6e

Please sign in to comment.