From 4d74d9ef53a6c7b8affc1143227cf5ba37a7dd8b Mon Sep 17 00:00:00 2001 From: Olivier Dony Date: Mon, 30 Jun 2014 11:58:11 +0200 Subject: [PATCH] [FIX] auth_signup: discard signup values that may overwriting existing info --- addons/auth_signup/res_users.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/addons/auth_signup/res_users.py b/addons/auth_signup/res_users.py index 5a744c21c81..576a778be45 100644 --- a/addons/auth_signup/res_users.py +++ b/addons/auth_signup/res_users.py @@ -201,6 +201,14 @@ class res_users(osv.Model): partner.write({'signup_token': False, 'signup_type': False, 'signup_expiration': False}) partner_user = partner.user_ids and partner.user_ids[0] or False + + # avoid overwriting existing (presumably correct) values with geolocation data + if partner.country_id or partner.zip or partner.city: + values.pop('city', None) + values.pop('country_id', None) + if partner.lang: + values.pop('lang', None) + if partner_user: # user exists, modify it according to values values.pop('login', None)