mirror of
https://github.com/frappe/erpnext.git
synced 2026-03-12 06:48:31 +00:00
fix: reposting file attachment permission issue (#42068)
This commit is contained in:
@@ -128,9 +128,7 @@ frappe.ui.form.on("Repost Item Valuation", {
|
||||
method: "restart_reposting",
|
||||
doc: frm.doc,
|
||||
callback: function (r) {
|
||||
if (!r.exc) {
|
||||
frm.refresh();
|
||||
}
|
||||
frm.reload_doc();
|
||||
},
|
||||
});
|
||||
},
|
||||
|
||||
@@ -218,13 +218,14 @@
|
||||
"fieldname": "reposting_data_file",
|
||||
"fieldtype": "Attach",
|
||||
"label": "Reposting Data File",
|
||||
"no_copy": 1,
|
||||
"read_only": 1
|
||||
}
|
||||
],
|
||||
"index_web_pages_for_search": 1,
|
||||
"is_submittable": 1,
|
||||
"links": [],
|
||||
"modified": "2024-03-27 13:10:32.544839",
|
||||
"modified": "2024-06-27 16:55:23.150146",
|
||||
"modified_by": "Administrator",
|
||||
"module": "Stock",
|
||||
"name": "Repost Item Valuation",
|
||||
|
||||
@@ -174,7 +174,7 @@ class RepostItemValuation(Document):
|
||||
def clear_attachment(self):
|
||||
if attachments := get_attachments(self.doctype, self.name):
|
||||
attachment = attachments[0]
|
||||
frappe.delete_doc("File", attachment.name)
|
||||
frappe.delete_doc("File", attachment.name, ignore_permissions=True)
|
||||
|
||||
if self.reposting_data_file:
|
||||
self.db_set("reposting_data_file", None)
|
||||
@@ -214,6 +214,7 @@ class RepostItemValuation(Document):
|
||||
self.distinct_item_and_warehouse = None
|
||||
self.items_to_be_repost = None
|
||||
self.gl_reposting_index = 0
|
||||
self.clear_attachment()
|
||||
self.db_update()
|
||||
|
||||
def deduplicate_similar_repost(self):
|
||||
@@ -266,6 +267,7 @@ def repost(doc):
|
||||
repost_gl_entries(doc)
|
||||
|
||||
doc.set_status("Completed")
|
||||
doc.db_set("reposting_data_file", None)
|
||||
remove_attached_file(doc.name)
|
||||
|
||||
except Exception as e:
|
||||
@@ -310,7 +312,7 @@ def remove_attached_file(docname):
|
||||
if file_name := frappe.db.get_value(
|
||||
"File", {"attached_to_name": docname, "attached_to_doctype": "Repost Item Valuation"}, "name"
|
||||
):
|
||||
frappe.delete_doc("File", file_name, delete_permanently=True)
|
||||
frappe.delete_doc("File", file_name, ignore_permissions=True, delete_permanently=True)
|
||||
|
||||
|
||||
def repost_sl_entries(doc):
|
||||
|
||||
Reference in New Issue
Block a user