CalvinBackup/erpnext
1
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-03-15 00:06:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: reposting file attachment permission issue (backport #42068) (#42075)

Browse Source 
* fix: reposting file attachment permission issue (#42068)

(cherry picked from commit 03e674e21d)

# Conflicts:
#	erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.json

* chore: fix conflicts

---------

Co-authored-by: rohitwaghchaure <rohitw1991@gmail.com>
This commit is contained in:
mergify[bot]
2024-06-29 07:32:45 +05:30
committed by GitHub
parent d396c18689
commit 1f3374fcdf
3 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.js
View File

@@ -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();
},
});
},

5
erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.json
View File

@@ -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": "2023-05-31 12:48:57.138693",
"modified": "2024-06-27 16:55:23.150146",
"modified_by": "Administrator",
"module": "Stock",
"name": "Repost Item Valuation",
@@ -276,4 +277,4 @@
"sort_field": "modified",
"sort_order": "DESC",
"states": []
}
}

6
erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.py
View File

@@ -179,7 +179,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)
@@ -219,6 +219,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):
@@ -271,6 +272,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:
@@ -315,7 +317,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):