From 64fe2749cb33f776ae9047ee2d29b660477ba566 Mon Sep 17 00:00:00 2001 From: Shreya Shah Date: Mon, 16 Apr 2018 14:55:34 +0530 Subject: [PATCH] avoid link validation error in authentication (#5415) --- frappe/core/doctype/activity_log/activity_log.py | 2 +- frappe/model/document.py | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/frappe/core/doctype/activity_log/activity_log.py b/frappe/core/doctype/activity_log/activity_log.py index 33e444650b..94a4803279 100644 --- a/frappe/core/doctype/activity_log/activity_log.py +++ b/frappe/core/doctype/activity_log/activity_log.py @@ -41,7 +41,7 @@ def add_authentication_log(subject, user, operation="Login", status="Success"): "status": status, "subject": subject, "operation": operation, - }).insert(ignore_permissions=True) + }).insert(ignore_permissions=True, ignore_links=True) def clear_authentication_logs(): """clear 100 day old authentication logs""" diff --git a/frappe/model/document.py b/frappe/model/document.py index 02c77a2884..0e232fc006 100644 --- a/frappe/model/document.py +++ b/frappe/model/document.py @@ -184,7 +184,7 @@ class Document(BaseDocument): frappe.flags.error_message = _('Insufficient Permission for {0}').format(self.doctype) raise frappe.PermissionError - def insert(self, ignore_permissions=None, ignore_if_duplicate=False, ignore_mandatory=None): + def insert(self, ignore_permissions=None, ignore_links=None, ignore_if_duplicate=False, ignore_mandatory=None): """Insert the document in the database (as a new document). This will check for user permissions and execute `before_insert`, `validate`, `on_update`, `after_insert` methods if they are written. @@ -198,6 +198,9 @@ class Document(BaseDocument): if ignore_permissions!=None: self.flags.ignore_permissions = ignore_permissions + if ignore_links!=None: + self.flags.ignore_links = ignore_links + if ignore_mandatory!=None: self.flags.ignore_mandatory = ignore_mandatory