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

Merge pull request #31360 from sokumon/attach-file-spam-issue

Browse source 
fix: check if duplicate_file url matches incoming file url
This commit is contained in:
Soham Kulkarni 2025-02-25 14:33:35 +05:30 • committed by GitHub
commit 0040c7f852
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
frappe/core/doctype/file/file.py
View file

@ -386,8 +386,6 @@ class File(Document):
if self.name:
filters.update({"name": ("!=", self.name)})
else:
filters.update({"file_name": self.file_name})
if self.attached_to_doctype and self.attached_to_name:
filters.update(
@ -663,7 +661,11 @@ class File(Document):
if duplicate_file:
file_doc: File = frappe.get_cached_doc("File", duplicate_file.name)
if file_doc.exists_on_disk():
self.file_url = duplicate_file.file_url
if self.exists_on_disk():
if not self.file_url:
self.file_url = duplicate_file.file_url
else:
self.file_url = duplicate_file.file_url
file_exists = True
if not file_exists: