Skip to content

Commit

Permalink
fix: reposting file attachment permission issue (backport #42068) (#4…
Browse files Browse the repository at this point in the history
…2075)

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

(cherry picked from commit 03e674e)

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

* chore: fix conflicts

---------

Co-authored-by: rohitwaghchaure <rohitw1991@gmail.com>
  • Loading branch information
mergify[bot] and rohitwaghchaure authored Jun 29, 2024
1 parent d396c18 commit 1f3374f
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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();
},
});
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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",
Expand Down Expand Up @@ -276,4 +277,4 @@
"sort_field": "modified",
"sort_order": "DESC",
"states": []
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ def set_status(self, status=None, write=True):
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)
Expand Down Expand Up @@ -219,6 +219,7 @@ def restart_reposting(self):
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):
Expand Down Expand Up @@ -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:
Expand Down Expand Up @@ -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):
Expand Down

0 comments on commit 1f3374f

Please sign in to comment.