From b2a84fe478ff811662221d438ace338431286a63 Mon Sep 17 00:00:00 2001 From: Saqib Ansari Date: Mon, 6 Apr 2020 16:32:33 +0530 Subject: [PATCH 1/2] chore: rename bank reconciliation to bank clearance --- .../accounting_period/accounting_period.py | 2 +- .../README.md | 0 .../doctype/bank_clearance/__init__.py | 0 .../bank_clearance.js} | 2 +- .../bank_clearance/bank_clearance.json | 130 +++++ .../bank_clearance.py} | 2 +- .../bank_clearance/test_bank_clearance.py | 10 + .../doctype/bank_clearance_detail/README.md | 1 + .../__init__.py | 0 .../bank_clearance_detail.json} | 2 +- .../bank_clearance_detail.py} | 2 +- .../bank_reconciliation.json | 484 ------------------ .../test_bank_reconciliation.js | 22 - .../test_bank_reconciliation.py | 8 - .../bank_reconciliation_detail/README.md | 1 - .../bank_reconciliation_detail/__init__.py | 1 - erpnext/config/accounts.py | 4 +- .../v12_0/rename_bank_reconciliation.py | 14 + .../rename_bank_reconciliation_fields.py | 6 +- 19 files changed, 165 insertions(+), 526 deletions(-) rename erpnext/accounts/doctype/{bank_reconciliation => bank_clearance}/README.md (100%) create mode 100644 erpnext/accounts/doctype/bank_clearance/__init__.py rename erpnext/accounts/doctype/{bank_reconciliation/bank_reconciliation.js => bank_clearance/bank_clearance.js} (97%) create mode 100644 erpnext/accounts/doctype/bank_clearance/bank_clearance.json rename erpnext/accounts/doctype/{bank_reconciliation/bank_reconciliation.py => bank_clearance/bank_clearance.py} (99%) create mode 100644 erpnext/accounts/doctype/bank_clearance/test_bank_clearance.py create mode 100644 erpnext/accounts/doctype/bank_clearance_detail/README.md rename erpnext/accounts/doctype/{bank_reconciliation => bank_clearance_detail}/__init__.py (100%) rename erpnext/accounts/doctype/{bank_reconciliation_detail/bank_reconciliation_detail.json => bank_clearance_detail/bank_clearance_detail.json} (99%) rename erpnext/accounts/doctype/{bank_reconciliation_detail/bank_reconciliation_detail.py => bank_clearance_detail/bank_clearance_detail.py} (84%) delete mode 100644 erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.json delete mode 100644 erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.js delete mode 100644 erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.py delete mode 100644 erpnext/accounts/doctype/bank_reconciliation_detail/README.md delete mode 100644 erpnext/accounts/doctype/bank_reconciliation_detail/__init__.py create mode 100644 erpnext/patches/v12_0/rename_bank_reconciliation.py diff --git a/erpnext/accounts/doctype/accounting_period/accounting_period.py b/erpnext/accounts/doctype/accounting_period/accounting_period.py index f48d6dfc953..df6cedd7cf8 100644 --- a/erpnext/accounts/doctype/accounting_period/accounting_period.py +++ b/erpnext/accounts/doctype/accounting_period/accounting_period.py @@ -42,7 +42,7 @@ class AccountingPeriod(Document): def get_doctypes_for_closing(self): docs_for_closing = [] doctypes = ["Sales Invoice", "Purchase Invoice", "Journal Entry", "Payroll Entry", \ - "Bank Reconciliation", "Asset", "Stock Entry"] + "Bank Clearance", "Asset", "Stock Entry"] closed_doctypes = [{"document_type": doctype, "closed": 1} for doctype in doctypes] for closed_doctype in closed_doctypes: docs_for_closing.append(closed_doctype) diff --git a/erpnext/accounts/doctype/bank_reconciliation/README.md b/erpnext/accounts/doctype/bank_clearance/README.md similarity index 100% rename from erpnext/accounts/doctype/bank_reconciliation/README.md rename to erpnext/accounts/doctype/bank_clearance/README.md diff --git a/erpnext/accounts/doctype/bank_clearance/__init__.py b/erpnext/accounts/doctype/bank_clearance/__init__.py new file mode 100644 index 00000000000..e69de29bb2d diff --git a/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.js b/erpnext/accounts/doctype/bank_clearance/bank_clearance.js similarity index 97% rename from erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.js rename to erpnext/accounts/doctype/bank_clearance/bank_clearance.js index 19fadbf6ded..ba3f2face63 100644 --- a/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.js +++ b/erpnext/accounts/doctype/bank_clearance/bank_clearance.js @@ -1,7 +1,7 @@ // Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors // License: GNU General Public License v3. See license.txt -frappe.ui.form.on("Bank Reconciliation", { +frappe.ui.form.on("Bank Clearance", { setup: function(frm) { frm.add_fetch("account", "account_currency", "account_currency"); }, diff --git a/erpnext/accounts/doctype/bank_clearance/bank_clearance.json b/erpnext/accounts/doctype/bank_clearance/bank_clearance.json new file mode 100644 index 00000000000..a436d1effb8 --- /dev/null +++ b/erpnext/accounts/doctype/bank_clearance/bank_clearance.json @@ -0,0 +1,130 @@ +{ + "allow_copy": 1, + "creation": "2013-01-10 16:34:05", + "doctype": "DocType", + "document_type": "Document", + "engine": "InnoDB", + "field_order": [ + "account", + "account_currency", + "from_date", + "to_date", + "column_break_5", + "bank_account", + "include_reconciled_entries", + "include_pos_transactions", + "get_payment_entries", + "section_break_10", + "payment_entries", + "update_clearance_date", + "total_amount" + ], + "fields": [ + { + "fetch_from": "bank_account.account", + "fetch_if_empty": 1, + "fieldname": "account", + "fieldtype": "Link", + "in_list_view": 1, + "label": "Account", + "options": "Account", + "reqd": 1 + }, + { + "fieldname": "account_currency", + "fieldtype": "Link", + "hidden": 1, + "label": "Account Currency", + "options": "Currency", + "print_hide": 1 + }, + { + "fieldname": "from_date", + "fieldtype": "Date", + "in_list_view": 1, + "label": "From Date", + "reqd": 1 + }, + { + "fieldname": "to_date", + "fieldtype": "Date", + "in_list_view": 1, + "label": "To Date", + "reqd": 1 + }, + { + "fieldname": "column_break_5", + "fieldtype": "Column Break" + }, + { + "description": "Select the Bank Account to reconcile.", + "fieldname": "bank_account", + "fieldtype": "Link", + "label": "Bank Account", + "options": "Bank Account" + }, + { + "default": "0", + "fieldname": "include_reconciled_entries", + "fieldtype": "Check", + "in_list_view": 1, + "label": "Include Reconciled Entries" + }, + { + "default": "0", + "fieldname": "include_pos_transactions", + "fieldtype": "Check", + "label": "Include POS Transactions" + }, + { + "fieldname": "get_payment_entries", + "fieldtype": "Button", + "label": "Get Payment Entries" + }, + { + "fieldname": "section_break_10", + "fieldtype": "Section Break" + }, + { + "allow_bulk_edit": 1, + "fieldname": "payment_entries", + "fieldtype": "Table", + "label": "Payment Entries", + "options": "Bank Clearance Detail" + }, + { + "fieldname": "update_clearance_date", + "fieldtype": "Button", + "label": "Update Clearance Date" + }, + { + "fieldname": "total_amount", + "fieldtype": "Currency", + "label": "Total Amount", + "options": "account_currency", + "read_only": 1 + } + ], + "hide_toolbar": 1, + "icon": "fa fa-check", + "idx": 1, + "issingle": 1, + "modified": "2020-04-06 16:12:06.628008", + "modified_by": "Administrator", + "module": "Accounts", + "name": "Bank Clearance", + "owner": "Administrator", + "permissions": [ + { + "create": 1, + "read": 1, + "role": "Accounts User", + "share": 1, + "write": 1 + } + ], + "quick_entry": 1, + "read_only": 1, + "sort_field": "modified", + "sort_order": "ASC" +} \ No newline at end of file diff --git a/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.py b/erpnext/accounts/doctype/bank_clearance/bank_clearance.py similarity index 99% rename from erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.py rename to erpnext/accounts/doctype/bank_clearance/bank_clearance.py index 48fd154a4db..6fec3ab3681 100644 --- a/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.py +++ b/erpnext/accounts/doctype/bank_clearance/bank_clearance.py @@ -11,7 +11,7 @@ form_grid_templates = { "journal_entries": "templates/form_grid/bank_reconciliation_grid.html" } -class BankReconciliation(Document): +class BankClearance(Document): def get_payment_entries(self): if not (self.from_date and self.to_date): frappe.throw(_("From Date and To Date are Mandatory")) diff --git a/erpnext/accounts/doctype/bank_clearance/test_bank_clearance.py b/erpnext/accounts/doctype/bank_clearance/test_bank_clearance.py new file mode 100644 index 00000000000..833abde5ce3 --- /dev/null +++ b/erpnext/accounts/doctype/bank_clearance/test_bank_clearance.py @@ -0,0 +1,10 @@ +# -*- coding: utf-8 -*- +# Copyright (c) 2020, Frappe Technologies Pvt. Ltd. and Contributors +# See license.txt +from __future__ import unicode_literals + +# import frappe +import unittest + +class TestBankClearance(unittest.TestCase): + pass diff --git a/erpnext/accounts/doctype/bank_clearance_detail/README.md b/erpnext/accounts/doctype/bank_clearance_detail/README.md new file mode 100644 index 00000000000..ee83a44765c --- /dev/null +++ b/erpnext/accounts/doctype/bank_clearance_detail/README.md @@ -0,0 +1 @@ +Detail of transaction for parent Bank Clearance. \ No newline at end of file diff --git a/erpnext/accounts/doctype/bank_reconciliation/__init__.py b/erpnext/accounts/doctype/bank_clearance_detail/__init__.py similarity index 100% rename from erpnext/accounts/doctype/bank_reconciliation/__init__.py rename to erpnext/accounts/doctype/bank_clearance_detail/__init__.py diff --git a/erpnext/accounts/doctype/bank_reconciliation_detail/bank_reconciliation_detail.json b/erpnext/accounts/doctype/bank_clearance_detail/bank_clearance_detail.json similarity index 99% rename from erpnext/accounts/doctype/bank_reconciliation_detail/bank_reconciliation_detail.json rename to erpnext/accounts/doctype/bank_clearance_detail/bank_clearance_detail.json index 183068c3aef..04988bf66d8 100644 --- a/erpnext/accounts/doctype/bank_reconciliation_detail/bank_reconciliation_detail.json +++ b/erpnext/accounts/doctype/bank_clearance_detail/bank_clearance_detail.json @@ -326,7 +326,7 @@ "modified": "2019-01-07 16:52:07.174687", "modified_by": "Administrator", "module": "Accounts", - "name": "Bank Reconciliation Detail", + "name": "Bank Clearance Detail", "owner": "Administrator", "permissions": [], "quick_entry": 1, diff --git a/erpnext/accounts/doctype/bank_reconciliation_detail/bank_reconciliation_detail.py b/erpnext/accounts/doctype/bank_clearance_detail/bank_clearance_detail.py similarity index 84% rename from erpnext/accounts/doctype/bank_reconciliation_detail/bank_reconciliation_detail.py rename to erpnext/accounts/doctype/bank_clearance_detail/bank_clearance_detail.py index 990b2a640f0..ecc536733f2 100644 --- a/erpnext/accounts/doctype/bank_reconciliation_detail/bank_reconciliation_detail.py +++ b/erpnext/accounts/doctype/bank_clearance_detail/bank_clearance_detail.py @@ -5,5 +5,5 @@ from __future__ import unicode_literals import frappe from frappe.model.document import Document -class BankReconciliationDetail(Document): +class BankClearanceDetail(Document): pass \ No newline at end of file diff --git a/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.json b/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.json deleted file mode 100644 index b85ef3e9c4d..00000000000 --- a/erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.json +++ /dev/null @@ -1,484 +0,0 @@ -{ - "allow_copy": 1, - "allow_events_in_timeline": 0, - "allow_guest_to_view": 0, - "allow_import": 0, - "allow_rename": 0, - "beta": 0, - "creation": "2013-01-10 16:34:05", - "custom": 0, - "docstatus": 0, - "doctype": "DocType", - "document_type": "Document", - "editable_grid": 0, - "fields": [ - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_from": "bank_account.account", - "fetch_if_empty": 1, - "fieldname": "account", - "fieldtype": "Link", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 1, - "in_standard_filter": 0, - "label": "Account", - "length": 0, - "no_copy": 0, - "options": "Account", - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 1, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "account_currency", - "fieldtype": "Link", - "hidden": 1, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Account Currency", - "length": 0, - "no_copy": 0, - "options": "Currency", - "permlevel": 0, - "print_hide": 1, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "from_date", - "fieldtype": "Date", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 1, - "in_standard_filter": 0, - "label": "From Date", - "length": 0, - "no_copy": 0, - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 1, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "to_date", - "fieldtype": "Date", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 1, - "in_standard_filter": 0, - "label": "To Date", - "length": 0, - "no_copy": 0, - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 1, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "column_break_5", - "fieldtype": "Column Break", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "length": 0, - "no_copy": 0, - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "description": "Select the Bank Account to reconcile.", - "fetch_if_empty": 0, - "fieldname": "bank_account", - "fieldtype": "Link", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Bank Account", - "length": 0, - "no_copy": 0, - "options": "Bank Account", - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "include_reconciled_entries", - "fieldtype": "Check", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 1, - "in_standard_filter": 0, - "label": "Include Reconciled Entries", - "length": 0, - "no_copy": 0, - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "include_pos_transactions", - "fieldtype": "Check", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Include POS Transactions", - "length": 0, - "no_copy": 0, - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "get_payment_entries", - "fieldtype": "Button", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Get Payment Entries", - "length": 0, - "no_copy": 0, - "options": "", - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "section_break_10", - "fieldtype": "Section Break", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "length": 0, - "no_copy": 0, - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 1, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "payment_entries", - "fieldtype": "Table", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Payment Entries", - "length": 0, - "no_copy": 0, - "options": "Bank Reconciliation Detail", - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "update_clearance_date", - "fieldtype": "Button", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Update Clearance Date", - "length": 0, - "no_copy": 0, - "options": "", - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_in_quick_entry": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fetch_if_empty": 0, - "fieldname": "total_amount", - "fieldtype": "Currency", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Total Amount", - "length": 0, - "no_copy": 0, - "options": "account_currency", - "permlevel": 0, - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 1, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "translatable": 0, - "unique": 0 - } - ], - "has_web_view": 0, - "hide_toolbar": 1, - "icon": "fa fa-check", - "idx": 1, - "in_create": 0, - "is_submittable": 0, - "issingle": 1, - "istable": 0, - "max_attachments": 0, - "menu_index": 0, - "modified": "2020-01-22 00:00:00.000000", - "modified_by": "Administrator", - "module": "Accounts", - "name": "Bank Reconciliation", - "owner": "Administrator", - "permissions": [ - { - "amend": 0, - "cancel": 0, - "create": 1, - "delete": 0, - "email": 0, - "export": 0, - "if_owner": 0, - "import": 0, - "permlevel": 0, - "print": 0, - "read": 1, - "report": 0, - "role": "Accounts User", - "set_user_permissions": 0, - "share": 1, - "submit": 0, - "write": 1 - } - ], - "quick_entry": 1, - "read_only": 1, - "show_name_in_global_search": 0, - "sort_order": "ASC", - "track_changes": 0, - "track_seen": 0, - "track_views": 0 -} diff --git a/erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.js b/erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.js deleted file mode 100644 index f52f6fb431a..00000000000 --- a/erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.js +++ /dev/null @@ -1,22 +0,0 @@ -QUnit.module('Account'); - -QUnit.test("test Bank Reconciliation", function(assert) { - assert.expect(0); - let done = assert.async(); - frappe.run_serially([ - () => frappe.set_route('Form', 'Bank Reconciliation'), - () => cur_frm.set_value('bank_account','Cash - FT'), - () => frappe.click_button('Get Payment Entries'), - () => { - for(var i=0;i<=cur_frm.doc.payment_entries.length-1;i++){ - cur_frm.doc.payment_entries[i].clearance_date = frappe.datetime.add_days(frappe.datetime.now_date(), 2); - } - }, - () => {cur_frm.refresh_fields('payment_entries');}, - () => frappe.click_button('Update Clearance Date'), - () => frappe.timeout(0.5), - () => frappe.click_button('Close'), - () => done() - ]); -}); - diff --git a/erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.py b/erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.py deleted file mode 100644 index 932fb3384cf..00000000000 --- a/erpnext/accounts/doctype/bank_reconciliation/test_bank_reconciliation.py +++ /dev/null @@ -1,8 +0,0 @@ -# -*- coding: utf-8 -*- -# Copyright (c) 2018, Frappe Technologies Pvt. Ltd. and Contributors -# See license.txt -from __future__ import unicode_literals -import unittest - -class TestBankReconciliation(unittest.TestCase): - pass diff --git a/erpnext/accounts/doctype/bank_reconciliation_detail/README.md b/erpnext/accounts/doctype/bank_reconciliation_detail/README.md deleted file mode 100644 index 07d0731ed27..00000000000 --- a/erpnext/accounts/doctype/bank_reconciliation_detail/README.md +++ /dev/null @@ -1 +0,0 @@ -Detail of transaction for parent Bank Reconciliation. \ No newline at end of file diff --git a/erpnext/accounts/doctype/bank_reconciliation_detail/__init__.py b/erpnext/accounts/doctype/bank_reconciliation_detail/__init__.py deleted file mode 100644 index baffc488252..00000000000 --- a/erpnext/accounts/doctype/bank_reconciliation_detail/__init__.py +++ /dev/null @@ -1 +0,0 @@ -from __future__ import unicode_literals diff --git a/erpnext/config/accounts.py b/erpnext/config/accounts.py index 08711fc09ea..b9b0da48c95 100644 --- a/erpnext/config/accounts.py +++ b/erpnext/config/accounts.py @@ -183,8 +183,8 @@ def get_data(): }, { "type": "doctype", - "label": _("Update Bank Transaction Dates"), - "name": "Bank Reconciliation", + "label": _("Update Bank Clearance Dates"), + "name": "Bank Clearance", "description": _("Update bank payment dates with journals.") }, { diff --git a/erpnext/patches/v12_0/rename_bank_reconciliation.py b/erpnext/patches/v12_0/rename_bank_reconciliation.py new file mode 100644 index 00000000000..556224490c7 --- /dev/null +++ b/erpnext/patches/v12_0/rename_bank_reconciliation.py @@ -0,0 +1,14 @@ +# Copyright (c) 2018, Frappe and Contributors +# License: GNU General Public License v3. See license.txt + +from __future__ import unicode_literals +import frappe +from frappe.model.utils.rename_field import rename_field + +def execute(): + if frappe.db.table_exists("Bank Reconciliation"): + frappe.rename_doc('DocType', 'Bank Reconciliation', 'Bank Clearance', force=True) + frappe.reload_doc('Accounts', 'doctype', 'Bank Clearance') + + frappe.rename_doc('DocType', 'Bank Reconciliation Detail', 'Bank Clearance Detail', force=True) + frappe.reload_doc('Accounts', 'doctype', 'Bank Clearance Detail') diff --git a/erpnext/patches/v12_0/rename_bank_reconciliation_fields.py b/erpnext/patches/v12_0/rename_bank_reconciliation_fields.py index caeda8ae61b..978b1c92b96 100644 --- a/erpnext/patches/v12_0/rename_bank_reconciliation_fields.py +++ b/erpnext/patches/v12_0/rename_bank_reconciliation_fields.py @@ -9,6 +9,6 @@ def _rename_single_field(**kwargs): frappe.db.sql("UPDATE tabSingles SET field='{new_name}' WHERE doctype='{doctype}' AND field='{old_name}';".format(**kwargs)) #nosec def execute(): - _rename_single_field(doctype = "Bank Reconciliation", old_name = "bank_account" , new_name = "account") - _rename_single_field(doctype = "Bank Reconciliation", old_name = "bank_account_no", new_name = "bank_account") - frappe.reload_doc("Accounts", "doctype", "Bank Reconciliation") + _rename_single_field(doctype = "Bank Clearance", old_name = "bank_account" , new_name = "account") + _rename_single_field(doctype = "Bank Clearance", old_name = "bank_account_no", new_name = "bank_account") + frappe.reload_doc("Accounts", "doctype", "Bank Clearance") From 9de76de621707d50a1e9376a9b20c63a4bcab160 Mon Sep 17 00:00:00 2001 From: Saqib Ansari Date: Mon, 20 Apr 2020 12:19:27 +0530 Subject: [PATCH 2/2] fix: patch --- .../desk_page/accounting/accounting.json | 2 +- erpnext/patches.txt | 1 + .../patches/v12_0/rename_bank_reconciliation.py | 16 +++++++++------- 3 files changed, 11 insertions(+), 8 deletions(-) diff --git a/erpnext/accounts/desk_page/accounting/accounting.json b/erpnext/accounts/desk_page/accounting/accounting.json index 7a777f9fd23..d0b747d83f0 100644 --- a/erpnext/accounts/desk_page/accounting/accounting.json +++ b/erpnext/accounts/desk_page/accounting/accounting.json @@ -39,7 +39,7 @@ "title": "Bank Statement" }, { - "links": "[\n {\n \"description\": \"Match non-linked Invoices and Payments.\",\n \"label\": \"Match Payments with Invoices\",\n \"name\": \"Payment Reconciliation\",\n \"type\": \"doctype\"\n },\n {\n \"description\": \"Update bank payment dates with journals.\",\n \"label\": \"Update Bank Transaction Dates\",\n \"name\": \"Bank Reconciliation\",\n \"type\": \"doctype\"\n },\n {\n \"label\": \"Invoice Discounting\",\n \"name\": \"Invoice Discounting\",\n \"type\": \"doctype\"\n },\n {\n \"dependencies\": [\n \"Journal Entry\"\n ],\n \"doctype\": \"Journal Entry\",\n \"is_query_report\": true,\n \"label\": \"Bank Reconciliation Statement\",\n \"name\": \"Bank Reconciliation Statement\",\n \"type\": \"report\"\n },\n {\n \"icon\": \"fa fa-bar-chart\",\n \"label\": \"Bank Reconciliation\",\n \"name\": \"bank-reconciliation\",\n \"type\": \"page\"\n },\n {\n \"dependencies\": [\n \"Journal Entry\"\n ],\n \"doctype\": \"Journal Entry\",\n \"is_query_report\": true,\n \"label\": \"Bank Clearance Summary\",\n \"name\": \"Bank Clearance Summary\",\n \"type\": \"report\"\n },\n {\n \"label\": \"Bank Guarantee\",\n \"name\": \"Bank Guarantee\",\n \"type\": \"doctype\"\n },\n {\n \"description\": \"Setup cheque dimensions for printing\",\n \"label\": \"Cheque Print Template\",\n \"name\": \"Cheque Print Template\",\n \"type\": \"doctype\"\n }\n]", + "links": "[\n {\n \"description\": \"Match non-linked Invoices and Payments.\",\n \"label\": \"Match Payments with Invoices\",\n \"name\": \"Payment Reconciliation\",\n \"type\": \"doctype\"\n },\n {\n \"description\": \"Update bank payment dates with journals.\",\n \"label\": \"Update Bank Clearance Dates\",\n \"name\": \"Bank Clearance\",\n \"type\": \"doctype\"\n },\n {\n \"label\": \"Invoice Discounting\",\n \"name\": \"Invoice Discounting\",\n \"type\": \"doctype\"\n },\n {\n \"dependencies\": [\n \"Journal Entry\"\n ],\n \"doctype\": \"Journal Entry\",\n \"is_query_report\": true,\n \"label\": \"Bank Reconciliation Statement\",\n \"name\": \"Bank Reconciliation Statement\",\n \"type\": \"report\"\n },\n {\n \"icon\": \"fa fa-bar-chart\",\n \"label\": \"Bank Reconciliation\",\n \"name\": \"bank-reconciliation\",\n \"type\": \"page\"\n },\n {\n \"dependencies\": [\n \"Journal Entry\"\n ],\n \"doctype\": \"Journal Entry\",\n \"is_query_report\": true,\n \"label\": \"Bank Clearance Summary\",\n \"name\": \"Bank Clearance Summary\",\n \"type\": \"report\"\n },\n {\n \"label\": \"Bank Guarantee\",\n \"name\": \"Bank Guarantee\",\n \"type\": \"doctype\"\n },\n {\n \"description\": \"Setup cheque dimensions for printing\",\n \"label\": \"Cheque Print Template\",\n \"name\": \"Cheque Print Template\",\n \"type\": \"doctype\"\n }\n]", "title": "Banking and Payments" }, { diff --git a/erpnext/patches.txt b/erpnext/patches.txt index 9ddd7cc24da..ff58f40d370 100644 --- a/erpnext/patches.txt +++ b/erpnext/patches.txt @@ -660,5 +660,6 @@ erpnext.patches.v12_0.set_published_in_hub_tracked_item erpnext.patches.v12_0.set_job_offer_applicant_email erpnext.patches.v12_0.create_irs_1099_field_united_states erpnext.patches.v12_0.move_bank_account_swift_number_to_bank +erpnext.patches.v12_0.rename_bank_reconciliation erpnext.patches.v12_0.rename_bank_reconciliation_fields # 2020-01-22 erpnext.patches.v12_0.set_received_qty_in_material_request_as_per_stock_uom \ No newline at end of file diff --git a/erpnext/patches/v12_0/rename_bank_reconciliation.py b/erpnext/patches/v12_0/rename_bank_reconciliation.py index 556224490c7..eda47a95e03 100644 --- a/erpnext/patches/v12_0/rename_bank_reconciliation.py +++ b/erpnext/patches/v12_0/rename_bank_reconciliation.py @@ -3,12 +3,14 @@ from __future__ import unicode_literals import frappe -from frappe.model.utils.rename_field import rename_field def execute(): - if frappe.db.table_exists("Bank Reconciliation"): - frappe.rename_doc('DocType', 'Bank Reconciliation', 'Bank Clearance', force=True) - frappe.reload_doc('Accounts', 'doctype', 'Bank Clearance') - - frappe.rename_doc('DocType', 'Bank Reconciliation Detail', 'Bank Clearance Detail', force=True) - frappe.reload_doc('Accounts', 'doctype', 'Bank Clearance Detail') + if frappe.db.table_exists("Bank Reconciliation"): + frappe.rename_doc('DocType', 'Bank Reconciliation', 'Bank Clearance', force=True) + frappe.reload_doc('Accounts', 'doctype', 'Bank Clearance') + + frappe.rename_doc('DocType', 'Bank Reconciliation Detail', 'Bank Clearance Detail', force=True) + frappe.reload_doc('Accounts', 'doctype', 'Bank Clearance Detail') + + frappe.delete_doc("DocType", "Bank Reconciliation") + frappe.delete_doc("DocType", "Bank Reconciliation Detail")