From 5cd3f8534b55658d24b7aff48cdeb5d208e10c2e Mon Sep 17 00:00:00 2001 From: Sagar Vora Date: Tue, 9 Mar 2021 16:36:36 +0530 Subject: [PATCH] fix: create gender only if provided while creating oauth user --- frappe/utils/oauth.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/frappe/utils/oauth.py b/frappe/utils/oauth.py index 40bde6c2cc..6596701ee3 100644 --- a/frappe/utils/oauth.py +++ b/frappe/utils/oauth.py @@ -231,14 +231,14 @@ def update_oauth_user(user, data, provider): save = True user = frappe.new_doc("User") - gender = (data.get("gender") or "").title() + gender = data.get("gender", "").title() - if not frappe.db.exists("Gender", gender): + if gender and not frappe.db.exists("Gender", gender): doc = frappe.new_doc("Gender", {"gender": gender}) doc.insert(ignore_permissions=True) user.update({ - "doctype":"User", + "doctype": "User", "first_name": get_first_name(data), "last_name": get_last_name(data), "email": get_email(data),