vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: adds check before enabling notification settings

Browse source
This commit is contained in:
Parth Kharwar 2021-02-26 16:40:09 +05:30
parent 9e5ca78d78
commit a8d11f23d8
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
frappe/desk/doctype/notification_settings/notification_settings.py
View file

@ -44,7 +44,8 @@ def create_notification_settings(user):
_doc.insert(ignore_permissions=True)
def enable_disable_notifications(user, enabled):
frappe.set_value("Notification Settings", user, 'enabled', enabled)
if frappe.db.exists("Notification Settings", user):
frappe.set_value("Notification Settings", user, 'enabled', enabled)
@frappe.whitelist()
@ -78,4 +79,4 @@ def get_permission_query_conditions(user):
@frappe.whitelist()
def set_seen_value(value, user):
frappe.db.set_value('Notification Settings', user, 'seen', value, update_modified=False)
frappe.db.set_value('Notification Settings', user, 'seen', value, update_modified=False)