Merge pull request #36918 from frappe/mergify/bp/version-14-hotfix/pr…

…-36908 fix: only show "Unreconcile" if reconciled (#36908)
frappe · Sep 2, 2023 · 78051e7 · 78051e7
2 parents 543e4c8 + 61752ac
commit 78051e7
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/erpnext/accounts/doctype/bank_transaction/bank_transaction.js b/erpnext/accounts/doctype/bank_transaction/bank_transaction.js
@@ -13,10 +13,11 @@ frappe.ui.form.on("Bank Transaction", {
 		});
 	},
 	refresh(frm) {
-		frm.add_custom_button(__('Unreconcile Transaction'), () => {
-			frm.call('remove_payment_entries')
-			.then( () => frm.refresh() );
-		});
+		if (!frm.is_dirty() && frm.doc.payment_entries.length > 0) {
+			frm.add_custom_button(__("Unreconcile Transaction"), () => {
+				frm.call("remove_payment_entries").then(() => frm.refresh());
+			});
+		}
 	},
 	bank_account: function (frm) {
 		set_bank_statement_filter(frm);