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

fix: Check is_private when checking for duplicates (#7957)

Browse source 
* fix: Check is_private when checking for duplicates

* fix: Fallback file_size
This commit is contained in:
Faris Ansari 2019-07-23 19:11:19 +05:30 • committed by Suraj Shetty
parent a0de7c0dfb
commit 4aed6fa6d5
1 changed files with 15 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -188,14 +188,20 @@ class File(NestedSet):
# check duplicate name
# check duplicate assignement
n_records = frappe.db.sql("""select name from `tabFile`
where content_hash=%s
and name!=%s
and attached_to_doctype=%s
and attached_to_name=%s""", (self.content_hash, self.name, self.attached_to_doctype,
self.attached_to_name))
if len(n_records) > 0:
self.duplicate_entry = n_records[0][0]
filters = {
'content_hash': self.content_hash,
'is_private': self.is_private,
'name': ('!=', self.name)
}
if self.attached_to_doctype and self.attached_to_name:
filters.update({
'attached_to_doctype': self.attached_to_doctype,
'attached_to_name': self.attached_to_name
})
duplicate_file = frappe.db.get_value('File', filters)
if duplicate_file:
self.duplicate_entry = duplicate_file
frappe.throw(_("Same file has already been attached to the record"),
frappe.DuplicateEntryError)
@ -451,7 +457,7 @@ class File(NestedSet):
return
self.file_url = unquote(self.file_url)
self.file_size = frappe.form_dict.file_size
self.file_size = frappe.form_dict.file_size or self.file_size
def get_uploaded_content(self):