From ed79b224ccaf482d6c3562cca99dc484dad73fd2 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 10 May 2021 20:52:02 +0530
Subject: [PATCH 001/106] fix(Asset): Group buttons under 'Action'

---
 erpnext/assets/doctype/asset/asset.js | 32 +++++++++++++--------------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.js b/erpnext/assets/doctype/asset/asset.js
index 6f1bb28f37..657c169fd1 100644
--- a/erpnext/assets/doctype/asset/asset.js
+++ b/erpnext/assets/doctype/asset/asset.js
@@ -78,36 +78,36 @@ frappe.ui.form.on('Asset', {
 		frm.toggle_display("next_depreciation_date", frm.doc.docstatus < 1);
 		frm.events.make_schedules_editable(frm);
 
+		if (frm.doc.purchase_receipt || !frm.doc.is_existing_asset) {
+			frm.add_custom_button("General Ledger", function() {
+				frappe.route_options = {
+					"voucher_no": frm.doc.name,
+					"from_date": frm.doc.available_for_use_date,
+					"to_date": frm.doc.available_for_use_date,
+					"company": frm.doc.company
+				};
+				frappe.set_route("query-report", "General Ledger");
+			});
+		}
+
 		if (frm.doc.docstatus==1) {
 			if (in_list(["Submitted", "Partially Depreciated", "Fully Depreciated"], frm.doc.status)) {
 				frm.add_custom_button("Transfer Asset", function() {
 					erpnext.asset.transfer_asset(frm);
-				});
+				}, __("Actions"));
 
 				frm.add_custom_button("Scrap Asset", function() {
 					erpnext.asset.scrap_asset(frm);
-				});
+				}, __("Actions"));
 
 				frm.add_custom_button("Sell Asset", function() {
 					frm.trigger("make_sales_invoice");
-				});
+				}, __("Actions"));
 
 			} else if (frm.doc.status=='Scrapped') {
 				frm.add_custom_button("Restore Asset", function() {
 					erpnext.asset.restore_asset(frm);
-				});
-			}
-
-			if (frm.doc.purchase_receipt || !frm.doc.is_existing_asset) {
-				frm.add_custom_button("General Ledger", function() {
-					frappe.route_options = {
-						"voucher_no": frm.doc.name,
-						"from_date": frm.doc.available_for_use_date,
-						"to_date": frm.doc.available_for_use_date,
-						"company": frm.doc.company
-					};
-					frappe.set_route("query-report", "General Ledger");
-				});
+				}, __("Actions"));
 			}
 
 			if (frm.doc.maintenance_required && !frm.doc.maintenance_schedule) {

From 322975a03c98bfa72d3372c9cadca0be892855fb Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 10 May 2021 21:09:13 +0530
Subject: [PATCH 002/106] feat(Asset): Add 'Create > Asset Repair' button

---
 erpnext/assets/doctype/asset/asset.js | 24 ++++++++++++++++++++++++
 erpnext/assets/doctype/asset/asset.py | 12 +++++++++++-
 2 files changed, 35 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/asset.js b/erpnext/assets/doctype/asset/asset.js
index 657c169fd1..1f3978c0b0 100644
--- a/erpnext/assets/doctype/asset/asset.js
+++ b/erpnext/assets/doctype/asset/asset.js
@@ -115,6 +115,15 @@ frappe.ui.form.on('Asset', {
 					frm.trigger("create_asset_maintenance");
 				}, __('Create'));
 			}
+			if (frm.doc.docstatus == 1) {
+				frm.add_custom_button(__("Asset Repair"), function() {
+					// frappe.model.open_mapped_doc({
+					// 	method: 'erpnext.stock.doctype.delivery_trip.delivery_trip.make_expense_claim',
+					// 	frm: cur_frm,
+					// });
+					frm.trigger("create_asset_repair");
+				}, __("Create"));
+			}
 			if (frm.doc.status != 'Fully Depreciated') {
 				frm.add_custom_button(__("Asset Value Adjustment"), function() {
 					frm.trigger("create_asset_adjustment");
@@ -304,6 +313,21 @@ frappe.ui.form.on('Asset', {
 		})
 	},
 
+	create_asset_repair: function(frm) {
+		frappe.call({
+			args: {
+				"asset": frm.doc.name,
+				"item_code": frm.doc.item_code,
+				"item_name": frm.doc.item_name
+			},
+			method: "erpnext.assets.doctype.asset.asset.create_asset_repair",
+			callback: function(r) {
+				var doclist = frappe.model.sync(r.message);
+				frappe.set_route("Form", doclist[0].doctype, doclist[0].name);
+			}
+		})
+	},
+
 	create_asset_adjustment: function(frm) {
 		frappe.call({
 			args: {
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 9aff1440d6..962f78fa66 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -637,9 +637,19 @@ def create_asset_maintenance(asset, item_code, item_name, asset_category, compan
 	})
 	return asset_maintenance
 
+@frappe.whitelist()
+def create_asset_repair(asset, item_code, item_name):
+	asset_repair = frappe.new_doc("Asset Repair")
+	asset_repair.update({
+		"asset_name": asset,
+		"item_code": item_code,
+		"item_name": item_name
+	})
+	return asset_repair
+
 @frappe.whitelist()
 def create_asset_adjustment(asset, asset_category, company):
-	asset_maintenance = frappe.new_doc("Asset Value Adjustment")
+	asset_maintenance = frappe.get_doc("Asset Value Adjustment")
 	asset_maintenance.update({
 		"asset": asset,
 		"company": company,

From 3e0e3f0e1b062e38312950bc26b2ca29c86ee47b Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 10 May 2021 23:08:12 +0530
Subject: [PATCH 003/106] fix(Asset Repair): Replace 'Item Name' and 'Item
 Code' with 'Asset Name'

---
 erpnext/assets/doctype/asset/asset.js         |  3 +-
 erpnext/assets/doctype/asset/asset.py         |  7 ++--
 .../doctype/asset_repair/asset_repair.json    | 42 ++++++++-----------
 3 files changed, 21 insertions(+), 31 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.js b/erpnext/assets/doctype/asset/asset.js
index 1f3978c0b0..92a6648c63 100644
--- a/erpnext/assets/doctype/asset/asset.js
+++ b/erpnext/assets/doctype/asset/asset.js
@@ -317,8 +317,7 @@ frappe.ui.form.on('Asset', {
 		frappe.call({
 			args: {
 				"asset": frm.doc.name,
-				"item_code": frm.doc.item_code,
-				"item_name": frm.doc.item_name
+				"asset_name": frm.doc.asset_name
 			},
 			method: "erpnext.assets.doctype.asset.asset.create_asset_repair",
 			callback: function(r) {
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 962f78fa66..b9f413854f 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -638,12 +638,11 @@ def create_asset_maintenance(asset, item_code, item_name, asset_category, compan
 	return asset_maintenance
 
 @frappe.whitelist()
-def create_asset_repair(asset, item_code, item_name):
+def create_asset_repair(asset, asset_name):
 	asset_repair = frappe.new_doc("Asset Repair")
 	asset_repair.update({
-		"asset_name": asset,
-		"item_code": item_code,
-		"item_name": item_name
+		"asset": asset,
+		"asset_name": asset_name
 	})
 	return asset_repair
 
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index d338fc0fb7..853534eb31 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -8,10 +8,9 @@
  "engine": "InnoDB",
  "field_order": [
   "naming_series",
-  "asset_name",
   "column_break_2",
-  "item_code",
-  "item_name",
+  "asset",
+  "asset_name",
   "section_break_5",
   "failure_date",
   "assign_to",
@@ -30,15 +29,6 @@
   "amended_from"
  ],
  "fields": [
-  {
-   "columns": 1,
-   "fieldname": "asset_name",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "label": "Asset",
-   "options": "Asset",
-   "reqd": 1
-  },
   {
    "fieldname": "naming_series",
    "fieldtype": "Select",
@@ -50,18 +40,6 @@
    "fieldname": "column_break_2",
    "fieldtype": "Column Break"
   },
-  {
-   "fetch_from": "asset_name.item_code",
-   "fieldname": "item_code",
-   "fieldtype": "Read Only",
-   "label": "Item Code"
-  },
-  {
-   "fetch_from": "asset_name.item_name",
-   "fieldname": "item_name",
-   "fieldtype": "Read Only",
-   "label": "Item Name"
-  },
   {
    "fieldname": "section_break_5",
    "fieldtype": "Section Break",
@@ -159,12 +137,26 @@
    "options": "Asset Repair",
    "print_hide": 1,
    "read_only": 1
+  },
+  {
+   "columns": 1,
+   "fieldname": "asset",
+   "fieldtype": "Link",
+   "in_list_view": 1,
+   "label": "Asset",
+   "options": "Asset",
+   "reqd": 1
+  },
+  {
+   "fieldname": "asset_name",
+   "fieldtype": "Read Only",
+   "label": "Asset Name"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-01-22 15:08:12.495850",
+ "modified": "2021-05-10 22:48:42.165513",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From bbbd61dab8830a7be0689b7d7b26f4873ee452dc Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 10 May 2021 23:24:34 +0530
Subject: [PATCH 004/106] fix(Asset): Group all buttons under 'Manage'

---
 erpnext/assets/doctype/asset/asset.js | 60 +++++++++++++--------------
 1 file changed, 28 insertions(+), 32 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.js b/erpnext/assets/doctype/asset/asset.js
index 92a6648c63..2a57183a80 100644
--- a/erpnext/assets/doctype/asset/asset.js
+++ b/erpnext/assets/doctype/asset/asset.js
@@ -78,65 +78,61 @@ frappe.ui.form.on('Asset', {
 		frm.toggle_display("next_depreciation_date", frm.doc.docstatus < 1);
 		frm.events.make_schedules_editable(frm);
 
-		if (frm.doc.purchase_receipt || !frm.doc.is_existing_asset) {
-			frm.add_custom_button("General Ledger", function() {
-				frappe.route_options = {
-					"voucher_no": frm.doc.name,
-					"from_date": frm.doc.available_for_use_date,
-					"to_date": frm.doc.available_for_use_date,
-					"company": frm.doc.company
-				};
-				frappe.set_route("query-report", "General Ledger");
-			});
-		}
-
 		if (frm.doc.docstatus==1) {
 			if (in_list(["Submitted", "Partially Depreciated", "Fully Depreciated"], frm.doc.status)) {
 				frm.add_custom_button("Transfer Asset", function() {
 					erpnext.asset.transfer_asset(frm);
-				}, __("Actions"));
+				}, __("Manage"));
 
 				frm.add_custom_button("Scrap Asset", function() {
 					erpnext.asset.scrap_asset(frm);
-				}, __("Actions"));
+				}, __("Manage"));
 
 				frm.add_custom_button("Sell Asset", function() {
 					frm.trigger("make_sales_invoice");
-				}, __("Actions"));
+				}, __("Manage"));
 
 			} else if (frm.doc.status=='Scrapped') {
 				frm.add_custom_button("Restore Asset", function() {
 					erpnext.asset.restore_asset(frm);
-				}, __("Actions"));
+				}, __("Manage"));
 			}
 
 			if (frm.doc.maintenance_required && !frm.doc.maintenance_schedule) {
-				frm.add_custom_button(__("Asset Maintenance"), function() {
+				frm.add_custom_button(__("Maintain Asset"), function() {
 					frm.trigger("create_asset_maintenance");
-				}, __('Create'));
-			}
-			if (frm.doc.docstatus == 1) {
-				frm.add_custom_button(__("Asset Repair"), function() {
-					// frappe.model.open_mapped_doc({
-					// 	method: 'erpnext.stock.doctype.delivery_trip.delivery_trip.make_expense_claim',
-					// 	frm: cur_frm,
-					// });
-					frm.trigger("create_asset_repair");
-				}, __("Create"));
+				}, __("Manage"));
 			}
+		
+			frm.add_custom_button(__("Repair Asset"), function() {
+				frm.trigger("create_asset_repair");
+			}, __("Manage"));
+			
 			if (frm.doc.status != 'Fully Depreciated') {
-				frm.add_custom_button(__("Asset Value Adjustment"), function() {
+				frm.add_custom_button(__("Adjust Asset Value"), function() {
 					frm.trigger("create_asset_adjustment");
-				}, __('Create'));
+				}, __("Manage"));
 			}
 
 			if (!frm.doc.calculate_depreciation) {
-				frm.add_custom_button(__("Depreciation Entry"), function() {
+				frm.add_custom_button(__("Create Depreciation Entry"), function() {
 					frm.trigger("make_journal_entry");
-				}, __('Create'));
+				}, __("Manage"));
+			}
+
+			if (frm.doc.purchase_receipt || !frm.doc.is_existing_asset) {
+				frm.add_custom_button("View General Ledger", function() {
+					frappe.route_options = {
+						"voucher_no": frm.doc.name,
+						"from_date": frm.doc.available_for_use_date,
+						"to_date": frm.doc.available_for_use_date,
+						"company": frm.doc.company
+					};
+					frappe.set_route("query-report", "General Ledger");
+				}, __("Manage"));
 			}
 
-			frm.page.set_inner_btn_group_as_primary(__('Create'));
+			frm.page.set_inner_btn_group_as_primary(__("Manage"));
 			frm.trigger("setup_chart");
 		}
 

From 1c26b27a8929667961d1a587db14bc74f480eb3f Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 00:08:39 +0530
Subject: [PATCH 005/106] fix(Asset Repair): Display 'Completion Date' and
 'Repair Status' only after save

---
 erpnext/assets/doctype/asset_repair/asset_repair.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 4ba2b4474a..f5eeeda5fb 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -2,6 +2,10 @@
 // For license information, please see license.txt
 
 frappe.ui.form.on('Asset Repair', {
+	refresh: function(frm) {
+		frm.toggle_display(['completion_date', 'repair_status'], !(frm.doc.__islocal));
+	},
+
 	repair_status: (frm) => {
 		if (frm.doc.completion_date && frm.doc.repair_status == "Completed") {
 			frappe.call ({

From 30c4a56491f9aa86f491ced75d0d2eb70f302d3b Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 18:43:36 +0530
Subject: [PATCH 006/106] feat(Asset Repair): Change Asset's status according
 to repair_status

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 049b931b5e..884dc19588 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -13,6 +13,10 @@ class AssetRepair(Document):
 		if self.repair_status == "Completed" and not self.completion_date:
 			frappe.throw(_("Please select Completion Date for Completed Repair"))
 
+		if self.repair_status == 'Pending':
+			frappe.db.set_value('Asset', self.asset, 'status', 'Out of Order')
+		else:
+			frappe.db.set_value('Asset', self.asset, 'status', 'Submitted')
 
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):

From deadcd9e97c201bff3d36b237c47c2054a7c2297 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 21:43:56 +0530
Subject: [PATCH 007/106] feat(Asset Repair): Add payable_account field

---
 .../doctype/asset_repair/asset_repair.json    | 22 +++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 853534eb31..4ed9916392 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -7,9 +7,9 @@
  "editable_grid": 1,
  "engine": "InnoDB",
  "field_order": [
+  "asset",
   "naming_series",
   "column_break_2",
-  "asset",
   "asset_name",
   "section_break_5",
   "failure_date",
@@ -18,7 +18,10 @@
   "column_break_6",
   "completion_date",
   "repair_status",
+  "section_break_7",
   "repair_cost",
+  "column_break_8",
+  "payable_account",
   "section_break_9",
   "description",
   "column_break_9",
@@ -151,12 +154,27 @@
    "fieldname": "asset_name",
    "fieldtype": "Read Only",
    "label": "Asset Name"
+  },
+  {
+   "fieldname": "payable_account",
+   "fieldtype": "Link",
+   "label": "Payable Account",
+   "options": "Account"
+  },
+  {
+   "fieldname": "section_break_7",
+   "fieldtype": "Section Break",
+   "label": "Accounting Details"
+  },
+  {
+   "fieldname": "column_break_8",
+   "fieldtype": "Column Break"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-10 22:48:42.165513",
+ "modified": "2021-05-11 05:11:58.330860",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 88ac9b2ec9862fcb5ed901f389baef9a8a97511c Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 21:46:49 +0530
Subject: [PATCH 008/106] fix(Company): Rename 'Fixed Asset Depreciation
 Settings' to 'Fixed Asset Deafults'

---
 erpnext/setup/doctype/company/company.json | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/erpnext/setup/doctype/company/company.json b/erpnext/setup/doctype/company/company.json
index 83cbf475ab..acd05c43e0 100644
--- a/erpnext/setup/doctype/company/company.json
+++ b/erpnext/setup/doctype/company/company.json
@@ -74,7 +74,7 @@
   "stock_received_but_not_billed",
   "service_received_but_not_billed",
   "expenses_included_in_valuation",
-  "fixed_asset_depreciation_settings",
+  "fixed_asset_defaults",
   "accumulated_depreciation_account",
   "depreciation_expense_account",
   "series_for_depreciation_entry",
@@ -520,12 +520,6 @@
    "no_copy": 1,
    "options": "Account"
   },
-  {
-   "collapsible": 1,
-   "fieldname": "fixed_asset_depreciation_settings",
-   "fieldtype": "Section Break",
-   "label": "Fixed Asset Depreciation Settings"
-  },
   {
    "fieldname": "accumulated_depreciation_account",
    "fieldtype": "Link",
@@ -740,6 +734,12 @@
    "fieldtype": "Link",
    "label": "Default Payment Discount Account",
    "options": "Account"
+  },
+  {
+   "collapsible": 1,
+   "fieldname": "fixed_asset_defaults",
+   "fieldtype": "Section Break",
+   "label": "Fixed Asset Defaults"
   }
  ],
  "icon": "fa fa-building",
@@ -747,7 +747,7 @@
  "image_field": "company_logo",
  "is_tree": 1,
  "links": [],
- "modified": "2021-02-16 15:53:37.167589",
+ "modified": "2021-05-11 21:45:22.803065",
  "modified_by": "Administrator",
  "module": "Setup",
  "name": "Company",

From 11594f870ea58ba0984956acd6d4abd9f7183948 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 21:47:58 +0530
Subject: [PATCH 009/106] fix(Company): Add 'Repair and Maintenance Account'
 field

---
 erpnext/setup/doctype/company/company.json | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/erpnext/setup/doctype/company/company.json b/erpnext/setup/doctype/company/company.json
index acd05c43e0..0bc85ea092 100644
--- a/erpnext/setup/doctype/company/company.json
+++ b/erpnext/setup/doctype/company/company.json
@@ -83,6 +83,7 @@
   "disposal_account",
   "depreciation_cost_center",
   "capital_work_in_progress_account",
+  "repair_and_maintenance_account",
   "asset_received_but_not_billed",
   "budget_detail",
   "exception_budget_approver_role",
@@ -740,6 +741,11 @@
    "fieldname": "fixed_asset_defaults",
    "fieldtype": "Section Break",
    "label": "Fixed Asset Defaults"
+  },
+  {
+   "fieldname": "repair_and_maintenance_account",
+   "fieldtype": "Data",
+   "label": "Repair and Maintenance Account"
   }
  ],
  "icon": "fa fa-building",
@@ -747,7 +753,7 @@
  "image_field": "company_logo",
  "is_tree": 1,
  "links": [],
- "modified": "2021-05-11 21:45:22.803065",
+ "modified": "2021-05-11 21:47:04.667950",
  "modified_by": "Administrator",
  "module": "Setup",
  "name": "Company",

From 1b2a5d1489d4e034dc04d0970b253cc1b6b89b3e Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 23:33:07 +0530
Subject: [PATCH 010/106] feat(Asset Repair): Add 'Capitalize Repair Cost'
 checkbox

---
 .../doctype/asset_repair/asset_repair.json    | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 4ed9916392..8762451dd9 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -18,8 +18,9 @@
   "column_break_6",
   "completion_date",
   "repair_status",
-  "section_break_7",
+  "accounting_details",
   "repair_cost",
+  "capitalize_repair_cost",
   "column_break_8",
   "payable_account",
   "section_break_9",
@@ -161,20 +162,26 @@
    "label": "Payable Account",
    "options": "Account"
   },
-  {
-   "fieldname": "section_break_7",
-   "fieldtype": "Section Break",
-   "label": "Accounting Details"
-  },
   {
    "fieldname": "column_break_8",
    "fieldtype": "Column Break"
+  },
+  {
+   "default": "0",
+   "fieldname": "capitalize_repair_cost",
+   "fieldtype": "Check",
+   "label": "Capitalize Repair Cost"
+  },
+  {
+   "fieldname": "accounting_details",
+   "fieldtype": "Section Break",
+   "label": "Accounting Details"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-11 05:11:58.330860",
+ "modified": "2021-05-11 23:25:48.956382",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 194a08e4ec1fbff44da24d653182ce27217368b5 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 23:34:21 +0530
Subject: [PATCH 011/106] fix(Asset Repair): Hide 'Accounting Details' section
 till doc gets saved

---
 .../assets/doctype/asset_repair/asset_repair.js | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index f5eeeda5fb..23bc7b1891 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -2,8 +2,23 @@
 // For license information, please see license.txt
 
 frappe.ui.form.on('Asset Repair', {
+	// setup: function(frm) {
+	// 	frm.add_fetch("company", "repair_and_maintenance_account", "payable_account");
+
+	// 	frm.set_query("payable_account", function() {
+	// 		return {
+	// 			filters: {
+	// 				"report_type": "Balance Sheet",
+	// 				"account_type": "Payable",
+	// 				"company": frm.doc.company,
+	// 				"is_group": 0
+	// 			}
+	// 		};
+	// 	});
+	// },
+
 	refresh: function(frm) {
-		frm.toggle_display(['completion_date', 'repair_status'], !(frm.doc.__islocal));
+		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details'], !(frm.doc.__islocal));
 	},
 
 	repair_status: (frm) => {

From 6d1cf76b77c1264e67085e2d5d08829e0b8afb67 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 11 May 2021 23:56:34 +0530
Subject: [PATCH 012/106] feat(Asset): Add 'Asset Value' field

---
 erpnext/assets/doctype/asset/asset.json | 9 ++++++++-
 erpnext/assets/doctype/asset/asset.py   | 3 +++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/asset.json b/erpnext/assets/doctype/asset/asset.json
index 421b9a6c37..4960c7e709 100644
--- a/erpnext/assets/doctype/asset/asset.json
+++ b/erpnext/assets/doctype/asset/asset.json
@@ -23,6 +23,7 @@
   "asset_name",
   "asset_category",
   "location",
+  "asset_value",
   "custodian",
   "department",
   "disposal_date",
@@ -480,6 +481,12 @@
    "fieldname": "section_break_36",
    "fieldtype": "Section Break",
    "label": "Finance Books"
+  },
+  {
+   "fieldname": "asset_value",
+   "fieldtype": "Currency",
+   "label": "Asset Value",
+   "read_only": 1
   }
  ],
  "idx": 72,
@@ -502,7 +509,7 @@
    "link_fieldname": "asset"
   }
  ],
- "modified": "2021-01-22 12:38:59.091510",
+ "modified": "2021-05-11 23:47:15.831720",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset",
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index b9f413854f..350220b897 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -96,6 +96,9 @@ class Asset(AccountsController):
 			finance_books = get_item_details(self.item_code, self.asset_category)
 			self.set('finance_books', finance_books)
 
+		if not(self.asset_value):
+			self.asset_value = self.gross_purchase_amount
+
 	def validate_asset_values(self):
 		if not self.asset_category:
 			self.asset_category = frappe.get_cached_value("Item", self.item_code, "asset_category")

From 258c2385836a844282aa657b2c21de4550b8752f Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 00:15:18 +0530
Subject: [PATCH 013/106] feat(Asset Repair): Add repair_cost to asset_value

---
 .../doctype/asset_repair/asset_repair.py      | 70 +++++++++++++++++++
 1 file changed, 70 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 884dc19588..b233358e80 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -7,6 +7,9 @@ import frappe
 from frappe import _
 from frappe.utils import time_diff_in_hours
 from frappe.model.document import Document
+from frappe.utils import flt
+from erpnext.accounts.general_ledger import make_gl_entries
+from erpnext.controllers.accounts_controller import AccountsController
 
 class AssetRepair(Document):
 	def validate(self):
@@ -18,6 +21,73 @@ class AssetRepair(Document):
 		else:
 			frappe.db.set_value('Asset', self.asset, 'status', 'Submitted')
 
+	def on_submit(self):
+		self.increase_asset_value()
+
+	def increase_asset_value(self):
+		if self.capitalize_repair_cost:
+			asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value') + self.repair_cost
+			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
+
+	# 	self.make_gl_entries()
+
+	# def on_cancel(self):
+	# 	if self.payable_account:
+	# 		self.make_gl_entries(cancel=True)
+
+	# def make_gl_entries(self, cancel=False):
+	# 	if flt(self.repair_cost) > 0:
+	# 		gl_entries = self.get_gl_entries()
+	# 		make_gl_entries(gl_entries, cancel)
+
+	# def get_gl_entries(self):
+	# 	gl_entry = []
+	# 	company = frappe.db.get_value('Asset', self.asset, 'company')
+	# 	repair_and_maintenance_account = frappe.db.get_value('Company', company, 'repair_and_maintenance_account')
+
+	# 	gl_entry = frappe.get_doc({
+	# 		"doctype": "GL Entry",
+	# 		"account": self.payable_account,
+	# 		"credit": self.repair_cost,
+	# 		"credit_in_account_currency": self.repair_cost,
+	# 		"against": repair_and_maintenance_account,
+	# 		"voucher_type": self.doctype,
+	# 		"voucher_no": self.name
+	# 	})
+	# 	gl_entry.insert()
+	# 	gl_entry = frappe.get_doc({
+	# 		"doctype": "GL Entry",
+	# 		"account": repair_and_maintenance_account,
+	# 		"debit": self.repair_cost,
+	# 		"credit_in_account_currency": self.repair_cost,
+	# 		"against": self.payable_account,
+	# 		"voucher_type": self.doctype,
+	# 		"voucher_no": self.name
+	# 	})
+	# 	gl_entry.insert()
+
+		# gl_entry.append(
+		# 	self.get_gl_dict({
+		# 		"account": self.payable_account,
+		# 		"credit": self.repair_cost,
+		# 		"credit_in_account_currency": self.repair_cost,
+		# 		"against": repair_and_maintenance_account,
+		# 		"against_voucher_type": self.doctype,
+		# 		"against_voucher": self.name
+		# 	})
+		# )
+
+		# gl_entry.append(
+		# 	self.get_gl_dict({
+		# 		"account": repair_and_maintenance_account,
+		# 		"debit": self.repair_cost,
+		# 		"credit_in_account_currency": self.repair_cost,
+		# 		"against": self.payable_account,
+		# 		"against_voucher_type": self.doctype,
+		# 		"against_voucher": self.name
+		# 	})
+		# )
+	
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):
 	downtime = time_diff_in_hours(completion_date, failure_date)

From c62890bcdbabd610497c76f8ddacd60f12222ef1 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 03:27:55 +0530
Subject: [PATCH 014/106] feat: Create 'Stock Item' child table

---
 erpnext/assets/doctype/stock_item/__init__.py |  0
 .../assets/doctype/stock_item/stock_item.json | 55 +++++++++++++++++++
 .../assets/doctype/stock_item/stock_item.py   |  8 +++
 3 files changed, 63 insertions(+)
 create mode 100644 erpnext/assets/doctype/stock_item/__init__.py
 create mode 100644 erpnext/assets/doctype/stock_item/stock_item.json
 create mode 100644 erpnext/assets/doctype/stock_item/stock_item.py

diff --git a/erpnext/assets/doctype/stock_item/__init__.py b/erpnext/assets/doctype/stock_item/__init__.py
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/erpnext/assets/doctype/stock_item/stock_item.json b/erpnext/assets/doctype/stock_item/stock_item.json
new file mode 100644
index 0000000000..b1f05db395
--- /dev/null
+++ b/erpnext/assets/doctype/stock_item/stock_item.json
@@ -0,0 +1,55 @@
+{
+ "actions": [],
+ "creation": "2021-05-12 02:41:54.161024",
+ "doctype": "DocType",
+ "editable_grid": 1,
+ "engine": "InnoDB",
+ "field_order": [
+  "item",
+  "valuation_rate",
+  "consumed_quantity",
+  "total_value"
+ ],
+ "fields": [
+  {
+   "fieldname": "item",
+   "fieldtype": "Link",
+   "in_list_view": 1,
+   "label": "Item",
+   "options": "Item"
+  },
+  {
+   "fetch_from": "item.valuation_rate",
+   "fieldname": "valuation_rate",
+   "fieldtype": "Currency",
+   "in_list_view": 1,
+   "label": "Valuation Rate",
+   "read_only": 1
+  },
+  {
+   "fieldname": "consumed_quantity",
+   "fieldtype": "Data",
+   "in_list_view": 1,
+   "label": "Consumed Quantity"
+  },
+  {
+   "fieldname": "total_value",
+   "fieldtype": "Currency",
+   "in_list_view": 1,
+   "label": "Total Value",
+   "read_only": 1
+  }
+ ],
+ "index_web_pages_for_search": 1,
+ "istable": 1,
+ "links": [],
+ "modified": "2021-05-12 03:19:55.006300",
+ "modified_by": "Administrator",
+ "module": "Assets",
+ "name": "Stock Item",
+ "owner": "Administrator",
+ "permissions": [],
+ "sort_field": "modified",
+ "sort_order": "DESC",
+ "track_changes": 1
+}
\ No newline at end of file
diff --git a/erpnext/assets/doctype/stock_item/stock_item.py b/erpnext/assets/doctype/stock_item/stock_item.py
new file mode 100644
index 0000000000..0e3cc3f8ba
--- /dev/null
+++ b/erpnext/assets/doctype/stock_item/stock_item.py
@@ -0,0 +1,8 @@
+# Copyright (c) 2021, Frappe Technologies Pvt. Ltd. and contributors
+# For license information, please see license.txt
+
+# import frappe
+from frappe.model.document import Document
+
+class StockItem(Document):
+	pass

From d2d31fd16424268140dc75c7611774bbf8cb5dab Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 03:28:45 +0530
Subject: [PATCH 015/106] feat(Asset Repair): Add 'Stock Items' table

---
 .../doctype/asset_repair/asset_repair.json       | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 8762451dd9..e82c5f2926 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -23,11 +23,13 @@
   "capitalize_repair_cost",
   "column_break_8",
   "payable_account",
+  "section_break_17",
+  "stock_items",
   "section_break_9",
   "description",
   "column_break_9",
   "actions_performed",
-  "section_break_17",
+  "section_break_23",
   "downtime",
   "column_break_19",
   "amended_from"
@@ -176,12 +178,22 @@
    "fieldname": "accounting_details",
    "fieldtype": "Section Break",
    "label": "Accounting Details"
+  },
+  {
+   "fieldname": "stock_items",
+   "fieldtype": "Table",
+   "label": "Stock Items",
+   "options": "Stock Item"
+  },
+  {
+   "fieldname": "section_break_23",
+   "fieldtype": "Section Break"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-11 23:25:48.956382",
+ "modified": "2021-05-12 03:23:30.618741",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 0739d4bb1b3c79734f1eec03194b504fb7886888 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 05:15:26 +0530
Subject: [PATCH 016/106] feat(Asset Repair): Add value of consumed stock items
 to asset value

---
 .../assets/doctype/asset_repair/asset_repair.js  | 11 ++++++++++-
 .../doctype/asset_repair/asset_repair.json       | 12 ++++++------
 .../assets/doctype/asset_repair/asset_repair.py  | 16 ++++++++++++++++
 3 files changed, 32 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 23bc7b1891..4641f3aaa2 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -17,8 +17,17 @@ frappe.ui.form.on('Asset Repair', {
 	// 	});
 	// },
 
+	// stock_items_add: function(frm){
+	// 	var table = frm.doc.stock_items;
+	// 	for(var i in table) {
+	// 		if (table[i].valuation_rate == 0) {  
+	// 			frm.set_value(table[i].total_value, (table[i].valuation_rate * table[i].consumed_quantity))
+	// 		}
+	// 	}
+	// },
+
 	refresh: function(frm) {
-		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details'], !(frm.doc.__islocal));
+		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'stock_items_section'], !(frm.doc.__islocal));
 	},
 
 	repair_status: (frm) => {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index e82c5f2926..a8a9f2d195 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -23,7 +23,7 @@
   "capitalize_repair_cost",
   "column_break_8",
   "payable_account",
-  "section_break_17",
+  "stock_items_section",
   "stock_items",
   "section_break_9",
   "description",
@@ -113,10 +113,6 @@
    "fieldtype": "Long Text",
    "label": "Actions performed"
   },
-  {
-   "fieldname": "section_break_17",
-   "fieldtype": "Section Break"
-  },
   {
    "allow_on_submit": 1,
    "fieldname": "downtime",
@@ -188,12 +184,16 @@
   {
    "fieldname": "section_break_23",
    "fieldtype": "Section Break"
+  },
+  {
+   "fieldname": "stock_items_section",
+   "fieldtype": "Section Break"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-12 03:23:30.618741",
+ "modified": "2021-05-12 04:45:38.714776",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index b233358e80..0980ddfa7f 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -16,17 +16,33 @@ class AssetRepair(Document):
 		if self.repair_status == "Completed" and not self.completion_date:
 			frappe.throw(_("Please select Completion Date for Completed Repair"))
 
+		self.update_status()
+		self.set_total_value()
+
+	def set_total_value(self):
+		for item in self.stock_items:
+			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
+
+	def update_status(self):
 		if self.repair_status == 'Pending':
 			frappe.db.set_value('Asset', self.asset, 'status', 'Out of Order')
 		else:
 			frappe.db.set_value('Asset', self.asset, 'status', 'Submitted')
 
 	def on_submit(self):
+		if self.repair_status == "Pending":
+			frappe.throw(_("Please update Repair Status."))
+
 		self.increase_asset_value()
 
 	def increase_asset_value(self):
 		if self.capitalize_repair_cost:
 			asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value') + self.repair_cost
+			for item in self.stock_items:
+				asset_value += item.total_value
+
+			print("*" * 20)
+			print(asset_value)
 			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
 
 	# 	self.make_gl_entries()

From e5ab5d8963efdc6848adf935eea122341aa1295c Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 20:39:53 +0530
Subject: [PATCH 017/106] feat(Asset Repair): Create GL Entries

---
 .../doctype/asset_repair/asset_repair.py      | 95 ++++++++-----------
 erpnext/setup/doctype/company/company.json    |  7 +-
 2 files changed, 41 insertions(+), 61 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 0980ddfa7f..d34970d16d 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -34,6 +34,7 @@ class AssetRepair(Document):
 			frappe.throw(_("Please update Repair Status."))
 
 		self.increase_asset_value()
+		self.make_gl_entries()
 
 	def increase_asset_value(self):
 		if self.capitalize_repair_cost:
@@ -45,64 +46,42 @@ class AssetRepair(Document):
 			print(asset_value)
 			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
 
-	# 	self.make_gl_entries()
-
-	# def on_cancel(self):
-	# 	if self.payable_account:
-	# 		self.make_gl_entries(cancel=True)
-
-	# def make_gl_entries(self, cancel=False):
-	# 	if flt(self.repair_cost) > 0:
-	# 		gl_entries = self.get_gl_entries()
-	# 		make_gl_entries(gl_entries, cancel)
-
-	# def get_gl_entries(self):
-	# 	gl_entry = []
-	# 	company = frappe.db.get_value('Asset', self.asset, 'company')
-	# 	repair_and_maintenance_account = frappe.db.get_value('Company', company, 'repair_and_maintenance_account')
-
-	# 	gl_entry = frappe.get_doc({
-	# 		"doctype": "GL Entry",
-	# 		"account": self.payable_account,
-	# 		"credit": self.repair_cost,
-	# 		"credit_in_account_currency": self.repair_cost,
-	# 		"against": repair_and_maintenance_account,
-	# 		"voucher_type": self.doctype,
-	# 		"voucher_no": self.name
-	# 	})
-	# 	gl_entry.insert()
-	# 	gl_entry = frappe.get_doc({
-	# 		"doctype": "GL Entry",
-	# 		"account": repair_and_maintenance_account,
-	# 		"debit": self.repair_cost,
-	# 		"credit_in_account_currency": self.repair_cost,
-	# 		"against": self.payable_account,
-	# 		"voucher_type": self.doctype,
-	# 		"voucher_no": self.name
-	# 	})
-	# 	gl_entry.insert()
-
-		# gl_entry.append(
-		# 	self.get_gl_dict({
-		# 		"account": self.payable_account,
-		# 		"credit": self.repair_cost,
-		# 		"credit_in_account_currency": self.repair_cost,
-		# 		"against": repair_and_maintenance_account,
-		# 		"against_voucher_type": self.doctype,
-		# 		"against_voucher": self.name
-		# 	})
-		# )
-
-		# gl_entry.append(
-		# 	self.get_gl_dict({
-		# 		"account": repair_and_maintenance_account,
-		# 		"debit": self.repair_cost,
-		# 		"credit_in_account_currency": self.repair_cost,
-		# 		"against": self.payable_account,
-		# 		"against_voucher_type": self.doctype,
-		# 		"against_voucher": self.name
-		# 	})
-		# )
+	def on_cancel(self):
+		if self.payable_account:
+			self.make_gl_entries(cancel=True)
+
+	def make_gl_entries(self, cancel=False):
+		if flt(self.repair_cost) > 0:
+			gl_entries = self.get_gl_entries()
+			make_gl_entries(gl_entries, cancel)
+
+	def get_gl_entries(self):
+		gl_entry = []
+		company = frappe.db.get_value('Asset', self.asset, 'company')
+		repair_and_maintenance_account = frappe.db.get_value('Company', company, 'repair_and_maintenance_account')
+
+		gl_entry = frappe.get_doc({
+			"doctype": "GL Entry",
+			"account": self.payable_account,
+			"credit": self.repair_cost,
+			"credit_in_account_currency": self.repair_cost,
+			"against": repair_and_maintenance_account,
+			"voucher_type": self.doctype,
+			"voucher_no": self.name,
+			"cost_center": "Main - F"
+		})
+		gl_entry.insert()
+		gl_entry = frappe.get_doc({
+			"doctype": "GL Entry",
+			"account": repair_and_maintenance_account,
+			"debit": self.repair_cost,
+			"debit_in_account_currency": self.repair_cost,
+			"against": self.payable_account,
+			"voucher_type": self.doctype,
+			"voucher_no": self.name,
+			"cost_center": "Main - F"
+		})
+		gl_entry.insert()
 	
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):
diff --git a/erpnext/setup/doctype/company/company.json b/erpnext/setup/doctype/company/company.json
index 0bc85ea092..51757f584e 100644
--- a/erpnext/setup/doctype/company/company.json
+++ b/erpnext/setup/doctype/company/company.json
@@ -744,8 +744,9 @@
   },
   {
    "fieldname": "repair_and_maintenance_account",
-   "fieldtype": "Data",
-   "label": "Repair and Maintenance Account"
+   "fieldtype": "Link",
+   "label": "Repair and Maintenance Account",
+   "options": "Account"
   }
  ],
  "icon": "fa fa-building",
@@ -753,7 +754,7 @@
  "image_field": "company_logo",
  "is_tree": 1,
  "links": [],
- "modified": "2021-05-11 21:47:04.667950",
+ "modified": "2021-05-12 16:51:08.187233",
  "modified_by": "Administrator",
  "module": "Setup",
  "name": "Company",

From f5a2ea9cb370727e9e1f5e3da2cf057155872f4d Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 21:06:20 +0530
Subject: [PATCH 018/106] feat(Asset Repair): Add 'Accounting Dimensions'
 section

---
 .../doctype/asset_repair/asset_repair.js      |  2 +-
 .../doctype/asset_repair/asset_repair.json    | 27 ++++++++++++++++++-
 .../doctype/asset_repair/asset_repair.py      |  4 +--
 3 files changed, 28 insertions(+), 5 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 4641f3aaa2..d61cd83e3e 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -27,7 +27,7 @@ frappe.ui.form.on('Asset Repair', {
 	// },
 
 	refresh: function(frm) {
-		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'stock_items_section'], !(frm.doc.__islocal));
+		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'stock_items_section', 'accounting_dimensions_section'], !(frm.doc.__islocal));
 	},
 
 	repair_status: (frm) => {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index a8a9f2d195..18bb2da658 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -18,6 +18,10 @@
   "column_break_6",
   "completion_date",
   "repair_status",
+  "accounting_dimensions_section",
+  "cost_center",
+  "column_break_14",
+  "project",
   "accounting_details",
   "repair_cost",
   "capitalize_repair_cost",
@@ -188,12 +192,33 @@
   {
    "fieldname": "stock_items_section",
    "fieldtype": "Section Break"
+  },
+  {
+   "fieldname": "accounting_dimensions_section",
+   "fieldtype": "Section Break",
+   "label": "Accounting Dimensions"
+  },
+  {
+   "fieldname": "cost_center",
+   "fieldtype": "Link",
+   "label": "Cost Center",
+   "options": "Cost Center"
+  },
+  {
+   "fieldname": "project",
+   "fieldtype": "Link",
+   "label": "Project",
+   "options": "Project"
+  },
+  {
+   "fieldname": "column_break_14",
+   "fieldtype": "Column Break"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-12 04:45:38.714776",
+ "modified": "2021-05-12 21:03:59.032864",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index d34970d16d..e4cbabee36 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -41,9 +41,7 @@ class AssetRepair(Document):
 			asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value') + self.repair_cost
 			for item in self.stock_items:
 				asset_value += item.total_value
-
-			print("*" * 20)
-			print(asset_value)
+				
 			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
 
 	def on_cancel(self):

From d554267b0567ef7e70a47708793fdab7aaaa30a5 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 21:08:55 +0530
Subject: [PATCH 019/106] feat(Asset Repair): Add 'Cost Center' to GL Entries

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index e4cbabee36..343e6c4c9c 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -41,7 +41,7 @@ class AssetRepair(Document):
 			asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value') + self.repair_cost
 			for item in self.stock_items:
 				asset_value += item.total_value
-				
+
 			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
 
 	def on_cancel(self):
@@ -66,7 +66,7 @@ class AssetRepair(Document):
 			"against": repair_and_maintenance_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
-			"cost_center": "Main - F"
+			"cost_center": self.cost_center
 		})
 		gl_entry.insert()
 		gl_entry = frappe.get_doc({
@@ -77,7 +77,7 @@ class AssetRepair(Document):
 			"against": self.payable_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
-			"cost_center": "Main - F"
+			"cost_center": self.cost_center
 		})
 		gl_entry.insert()
 	

From 234b473d9316ed3c536821a5079cb9d293118bf4 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 21:10:16 +0530
Subject: [PATCH 020/106] feat(Asset Repair): Remove 'Assign To'

---
 .../doctype/asset_repair/asset_repair.json     | 18 +-----------------
 1 file changed, 1 insertion(+), 17 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 18bb2da658..cda0edf1c5 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -13,8 +13,6 @@
   "asset_name",
   "section_break_5",
   "failure_date",
-  "assign_to",
-  "assign_to_name",
   "column_break_6",
   "completion_date",
   "repair_status",
@@ -62,20 +60,6 @@
    "label": "Failure Date",
    "reqd": 1
   },
-  {
-   "allow_on_submit": 1,
-   "fieldname": "assign_to",
-   "fieldtype": "Link",
-   "label": "Assign To",
-   "options": "User"
-  },
-  {
-   "allow_on_submit": 1,
-   "fetch_from": "assign_to.full_name",
-   "fieldname": "assign_to_name",
-   "fieldtype": "Read Only",
-   "label": "Assign To Name"
-  },
   {
    "fieldname": "column_break_6",
    "fieldtype": "Column Break"
@@ -218,7 +202,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-12 21:03:59.032864",
+ "modified": "2021-05-12 21:09:27.994356",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 7ad74cf8004092fe1be44f455bdcdafcbaecfa7f Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 12 May 2021 21:22:36 +0530
Subject: [PATCH 021/106] feat(Asset Repair): Add stock_consumption checkbox

---
 .../doctype/asset_repair/asset_repair.js      |  3 ++-
 .../doctype/asset_repair/asset_repair.json    | 20 +++++++++++++------
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index d61cd83e3e..605edccc11 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -27,7 +27,8 @@ frappe.ui.form.on('Asset Repair', {
 	// },
 
 	refresh: function(frm) {
-		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'stock_items_section', 'accounting_dimensions_section'], !(frm.doc.__islocal));
+		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'accounting_dimensions_section'], !(frm.doc.__islocal));
+		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption)
 	},
 
 	repair_status: (frm) => {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index cda0edf1c5..9ab9271fa9 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -23,9 +23,10 @@
   "accounting_details",
   "repair_cost",
   "capitalize_repair_cost",
+  "stock_consumption",
   "column_break_8",
   "payable_account",
-  "stock_items_section",
+  "stock_consumption_details_section",
   "stock_items",
   "section_break_9",
   "description",
@@ -173,10 +174,6 @@
    "fieldname": "section_break_23",
    "fieldtype": "Section Break"
   },
-  {
-   "fieldname": "stock_items_section",
-   "fieldtype": "Section Break"
-  },
   {
    "fieldname": "accounting_dimensions_section",
    "fieldtype": "Section Break",
@@ -197,12 +194,23 @@
   {
    "fieldname": "column_break_14",
    "fieldtype": "Column Break"
+  },
+  {
+   "default": "0",
+   "fieldname": "stock_consumption",
+   "fieldtype": "Check",
+   "label": "Stock Consumed During Repair"
+  },
+  {
+   "fieldname": "stock_consumption_details_section",
+   "fieldtype": "Section Break",
+   "label": "Stock Consumption Details"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-12 21:09:27.994356",
+ "modified": "2021-05-12 21:20:18.276755",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From c2b0102852acc1d826fdf9bc01ca7c9a75dbca6b Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 00:58:07 +0530
Subject: [PATCH 022/106] feat(Asset Repair): Add 'View General Ledger' button

---
 erpnext/assets/doctype/asset_repair/asset_repair.js | 9 +++++++++
 erpnext/assets/doctype/asset_repair/asset_repair.py | 9 +++++----
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 605edccc11..a5fda53429 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -29,6 +29,15 @@ frappe.ui.form.on('Asset Repair', {
 	refresh: function(frm) {
 		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'accounting_dimensions_section'], !(frm.doc.__islocal));
 		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption)
+
+		if (frm.doc.docstatus) {
+			frm.add_custom_button("View General Ledger", function() {
+				frappe.route_options = {
+					"voucher_no": frm.doc.name
+				};
+				frappe.set_route("query-report", "General Ledger");
+			});
+		}
 	},
 
 	repair_status: (frm) => {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 343e6c4c9c..9c3d880eac 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -5,11 +5,10 @@
 from __future__ import unicode_literals
 import frappe
 from frappe import _
-from frappe.utils import time_diff_in_hours
+from frappe.utils import time_diff_in_hours, getdate
 from frappe.model.document import Document
 from frappe.utils import flt
 from erpnext.accounts.general_ledger import make_gl_entries
-from erpnext.controllers.accounts_controller import AccountsController
 
 class AssetRepair(Document):
 	def validate(self):
@@ -66,7 +65,8 @@ class AssetRepair(Document):
 			"against": repair_and_maintenance_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
-			"cost_center": self.cost_center
+			"cost_center": self.cost_center,
+			"posting_date": getdate()
 		})
 		gl_entry.insert()
 		gl_entry = frappe.get_doc({
@@ -77,7 +77,8 @@ class AssetRepair(Document):
 			"against": self.payable_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
-			"cost_center": self.cost_center
+			"cost_center": self.cost_center,
+			"posting_date": getdate()
 		})
 		gl_entry.insert()
 	

From 385ca80ceb939fe76fecb5fcfc95742345729b04 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 02:53:57 +0530
Subject: [PATCH 023/106] feat(Asset Repair): Add total_repair_cost field

---
 .../doctype/asset_repair/asset_repair.js      |  2 +-
 .../doctype/asset_repair/asset_repair.json    |  8 ++-
 .../doctype/asset_repair/asset_repair.py      | 60 +++++++++++++++++--
 3 files changed, 62 insertions(+), 8 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index a5fda53429..9d06caeb98 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -28,7 +28,7 @@ frappe.ui.form.on('Asset Repair', {
 
 	refresh: function(frm) {
 		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'accounting_dimensions_section'], !(frm.doc.__islocal));
-		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption)
+		frm.toggle_display(['stock_consumption_details_section', 'total_repair_cost'], frm.doc.stock_consumption)
 
 		if (frm.doc.docstatus) {
 			frm.add_custom_button("View General Ledger", function() {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 9ab9271fa9..df9ab7d63b 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -26,6 +26,7 @@
   "stock_consumption",
   "column_break_8",
   "payable_account",
+  "total_repair_cost",
   "stock_consumption_details_section",
   "stock_items",
   "section_break_9",
@@ -205,12 +206,17 @@
    "fieldname": "stock_consumption_details_section",
    "fieldtype": "Section Break",
    "label": "Stock Consumption Details"
+  },
+  {
+   "fieldname": "total_repair_cost",
+   "fieldtype": "Currency",
+   "label": "Total Repair Cost"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-12 21:20:18.276755",
+ "modified": "2021-05-13 02:40:57.953076",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 9c3d880eac..2b69b5b110 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -16,18 +16,31 @@ class AssetRepair(Document):
 			frappe.throw(_("Please select Completion Date for Completed Repair"))
 
 		self.update_status()
-		self.set_total_value()
-
-	def set_total_value(self):
-		for item in self.stock_items:
-			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
-
+		self.set_total_value()		# change later
+		self.check_stock_items()
+		self.calculate_total_repair_cost()
+		
 	def update_status(self):
 		if self.repair_status == 'Pending':
 			frappe.db.set_value('Asset', self.asset, 'status', 'Out of Order')
 		else:
 			frappe.db.set_value('Asset', self.asset, 'status', 'Submitted')
 
+	def set_total_value(self):
+		for item in self.stock_items:
+			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
+
+	def check_stock_items(self):
+		if self.stock_consumption:
+			if not self.stock_items:
+				frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
+
+	def calculate_total_repair_cost(self):
+		self.total_repair_cost = self.repair_cost
+		if self.stock_consumption:
+			for item in self.stock_items:
+				self.total_repair_cost += item.total_value
+
 	def on_submit(self):
 		if self.repair_status == "Pending":
 			frappe.throw(_("Please update Repair Status."))
@@ -81,6 +94,41 @@ class AssetRepair(Document):
 			"posting_date": getdate()
 		})
 		gl_entry.insert()
+
+	# 	if self.capitalize_repair_cost:
+	# 		fixed_asset_account = self.get_fixed_asset_account()
+	# 		gl_entry = frappe.get_doc({
+	# 			"doctype": "GL Entry",
+	# 			"account": self.payable_account,
+	# 			"credit": self.total_repair_cost,
+	# 			"credit_in_account_currency": self.repair_cost,
+	# 			"against": repair_and_maintenance_account,
+	# 			"voucher_type": self.doctype,
+	# 			"voucher_no": self.name,
+	# 			"cost_center": self.cost_center,
+	# 			"posting_date": getdate()
+	# 		})
+	# 		gl_entry.insert()
+	# 		gl_entry = frappe.get_doc({
+	# 			"doctype": "GL Entry",
+	# 			"account": fixed_asset_account,
+	# 			"debit": self.total_repair_cost,
+	# 			"debit_in_account_currency": self.repair_cost,
+	# 			"against": self.payable_account,
+	# 			"voucher_type": self.doctype,
+	# 			"voucher_no": self.name,
+	# 			"cost_center": self.cost_center,
+	# 			"posting_date": getdate()
+	# 		})
+	# 		gl_entry.insert()
+
+	# def get_fixed_asset_account(self):
+	# 	asset_category = frappe.get_doc('Asset Category', frappe.db.get_value('Asset', self.asset, 'asset_category'))
+	# 	company = frappe.db.get_value('Asset', self.asset, 'company')
+	# 	for account in asset_category.accounts:
+	# 		if account.company_name == company:
+	# 			return account.fixed_asset_account
+			
 	
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):

From 8d05eca45e8e3dc6ed17d0471d278ef4f9226a39 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 03:26:03 +0530
Subject: [PATCH 024/106] feat(Asset Repair): Create GL Entries to capitalise
 repair cost

---
 .../doctype/asset_repair/asset_repair.py      | 74 +++++++++----------
 1 file changed, 37 insertions(+), 37 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 2b69b5b110..12dda1d4a1 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -73,8 +73,8 @@ class AssetRepair(Document):
 		gl_entry = frappe.get_doc({
 			"doctype": "GL Entry",
 			"account": self.payable_account,
-			"credit": self.repair_cost,
-			"credit_in_account_currency": self.repair_cost,
+			"credit": self.total_repair_cost,
+			"credit_in_account_currency": self.total_repair_cost,
 			"against": repair_and_maintenance_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
@@ -85,8 +85,8 @@ class AssetRepair(Document):
 		gl_entry = frappe.get_doc({
 			"doctype": "GL Entry",
 			"account": repair_and_maintenance_account,
-			"debit": self.repair_cost,
-			"debit_in_account_currency": self.repair_cost,
+			"debit": self.total_repair_cost,
+			"debit_in_account_currency": self.total_repair_cost,
 			"against": self.payable_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
@@ -95,39 +95,39 @@ class AssetRepair(Document):
 		})
 		gl_entry.insert()
 
-	# 	if self.capitalize_repair_cost:
-	# 		fixed_asset_account = self.get_fixed_asset_account()
-	# 		gl_entry = frappe.get_doc({
-	# 			"doctype": "GL Entry",
-	# 			"account": self.payable_account,
-	# 			"credit": self.total_repair_cost,
-	# 			"credit_in_account_currency": self.repair_cost,
-	# 			"against": repair_and_maintenance_account,
-	# 			"voucher_type": self.doctype,
-	# 			"voucher_no": self.name,
-	# 			"cost_center": self.cost_center,
-	# 			"posting_date": getdate()
-	# 		})
-	# 		gl_entry.insert()
-	# 		gl_entry = frappe.get_doc({
-	# 			"doctype": "GL Entry",
-	# 			"account": fixed_asset_account,
-	# 			"debit": self.total_repair_cost,
-	# 			"debit_in_account_currency": self.repair_cost,
-	# 			"against": self.payable_account,
-	# 			"voucher_type": self.doctype,
-	# 			"voucher_no": self.name,
-	# 			"cost_center": self.cost_center,
-	# 			"posting_date": getdate()
-	# 		})
-	# 		gl_entry.insert()
-
-	# def get_fixed_asset_account(self):
-	# 	asset_category = frappe.get_doc('Asset Category', frappe.db.get_value('Asset', self.asset, 'asset_category'))
-	# 	company = frappe.db.get_value('Asset', self.asset, 'company')
-	# 	for account in asset_category.accounts:
-	# 		if account.company_name == company:
-	# 			return account.fixed_asset_account
+		if self.capitalize_repair_cost:
+			fixed_asset_account = self.get_fixed_asset_account()
+			gl_entry = frappe.get_doc({
+				"doctype": "GL Entry",
+				"account": self.payable_account,
+				"credit": self.total_repair_cost,
+				"credit_in_account_currency": self.total_repair_cost,
+				"against": repair_and_maintenance_account,
+				"voucher_type": "Asset",		
+				"voucher_no": self.asset,
+				"cost_center": self.cost_center,
+				"posting_date": getdate()
+			})
+			gl_entry.insert()
+			gl_entry = frappe.get_doc({
+				"doctype": "GL Entry",
+				"account": fixed_asset_account,
+				"debit": self.total_repair_cost,
+				"debit_in_account_currency": self.total_repair_cost,
+				"against": self.payable_account,
+				"voucher_type": "Asset",
+				"voucher_no": self.asset,
+				"cost_center": self.cost_center,
+				"posting_date": getdate()
+			})
+			gl_entry.insert()
+
+	def get_fixed_asset_account(self):
+		asset_category = frappe.get_doc('Asset Category', frappe.db.get_value('Asset', self.asset, 'asset_category'))
+		company = frappe.db.get_value('Asset', self.asset, 'company')
+		for account in asset_category.accounts:
+			if account.company_name == company:
+				return account.fixed_asset_account
 			
 	
 @frappe.whitelist()

From f2fe55ceeb608c34e4a8a0ab73d2511e36603dee Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 03:42:32 +0530
Subject: [PATCH 025/106] feat(Asset Repair): Check for Cost Center and Payable
 Account

---
 .../assets/doctype/asset_repair/asset_repair.py | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 12dda1d4a1..dcad1c6181 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -42,12 +42,25 @@ class AssetRepair(Document):
 				self.total_repair_cost += item.total_value
 
 	def on_submit(self):
-		if self.repair_status == "Pending":
-			frappe.throw(_("Please update Repair Status."))
+		self.check_repair_status()
+		self.check_for_payable_account()
+		self.check_for_cost_center()
 
 		self.increase_asset_value()
 		self.make_gl_entries()
 
+	def check_repair_status(self):
+		if self.repair_status == "Pending":
+			frappe.throw(_("Please update Repair Status."))
+
+	def check_for_payable_account(self):
+		if not self.payable_account:
+			frappe.throw(_("Please enter Payable Account."))
+
+	def check_for_cost_center(self):
+		if not self.cost_center:
+			frappe.throw(_("Please enter Cost Center."))
+
 	def increase_asset_value(self):
 		if self.capitalize_repair_cost:
 			asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value') + self.repair_cost

From c8cb96a38fb811d9a7f1a8355d5071726087be49 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 04:04:52 +0530
Subject: [PATCH 026/106] feat(Asset Repair): Add Warehouse field

---
 .../assets/doctype/asset_repair/asset_repair.json    |  9 ++++++++-
 erpnext/assets/doctype/asset_repair/asset_repair.py  | 12 ++++++------
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index df9ab7d63b..ed7708c8df 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -28,6 +28,7 @@
   "payable_account",
   "total_repair_cost",
   "stock_consumption_details_section",
+  "warehouse",
   "stock_items",
   "section_break_9",
   "description",
@@ -211,12 +212,18 @@
    "fieldname": "total_repair_cost",
    "fieldtype": "Currency",
    "label": "Total Repair Cost"
+  },
+  {
+   "fieldname": "warehouse",
+   "fieldtype": "Link",
+   "label": "Warehouse",
+   "options": "Warehouse"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-13 02:40:57.953076",
+ "modified": "2021-05-13 03:50:39.146322",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index dcad1c6181..ed022fd808 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -17,7 +17,6 @@ class AssetRepair(Document):
 
 		self.update_status()
 		self.set_total_value()		# change later
-		self.check_stock_items()
 		self.calculate_total_repair_cost()
 		
 	def update_status(self):
@@ -30,11 +29,6 @@ class AssetRepair(Document):
 		for item in self.stock_items:
 			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
 
-	def check_stock_items(self):
-		if self.stock_consumption:
-			if not self.stock_items:
-				frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
-
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost
 		if self.stock_consumption:
@@ -43,6 +37,7 @@ class AssetRepair(Document):
 
 	def on_submit(self):
 		self.check_repair_status()
+		self.check_stock_items()
 		self.check_for_payable_account()
 		self.check_for_cost_center()
 
@@ -53,6 +48,11 @@ class AssetRepair(Document):
 		if self.repair_status == "Pending":
 			frappe.throw(_("Please update Repair Status."))
 
+	def check_stock_items(self):
+		if self.stock_consumption:
+			if not self.stock_items:
+				frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
+
 	def check_for_payable_account(self):
 		if not self.payable_account:
 			frappe.throw(_("Please enter Payable Account."))

From 1ac1cedfff947095879d032be1d74d433aca4c2d Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 04:45:21 +0530
Subject: [PATCH 027/106] feat(Asset Repair): Decrease stock quantity if
 consumed during Asset Repair

---
 .../doctype/asset_repair/asset_repair.py      | 24 +++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index ed022fd808..3b7587e61f 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -37,21 +37,25 @@ class AssetRepair(Document):
 
 	def on_submit(self):
 		self.check_repair_status()
-		self.check_stock_items()
+		self.check_for_stock_items_and_warehouse()
 		self.check_for_payable_account()
 		self.check_for_cost_center()
 
 		self.increase_asset_value()
+		if self.stock_consumption:
+			self.decrease_stock_quantity()
 		self.make_gl_entries()
 
 	def check_repair_status(self):
 		if self.repair_status == "Pending":
 			frappe.throw(_("Please update Repair Status."))
 
-	def check_stock_items(self):
+	def check_for_stock_items_and_warehouse(self):
 		if self.stock_consumption:
 			if not self.stock_items:
 				frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
+			if not self.warehouse:
+				frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Repair were taken."))
 
 	def check_for_payable_account(self):
 		if not self.payable_account:
@@ -68,6 +72,22 @@ class AssetRepair(Document):
 				asset_value += item.total_value
 
 			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
+		
+	def decrease_stock_quantity(self):
+		stock_entry = frappe.get_doc({
+			"doctype": "Stock Entry",
+			"stock_entry_type": "Material Issue"
+		})
+
+		for stock_item in self.stock_items:
+			stock_entry.append('items', {
+				"s_warehouse": self.warehouse,
+				"item_code": stock_item.item,
+				"qty": stock_item.consumed_quantity
+			})
+
+		stock_entry.insert()
+		stock_entry.submit()
 
 	def on_cancel(self):
 		if self.payable_account:

From ec72f8956f5c25dda1d19cd7691ac0a8349bae60 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:02:10 +0530
Subject: [PATCH 028/106] fix(Asset Repair): Revert to asset's previous status
 if repair is completed/cancelled

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 3b7587e61f..975e3674a4 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -23,7 +23,8 @@ class AssetRepair(Document):
 		if self.repair_status == 'Pending':
 			frappe.db.set_value('Asset', self.asset, 'status', 'Out of Order')
 		else:
-			frappe.db.set_value('Asset', self.asset, 'status', 'Submitted')
+			asset = frappe.get_doc('Asset', self.asset)
+			asset.set_status()
 
 	def set_total_value(self):
 		for item in self.stock_items:

From 68b78374b50ed09f83bb59103deeede2da416a6e Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:03:56 +0530
Subject: [PATCH 029/106] fix(Asset): Make Manage button white

---
 erpnext/assets/doctype/asset/asset.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/asset.js b/erpnext/assets/doctype/asset/asset.js
index 2a57183a80..1e67ec816b 100644
--- a/erpnext/assets/doctype/asset/asset.js
+++ b/erpnext/assets/doctype/asset/asset.js
@@ -132,7 +132,6 @@ frappe.ui.form.on('Asset', {
 				}, __("Manage"));
 			}
 
-			frm.page.set_inner_btn_group_as_primary(__("Manage"));
 			frm.trigger("setup_chart");
 		}
 

From f7eebf0e782f1e3feb82df0feacadcd8306b938b Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:08:33 +0530
Subject: [PATCH 030/106] feat(Asset Repair): Add 'Increase In Asset Life'
 field

---
 .../assets/doctype/asset_repair/asset_repair.json  | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index ed7708c8df..26f1c6d7cc 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -30,6 +30,8 @@
   "stock_consumption_details_section",
   "warehouse",
   "stock_items",
+  "asset_depreciation_details_section",
+  "increase_in_asset_life",
   "section_break_9",
   "description",
   "column_break_9",
@@ -218,12 +220,22 @@
    "fieldtype": "Link",
    "label": "Warehouse",
    "options": "Warehouse"
+  },
+  {
+   "fieldname": "asset_depreciation_details_section",
+   "fieldtype": "Section Break",
+   "label": "Asset Depreciation Details"
+  },
+  {
+   "fieldname": "increase_in_asset_life",
+   "fieldtype": "Int",
+   "label": "Increase In Asset Life"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-13 03:50:39.146322",
+ "modified": "2021-05-13 05:08:00.676616",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 2874d0853401d3b0570534d1c6dee34885ded1c0 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:22:35 +0530
Subject: [PATCH 031/106] fix(Asset Repair): Rearrange fields

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 26f1c6d7cc..13f4610f8e 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -13,19 +13,19 @@
   "asset_name",
   "section_break_5",
   "failure_date",
+  "repair_status",
   "column_break_6",
   "completion_date",
-  "repair_status",
   "accounting_dimensions_section",
   "cost_center",
   "column_break_14",
   "project",
   "accounting_details",
-  "repair_cost",
+  "payable_account",
   "capitalize_repair_cost",
   "stock_consumption",
   "column_break_8",
-  "payable_account",
+  "repair_cost",
   "total_repair_cost",
   "stock_consumption_details_section",
   "warehouse",
@@ -235,7 +235,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-13 05:08:00.676616",
+ "modified": "2021-05-13 05:11:42.550016",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 568369a5c20488d23cb61974f3d439e4603b756a Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:23:21 +0530
Subject: [PATCH 032/106] feat(Asset Maintenance): Add stock_consumption
 checkbox

---
 .../doctype/asset_maintenance/asset_maintenance.json     | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
index c0c2566fe2..669f1959e5 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
@@ -12,6 +12,7 @@
   "column_break_3",
   "item_code",
   "item_name",
+  "stock_consumption",
   "section_break_6",
   "maintenance_team",
   "column_break_9",
@@ -100,10 +101,16 @@
    "label": "Maintenance Tasks",
    "options": "Asset Maintenance Task",
    "reqd": 1
+  },
+  {
+   "default": "0",
+   "fieldname": "stock_consumption",
+   "fieldtype": "Check",
+   "label": "Stock Consumed During Maintenance"
   }
  ],
  "links": [],
- "modified": "2020-05-28 20:28:32.993823",
+ "modified": "2021-05-13 05:20:45.809270",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Maintenance",

From 04a909bd994667a3eb1cdcd0c5223a214c7ece0d Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:27:15 +0530
Subject: [PATCH 033/106] feat(Asset Maintenance): Add 'Stock Consumption
 Details' section

---
 .../asset_maintenance/asset_maintenance.js    |  3 +++
 .../asset_maintenance/asset_maintenance.json  | 24 +++++++++++++++++--
 2 files changed, 25 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
index 70b8654509..3830d1168c 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
@@ -30,7 +30,10 @@ frappe.ui.form.on('Asset Maintenance', {
 		if(!frm.is_new()) {
 			frm.trigger('make_dashboard');
 		}
+
+		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption)
 	},
+
 	make_dashboard: (frm) => {
 		if(!frm.is_new()) {
 			frappe.call({
diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
index 669f1959e5..da2fd75451 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
@@ -19,7 +19,10 @@
   "maintenance_manager",
   "maintenance_manager_name",
   "section_break_8",
-  "asset_maintenance_tasks"
+  "asset_maintenance_tasks",
+  "stock_consumption_details_section",
+  "warehouse",
+  "stock_items"
  ],
  "fields": [
   {
@@ -107,10 +110,27 @@
    "fieldname": "stock_consumption",
    "fieldtype": "Check",
    "label": "Stock Consumed During Maintenance"
+  },
+  {
+   "fieldname": "stock_consumption_details_section",
+   "fieldtype": "Section Break",
+   "label": "Stock Consumption Details"
+  },
+  {
+   "fieldname": "warehouse",
+   "fieldtype": "Link",
+   "label": "Warehouse",
+   "options": "Warehouse"
+  },
+  {
+   "fieldname": "stock_items",
+   "fieldtype": "Table",
+   "label": "Stock Items",
+   "options": "Stock Item"
   }
  ],
  "links": [],
- "modified": "2021-05-13 05:20:45.809270",
+ "modified": "2021-05-13 05:24:58.480132",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Maintenance",

From d1f521701cf7a60b3e90ac81b1609432af8baca6 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:33:19 +0530
Subject: [PATCH 034/106] feat(Asset Maintenance): Check if Warehouse and Stock
 Items were entered

---
 .../doctype/asset_maintenance/asset_maintenance.py     | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
index a506deec93..8bc7ea1b70 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
@@ -24,6 +24,16 @@ class AssetMaintenance(Document):
 			assign_tasks(self.name, task.assign_to, task.maintenance_task, task.next_due_date)
 		self.sync_maintenance_tasks()
 
+	def on_submit(self):
+		self.check_for_stock_items_and_warehouse()
+
+	def check_for_stock_items_and_warehouse(self):
+		if self.stock_consumption:
+			if not self.stock_items:
+				frappe.throw(_("Please enter Stock Items consumed during Asset Maintenance."))
+			if not self.warehouse:
+				frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Maintenance were taken."))
+
 	def sync_maintenance_tasks(self):
 		tasks_names = []
 		for task in self.get('asset_maintenance_tasks'):

From 70bad470f7116c9ace21373aed205f8e7735c235 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:35:02 +0530
Subject: [PATCH 035/106] feat(Asset Maintenance): Increase Asset value if
 Stock Items were consumed

---
 .../assets/doctype/asset_maintenance/asset_maintenance.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
index 8bc7ea1b70..ecd55e38f2 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
@@ -26,6 +26,7 @@ class AssetMaintenance(Document):
 
 	def on_submit(self):
 		self.check_for_stock_items_and_warehouse()
+		self.increase_asset_value()
 
 	def check_for_stock_items_and_warehouse(self):
 		if self.stock_consumption:
@@ -34,6 +35,13 @@ class AssetMaintenance(Document):
 			if not self.warehouse:
 				frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Maintenance were taken."))
 
+	def increase_asset_value(self):
+		asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value')
+		for item in self.stock_items:
+			asset_value += item.total_value
+
+		frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
+
 	def sync_maintenance_tasks(self):
 		tasks_names = []
 		for task in self.get('asset_maintenance_tasks'):

From a7bbaacfde57076d999f96577828698a41bad49a Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:38:54 +0530
Subject: [PATCH 036/106] fix(Asset Repair): Always add value of consumed Stock
 Items to Asset value

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 975e3674a4..3ac6f26f6c 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -67,12 +67,13 @@ class AssetRepair(Document):
 			frappe.throw(_("Please enter Cost Center."))
 
 	def increase_asset_value(self):
-		if self.capitalize_repair_cost:
-			asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value') + self.repair_cost
-			for item in self.stock_items:
-				asset_value += item.total_value
+		asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value')
+		for item in self.stock_items:
+			asset_value += item.total_value
 
-			frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
+		if self.capitalize_repair_cost:
+			asset_value += self.repair_cost
+		frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
 		
 	def decrease_stock_quantity(self):
 		stock_entry = frappe.get_doc({

From 15294d5543262d69a433861e67bf50042368d8f1 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:40:56 +0530
Subject: [PATCH 037/106] feat(Asset Maintenance): Decrease stock quantity if
 consumed during Asset Maintenance

---
 .../asset_maintenance/asset_maintenance.py    | 31 ++++++++++++++-----
 1 file changed, 24 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
index ecd55e38f2..e3e654c398 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
@@ -19,14 +19,15 @@ class AssetMaintenance(Document):
 			if not task.assign_to and self.docstatus == 0:
 				throw(_("Row #{}: Please asign task to a member.").format(task.idx))
 
+		if self.stock_consumption:
+			self.check_for_stock_items_and_warehouse()
+			self.increase_asset_value()
+			self.decrease_stock_quantity()
+
 	def on_update(self):
 		for task in self.get('asset_maintenance_tasks'):
 			assign_tasks(self.name, task.assign_to, task.maintenance_task, task.next_due_date)
-		self.sync_maintenance_tasks()
-
-	def on_submit(self):
-		self.check_for_stock_items_and_warehouse()
-		self.increase_asset_value()
+		self.sync_maintenance_tasks()		
 
 	def check_for_stock_items_and_warehouse(self):
 		if self.stock_consumption:
@@ -36,11 +37,27 @@ class AssetMaintenance(Document):
 				frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Maintenance were taken."))
 
 	def increase_asset_value(self):
-		asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value')
+		asset_value = frappe.db.get_value('Asset', self.asset_name, 'asset_value')
 		for item in self.stock_items:
 			asset_value += item.total_value
 
-		frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
+		frappe.db.set_value('Asset', self.asset_name, 'asset_value', asset_value)
+
+	def decrease_stock_quantity(self):
+		stock_entry = frappe.get_doc({
+			"doctype": "Stock Entry",
+			"stock_entry_type": "Material Issue"
+		})
+
+		for stock_item in self.stock_items:
+			stock_entry.append('items', {
+				"s_warehouse": self.warehouse,
+				"item_code": stock_item.item,
+				"qty": stock_item.consumed_quantity
+			})
+
+		stock_entry.insert()
+		stock_entry.submit()
 
 	def sync_maintenance_tasks(self):
 		tasks_names = []

From d5d7cacd6712e8636f3102b0ede9796cb1217954 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:52:18 +0530
Subject: [PATCH 038/106] fix(Asset Repair): Improve code

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 3ac6f26f6c..8ff3b796d2 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -38,12 +38,12 @@ class AssetRepair(Document):
 
 	def on_submit(self):
 		self.check_repair_status()
-		self.check_for_stock_items_and_warehouse()
 		self.check_for_payable_account()
 		self.check_for_cost_center()
 
 		self.increase_asset_value()
 		if self.stock_consumption:
+			self.check_for_stock_items_and_warehouse()
 			self.decrease_stock_quantity()
 		self.make_gl_entries()
 
@@ -52,11 +52,10 @@ class AssetRepair(Document):
 			frappe.throw(_("Please update Repair Status."))
 
 	def check_for_stock_items_and_warehouse(self):
-		if self.stock_consumption:
-			if not self.stock_items:
-				frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
-			if not self.warehouse:
-				frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Repair were taken."))
+		if not self.stock_items:
+			frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
+		if not self.warehouse:
+			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Repair were taken."))
 
 	def check_for_payable_account(self):
 		if not self.payable_account:

From 3ab852d4d00041fdd52250646d15b397d22001fa Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 05:56:43 +0530
Subject: [PATCH 039/106] fix(Asset Repair): Fetch 'Asset Name' when 'Asset' is
 selected

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 13f4610f8e..968809c7d5 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -143,6 +143,7 @@
    "reqd": 1
   },
   {
+   "fetch_from": "asset.asset_name",
    "fieldname": "asset_name",
    "fieldtype": "Read Only",
    "label": "Asset Name"
@@ -235,7 +236,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-13 05:11:42.550016",
+ "modified": "2021-05-13 05:55:16.131448",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From a8037c1896f5bd4b0aa78c6f3e5c2409779affcf Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 23:07:19 +0530
Subject: [PATCH 040/106] fix(Asset Repair): Add Purchase Invoice

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 968809c7d5..2c1552f026 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -22,11 +22,12 @@
   "project",
   "accounting_details",
   "payable_account",
-  "capitalize_repair_cost",
+  "purchase_invoice",
   "stock_consumption",
   "column_break_8",
   "repair_cost",
   "total_repair_cost",
+  "capitalize_repair_cost",
   "stock_consumption_details_section",
   "warehouse",
   "stock_items",
@@ -231,12 +232,18 @@
    "fieldname": "increase_in_asset_life",
    "fieldtype": "Int",
    "label": "Increase In Asset Life"
+  },
+  {
+   "fieldname": "purchase_invoice",
+   "fieldtype": "Link",
+   "label": "Purchase Invoice",
+   "options": "Purchase Invoice"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-13 05:55:16.131448",
+ "modified": "2021-05-13 23:01:03.638835",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 66e6f01e40a0f9eaf6242d4eec7e97bc410e830f Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 13 May 2021 23:17:17 +0530
Subject: [PATCH 041/106] fix(Asset Repair): Make Purchase Invoice mandatory if
 capitalize_repair_cost is checked

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 8ff3b796d2..2807c0f5bd 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -41,10 +41,13 @@ class AssetRepair(Document):
 		self.check_for_payable_account()
 		self.check_for_cost_center()
 
-		self.increase_asset_value()
+		if self.stock_consumption or self.capitalize_repair_cost:
+			self.increase_asset_value()
 		if self.stock_consumption:
 			self.check_for_stock_items_and_warehouse()
 			self.decrease_stock_quantity()
+		if self.capitalize_repair_cost:
+			self.check_for_purchase_invoice()
 		self.make_gl_entries()
 
 	def check_repair_status(self):
@@ -90,6 +93,10 @@ class AssetRepair(Document):
 		stock_entry.insert()
 		stock_entry.submit()
 
+	def check_for_purchase_invoice(self):
+		if not self.purchase_invoice:
+			frappe.throw(_("Please link Purchase Invoice."))
+
 	def on_cancel(self):
 		if self.payable_account:
 			self.make_gl_entries(cancel=True)

From 8a41f6354aa93dd1d366768a159e380a31766cdf Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 14 May 2021 02:35:13 +0530
Subject: [PATCH 042/106] fix(Asset Repair): Remove 'Payable Account' field

---
 .../assets/doctype/asset_repair/asset_repair.json | 15 ++++-----------
 .../assets/doctype/asset_repair/asset_repair.py   |  8 +-------
 2 files changed, 5 insertions(+), 18 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 2c1552f026..a2d962cd95 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -21,13 +21,12 @@
   "column_break_14",
   "project",
   "accounting_details",
-  "payable_account",
-  "purchase_invoice",
+  "repair_cost",
+  "capitalize_repair_cost",
   "stock_consumption",
   "column_break_8",
-  "repair_cost",
   "total_repair_cost",
-  "capitalize_repair_cost",
+  "purchase_invoice",
   "stock_consumption_details_section",
   "warehouse",
   "stock_items",
@@ -149,12 +148,6 @@
    "fieldtype": "Read Only",
    "label": "Asset Name"
   },
-  {
-   "fieldname": "payable_account",
-   "fieldtype": "Link",
-   "label": "Payable Account",
-   "options": "Account"
-  },
   {
    "fieldname": "column_break_8",
    "fieldtype": "Column Break"
@@ -243,7 +236,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-13 23:01:03.638835",
+ "modified": "2021-05-14 02:31:57.226273",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 2807c0f5bd..5849c8ddd6 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -38,7 +38,6 @@ class AssetRepair(Document):
 
 	def on_submit(self):
 		self.check_repair_status()
-		self.check_for_payable_account()
 		self.check_for_cost_center()
 
 		if self.stock_consumption or self.capitalize_repair_cost:
@@ -60,10 +59,6 @@ class AssetRepair(Document):
 		if not self.warehouse:
 			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Repair were taken."))
 
-	def check_for_payable_account(self):
-		if not self.payable_account:
-			frappe.throw(_("Please enter Payable Account."))
-
 	def check_for_cost_center(self):
 		if not self.cost_center:
 			frappe.throw(_("Please enter Cost Center."))
@@ -98,8 +93,7 @@ class AssetRepair(Document):
 			frappe.throw(_("Please link Purchase Invoice."))
 
 	def on_cancel(self):
-		if self.payable_account:
-			self.make_gl_entries(cancel=True)
+		self.make_gl_entries(cancel=True)
 
 	def make_gl_entries(self, cancel=False):
 		if flt(self.repair_cost) > 0:

From 6bb920f25e9cf4e36d37fea774d478bb1117b594 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 14 May 2021 03:20:15 +0530
Subject: [PATCH 043/106] fix(Asset Repair): Only create GL entries if repair
 cost is capitalised

---
 .../doctype/asset_repair/asset_repair.py      | 43 +++++--------------
 1 file changed, 10 insertions(+), 33 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 5849c8ddd6..74d3114ea4 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -47,7 +47,7 @@ class AssetRepair(Document):
 			self.decrease_stock_quantity()
 		if self.capitalize_repair_cost:
 			self.check_for_purchase_invoice()
-		self.make_gl_entries()
+			self.make_gl_entries()
 
 	def check_repair_status(self):
 		if self.repair_status == "Pending":
@@ -104,14 +104,16 @@ class AssetRepair(Document):
 		gl_entry = []
 		company = frappe.db.get_value('Asset', self.asset, 'company')
 		repair_and_maintenance_account = frappe.db.get_value('Company', company, 'repair_and_maintenance_account')
+		fixed_asset_account = self.get_fixed_asset_account()
+		expense_account = frappe.get_doc('Purchase Invoice', self.purchase_invoice).items[0].expense_account	
 
 		gl_entry = frappe.get_doc({
 			"doctype": "GL Entry",
-			"account": self.payable_account,
+			"account": expense_account,
 			"credit": self.total_repair_cost,
 			"credit_in_account_currency": self.total_repair_cost,
 			"against": repair_and_maintenance_account,
-			"voucher_type": self.doctype,
+			"voucher_type": self.doctype,		
 			"voucher_no": self.name,
 			"cost_center": self.cost_center,
 			"posting_date": getdate()
@@ -119,44 +121,19 @@ class AssetRepair(Document):
 		gl_entry.insert()
 		gl_entry = frappe.get_doc({
 			"doctype": "GL Entry",
-			"account": repair_and_maintenance_account,
+			"account": fixed_asset_account,
 			"debit": self.total_repair_cost,
 			"debit_in_account_currency": self.total_repair_cost,
-			"against": self.payable_account,
+			"against": expense_account,
 			"voucher_type": self.doctype,
 			"voucher_no": self.name,
 			"cost_center": self.cost_center,
-			"posting_date": getdate()
+			"posting_date": getdate(),
+			"against_voucher_type": "Purchase Invoice",
+			"against_voucher": self.purchase_invoice
 		})
 		gl_entry.insert()
 
-		if self.capitalize_repair_cost:
-			fixed_asset_account = self.get_fixed_asset_account()
-			gl_entry = frappe.get_doc({
-				"doctype": "GL Entry",
-				"account": self.payable_account,
-				"credit": self.total_repair_cost,
-				"credit_in_account_currency": self.total_repair_cost,
-				"against": repair_and_maintenance_account,
-				"voucher_type": "Asset",		
-				"voucher_no": self.asset,
-				"cost_center": self.cost_center,
-				"posting_date": getdate()
-			})
-			gl_entry.insert()
-			gl_entry = frappe.get_doc({
-				"doctype": "GL Entry",
-				"account": fixed_asset_account,
-				"debit": self.total_repair_cost,
-				"debit_in_account_currency": self.total_repair_cost,
-				"against": self.payable_account,
-				"voucher_type": "Asset",
-				"voucher_no": self.asset,
-				"cost_center": self.cost_center,
-				"posting_date": getdate()
-			})
-			gl_entry.insert()
-
 	def get_fixed_asset_account(self):
 		asset_category = frappe.get_doc('Asset Category', frappe.db.get_value('Asset', self.asset, 'asset_category'))
 		company = frappe.db.get_value('Asset', self.asset, 'company')

From 30fdebafa7dcb874b499505dc8e4515ae97fd58d Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 21 May 2021 10:12:28 +0530
Subject: [PATCH 044/106] feat(Asset): Modify depreciation schedule

---
 erpnext/assets/doctype/asset/asset.py         | 19 ++++++++++++++-----
 .../doctype/asset_repair/asset_repair.js      |  3 ++-
 .../doctype/asset_repair/asset_repair.py      | 11 ++++++++++-
 3 files changed, 26 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 350220b897..f837c5e7c1 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -171,15 +171,23 @@ class Asset(AccountsController):
 				d.precision("rate_of_depreciation"))
 
 	def make_depreciation_schedule(self):
-		if 'Manual' not in [d.depreciation_method for d in self.finance_books]:
+		if 'Manual' not in [d.depreciation_method for d in self.finance_books] and not self.schedules:
 			self.schedules = []
 
-		if self.get("schedules") or not self.available_for_use_date:
+		if not self.available_for_use_date:
 			return
 
 		for d in self.get('finance_books'):
 			self.validate_asset_finance_books(d)
 
+			start = 0
+			for n in range (len(self.schedules)):
+				if not self.schedules[n].journal_entry:
+					print("*"*100)
+					del self.schedules[n:]
+					start = n
+					break
+
 			value_after_depreciation = (flt(self.gross_purchase_amount) -
 				flt(self.opening_accumulated_depreciation))
 
@@ -192,9 +200,9 @@ class Asset(AccountsController):
 
 			if has_pro_rata:
 				number_of_pending_depreciations += 1
-
+			
 			skip_row = False
-			for n in range(number_of_pending_depreciations):
+			for n in range(start, number_of_pending_depreciations):
 				# If depreciation is already completed (for double declining balance)
 				if skip_row: continue
 
@@ -350,11 +358,12 @@ class Asset(AccountsController):
 			if d.finance_book_id not in finance_books:
 				accumulated_depreciation = flt(self.opening_accumulated_depreciation)
 				value_after_depreciation = flt(self.get_value_after_depreciation(d.finance_book_id))
-				finance_books.append(d.finance_book_id)
+				finance_books.append(int(d.finance_book_id))
 
 			depreciation_amount = flt(d.depreciation_amount, d.precision("depreciation_amount"))
 			value_after_depreciation -= flt(depreciation_amount)
 
+			# for the last row, if depreciation method = Straight Line
 			if straight_line_idx and i == max(straight_line_idx) - 1:
 				book = self.get('finance_books')[cint(d.finance_book_id) - 1]
 				depreciation_amount += flt(value_after_depreciation -
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 9d06caeb98..3328e664fc 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -28,7 +28,8 @@ frappe.ui.form.on('Asset Repair', {
 
 	refresh: function(frm) {
 		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'accounting_dimensions_section'], !(frm.doc.__islocal));
-		frm.toggle_display(['stock_consumption_details_section', 'total_repair_cost'], frm.doc.stock_consumption)
+		frm.toggle_display(['stock_consumption_details_section', 'total_repair_cost'], frm.doc.stock_consumption);
+		frm.toggle_display('asset_depreciation_details_section', frm.doc.capitalize_repair_cost);
 
 		if (frm.doc.docstatus) {
 			frm.add_custom_button("View General Ledger", function() {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 74d3114ea4..95abbd3e94 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -48,6 +48,7 @@ class AssetRepair(Document):
 		if self.capitalize_repair_cost:
 			self.check_for_purchase_invoice()
 			self.make_gl_entries()
+			self.modify_depreciation_schedule()
 
 	def check_repair_status(self):
 		if self.repair_status == "Pending":
@@ -140,8 +141,16 @@ class AssetRepair(Document):
 		for account in asset_category.accounts:
 			if account.company_name == company:
 				return account.fixed_asset_account
+
+	def modify_depreciation_schedule(self):
+		if self.increase_in_asset_life:
+			asset = frappe.get_doc('Asset', self.asset)
+			asset.flags.ignore_validate_update_after_submit = True
+			for row in asset.finance_books:
+				row.total_number_of_depreciations += self.increase_in_asset_life/row.frequency_of_depreciation
+			asset.prepare_depreciation_data()
+			asset.save()
 			
-	
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):
 	downtime = time_diff_in_hours(completion_date, failure_date)

From 4277877883c120789f3867d20b88c7eabd7726f9 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 21 May 2021 10:40:16 +0530
Subject: [PATCH 045/106] feat(Asset Repair): Change visibilty of sections

---
 .../doctype/asset_repair/asset_repair.js      | 28 +------------------
 .../doctype/asset_repair/asset_repair.json    |  7 +++--
 2 files changed, 6 insertions(+), 29 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 3328e664fc..7633a595a2 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -2,35 +2,9 @@
 // For license information, please see license.txt
 
 frappe.ui.form.on('Asset Repair', {
-	// setup: function(frm) {
-	// 	frm.add_fetch("company", "repair_and_maintenance_account", "payable_account");
-
-	// 	frm.set_query("payable_account", function() {
-	// 		return {
-	// 			filters: {
-	// 				"report_type": "Balance Sheet",
-	// 				"account_type": "Payable",
-	// 				"company": frm.doc.company,
-	// 				"is_group": 0
-	// 			}
-	// 		};
-	// 	});
-	// },
-
-	// stock_items_add: function(frm){
-	// 	var table = frm.doc.stock_items;
-	// 	for(var i in table) {
-	// 		if (table[i].valuation_rate == 0) {  
-	// 			frm.set_value(table[i].total_value, (table[i].valuation_rate * table[i].consumed_quantity))
-	// 		}
-	// 	}
-	// },
-
 	refresh: function(frm) {
 		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'accounting_dimensions_section'], !(frm.doc.__islocal));
-		frm.toggle_display(['stock_consumption_details_section', 'total_repair_cost'], frm.doc.stock_consumption);
-		frm.toggle_display('asset_depreciation_details_section', frm.doc.capitalize_repair_cost);
-
+	
 		if (frm.doc.docstatus) {
 			frm.add_custom_button("View General Ledger", function() {
 				frappe.route_options = {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index a2d962cd95..522f2874d9 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -201,11 +201,13 @@
    "label": "Stock Consumed During Repair"
   },
   {
+   "depends_on": "stock_consumption",
    "fieldname": "stock_consumption_details_section",
    "fieldtype": "Section Break",
    "label": "Stock Consumption Details"
   },
   {
+   "depends_on": "stock_consumption",
    "fieldname": "total_repair_cost",
    "fieldtype": "Currency",
    "label": "Total Repair Cost"
@@ -217,6 +219,7 @@
    "options": "Warehouse"
   },
   {
+   "depends_on": "capitalize_repair_cost",
    "fieldname": "asset_depreciation_details_section",
    "fieldtype": "Section Break",
    "label": "Asset Depreciation Details"
@@ -224,7 +227,7 @@
   {
    "fieldname": "increase_in_asset_life",
    "fieldtype": "Int",
-   "label": "Increase In Asset Life"
+   "label": "Increase In Asset Life(Months)"
   },
   {
    "fieldname": "purchase_invoice",
@@ -236,7 +239,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-14 02:31:57.226273",
+ "modified": "2021-05-21 10:37:35.002238",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 4b543de3293d0dfa1a2cbb51501ff7a863b64805 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 21 May 2021 23:40:36 +0530
Subject: [PATCH 046/106] feat(Asset Repair): Modify depreciation schedule when
 increase_in_asset_life is not a multiple of frequency_of_depreciation

---
 erpnext/assets/doctype/asset/asset.json       | 16 ++++++++++++-
 erpnext/assets/doctype/asset/asset.py         |  7 +++---
 .../doctype/asset_repair/asset_repair.py      | 24 +++++++++++++++++--
 3 files changed, 41 insertions(+), 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.json b/erpnext/assets/doctype/asset/asset.json
index 4960c7e709..8a0e3ad2a6 100644
--- a/erpnext/assets/doctype/asset/asset.json
+++ b/erpnext/assets/doctype/asset/asset.json
@@ -54,6 +54,8 @@
   "next_depreciation_date",
   "section_break_14",
   "schedules",
+  "to_date",
+  "edit_dates",
   "insurance_details",
   "policy_number",
   "insurer",
@@ -487,6 +489,18 @@
    "fieldtype": "Currency",
    "label": "Asset Value",
    "read_only": 1
+  },
+  {
+   "fieldname": "to_date",
+   "fieldtype": "Date",
+   "hidden": 1,
+   "label": "To Date"
+  },
+  {
+   "fieldname": "edit_dates",
+   "fieldtype": "Data",
+   "hidden": 1,
+   "label": "Edit Dates"
   }
  ],
  "idx": 72,
@@ -509,7 +523,7 @@
    "link_fieldname": "asset"
   }
  ],
- "modified": "2021-05-11 23:47:15.831720",
+ "modified": "2021-05-21 12:05:29.424083",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset",
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index f837c5e7c1..2d012d672e 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -228,11 +228,12 @@ class Asset(AccountsController):
 
 				# For last row
 				elif has_pro_rata and n == cint(number_of_pending_depreciations) - 1:
-					to_date = add_months(self.available_for_use_date,
-						n * cint(d.frequency_of_depreciation))
+					if not self.edit_dates:
+						self.to_date = add_months(self.available_for_use_date,
+							n * cint(d.frequency_of_depreciation))
 
 					depreciation_amount, days, months = get_pro_rata_amt(d,
-						depreciation_amount, schedule_date, to_date)
+						depreciation_amount, schedule_date, self.to_date)
 
 					monthly_schedule_date = add_months(schedule_date, 1)
 
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 95abbd3e94..8fd019febd 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -5,9 +5,8 @@
 from __future__ import unicode_literals
 import frappe
 from frappe import _
-from frappe.utils import time_diff_in_hours, getdate
+from frappe.utils import time_diff_in_hours, getdate, add_days, date_diff, add_months, flt, cint
 from frappe.model.document import Document
-from frappe.utils import flt
 from erpnext.accounts.general_ledger import make_gl_entries
 
 class AssetRepair(Document):
@@ -148,8 +147,29 @@ class AssetRepair(Document):
 			asset.flags.ignore_validate_update_after_submit = True
 			for row in asset.finance_books:
 				row.total_number_of_depreciations += self.increase_in_asset_life/row.frequency_of_depreciation
+
+				asset.edit_dates = ""
+				extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
+				if extra_months != 0:
+					self.calculate_last_schedule_date(asset, row, extra_months)
+					# fix depreciation amount
+
 			asset.prepare_depreciation_data()
 			asset.save()
+
+	# to help modify depreciation schedule when increase_in_asset_life is not a multiple of frequency_of_depreciation
+	def calculate_last_schedule_date(self, asset, row, extra_months):
+		asset.edit_dates = "Don't Edit"
+		number_of_pending_depreciations = cint(row.total_number_of_depreciations) - \
+			cint(asset.number_of_depreciations_booked)
+		last_schedule_date = asset.schedules[len(asset.schedules)-1].schedule_date
+		asset.to_date = add_months(last_schedule_date, extra_months)
+		schedule_date = add_months(row.depreciation_start_date,
+			number_of_pending_depreciations * cint(row.frequency_of_depreciation))
+
+		if asset.to_date > schedule_date:
+			row.total_number_of_depreciations += 1
+
 			
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):

From 28ca383534fad7734a82fe69502c1f6e1fcac72d Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 21 May 2021 23:55:51 +0530
Subject: [PATCH 047/106] feat(Asset): Edit value_after_depreciation

---
 erpnext/assets/doctype/asset/asset.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 2d012d672e..fb29ea0569 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -183,13 +183,12 @@ class Asset(AccountsController):
 			start = 0
 			for n in range (len(self.schedules)):
 				if not self.schedules[n].journal_entry:
-					print("*"*100)
 					del self.schedules[n:]
 					start = n
 					break
 
-			value_after_depreciation = (flt(self.gross_purchase_amount) -
-				flt(self.opening_accumulated_depreciation))
+			value_after_depreciation = (flt(self.asset_value) -
+				flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
 
 			d.value_after_depreciation = value_after_depreciation
 

From 1d7dda2664d8966601c9520efd199a99e2b35497 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 22 May 2021 07:18:31 +0530
Subject: [PATCH 048/106] fix(Asset): Fix depreciation_amount calculation

---
 erpnext/assets/doctype/asset/asset.py         | 37 +++++++++----------
 .../doctype/asset_repair/asset_repair.py      |  1 -
 2 files changed, 17 insertions(+), 21 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index fb29ea0569..e61887a1f1 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -205,8 +205,7 @@ class Asset(AccountsController):
 				# If depreciation is already completed (for double declining balance)
 				if skip_row: continue
 
-				depreciation_amount = self.get_depreciation_amount(value_after_depreciation,
-					d.total_number_of_depreciations, d)
+				depreciation_amount = get_depreciation_amount(self, value_after_depreciation, d)
 
 				if not has_pro_rata or n < cint(number_of_pending_depreciations) - 1:
 					schedule_date = add_months(d.depreciation_start_date,
@@ -377,24 +376,6 @@ class Asset(AccountsController):
 	def get_value_after_depreciation(self, idx):
 		return flt(self.get('finance_books')[cint(idx)-1].value_after_depreciation)
 
-	def get_depreciation_amount(self, depreciable_value, total_number_of_depreciations, row):
-		precision = self.precision("gross_purchase_amount")
-
-		if row.depreciation_method in ("Straight Line", "Manual"):
-			depreciation_left = (cint(row.total_number_of_depreciations) - cint(self.number_of_depreciations_booked))
-
-			if not depreciation_left:
-				frappe.msgprint(_("All the depreciations has been booked"))
-				depreciation_amount = flt(row.expected_value_after_useful_life)
-				return depreciation_amount
-
-			depreciation_amount = (flt(row.value_after_depreciation) -
-				flt(row.expected_value_after_useful_life)) / depreciation_left
-		else:
-			depreciation_amount = flt(depreciable_value * (flt(row.rate_of_depreciation) / 100), precision)
-
-		return depreciation_amount
-
 	def validate_expected_value_after_useful_life(self):
 		for row in self.get('finance_books'):
 			accumulated_depreciation_after_full_schedule = [d.accumulated_depreciation_amount
@@ -791,3 +772,19 @@ def get_total_days(date, frequency):
 		cint(frequency) * -1)
 
 	return date_diff(date, period_start_date)
+
+@erpnext.allow_regional
+def get_depreciation_amount(asset, depreciable_value, row):
+	depreciation_left = flt(row.total_number_of_depreciations) - flt(asset.number_of_depreciations_booked)
+
+	if row.depreciation_method in ("Straight Line", "Manual"):
+		if not asset.to_date:
+			depreciation_amount = (flt(row.value_after_depreciation) -
+				flt(row.expected_value_after_useful_life)) / depreciation_left
+		else:
+			depreciation_amount = (flt(row.value_after_depreciation) -
+				flt(row.expected_value_after_useful_life)) / (date_diff(asset.to_date, asset.available_for_use_date) / 365)
+	else:
+		depreciation_amount = flt(depreciable_value * (flt(row.rate_of_depreciation) / 100))
+
+	return depreciation_amount
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 8fd019febd..9973afd80a 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -152,7 +152,6 @@ class AssetRepair(Document):
 				extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
 				if extra_months != 0:
 					self.calculate_last_schedule_date(asset, row, extra_months)
-					# fix depreciation amount
 
 			asset.prepare_depreciation_data()
 			asset.save()

From 03a6977a01490c35d9ad51bb7a94dd8fd08d2b2e Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 22 May 2021 23:23:29 +0530
Subject: [PATCH 049/106] fix: Sider issues

---
 erpnext/assets/doctype/asset/asset.js                         | 2 +-
 erpnext/assets/doctype/asset/asset.py                         | 2 +-
 erpnext/assets/doctype/asset_maintenance/asset_maintenance.js | 2 +-
 erpnext/assets/doctype/asset_repair/asset_repair.py           | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.js b/erpnext/assets/doctype/asset/asset.js
index 1e67ec816b..922cc4a7b2 100644
--- a/erpnext/assets/doctype/asset/asset.js
+++ b/erpnext/assets/doctype/asset/asset.js
@@ -319,7 +319,7 @@ frappe.ui.form.on('Asset', {
 				var doclist = frappe.model.sync(r.message);
 				frappe.set_route("Form", doclist[0].doctype, doclist[0].name);
 			}
-		})
+		});
 	},
 
 	create_asset_adjustment: function(frm) {
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index e61887a1f1..1495a5fa5f 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -181,7 +181,7 @@ class Asset(AccountsController):
 			self.validate_asset_finance_books(d)
 
 			start = 0
-			for n in range (len(self.schedules)):
+			for n in range(len(self.schedules)):
 				if not self.schedules[n].journal_entry:
 					del self.schedules[n:]
 					start = n
diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
index 3830d1168c..19393b7e9d 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
@@ -31,7 +31,7 @@ frappe.ui.form.on('Asset Maintenance', {
 			frm.trigger('make_dashboard');
 		}
 
-		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption)
+		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption);
 	},
 
 	make_dashboard: (frm) => {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 9973afd80a..3e81ba55b0 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -5,7 +5,7 @@
 from __future__ import unicode_literals
 import frappe
 from frappe import _
-from frappe.utils import time_diff_in_hours, getdate, add_days, date_diff, add_months, flt, cint
+from frappe.utils import time_diff_in_hours, getdate, add_months, flt, cint
 from frappe.model.document import Document
 from erpnext.accounts.general_ledger import make_gl_entries
 

From 960840dc3ad9c4f12268dc06f3f00fcc7fb4ace8 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 01:53:13 +0530
Subject: [PATCH 050/106] fix(Asset Repair): Set completion_date

---
 .../assets/doctype/asset_repair/asset_repair.py    | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 3e81ba55b0..ad0792e3ea 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -5,18 +5,19 @@
 from __future__ import unicode_literals
 import frappe
 from frappe import _
-from frappe.utils import time_diff_in_hours, getdate, add_months, flt, cint
+from frappe.utils import time_diff_in_hours, getdate, nowdate, add_months, flt, cint
 from frappe.model.document import Document
 from erpnext.accounts.general_ledger import make_gl_entries
 
 class AssetRepair(Document):
 	def validate(self):
 		if self.repair_status == "Completed" and not self.completion_date:
-			frappe.throw(_("Please select Completion Date for Completed Repair"))
+			self.completion_date = nowdate()
 
 		self.update_status()
-		self.set_total_value()		# change later
-		self.calculate_total_repair_cost()
+		if self.stock_consumption:
+			self.set_total_value()		# change later
+			self.calculate_total_repair_cost()
 		
 	def update_status(self):
 		if self.repair_status == 'Pending':
@@ -31,9 +32,8 @@ class AssetRepair(Document):
 
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost
-		if self.stock_consumption:
-			for item in self.stock_items:
-				self.total_repair_cost += item.total_value
+		for item in self.stock_items:
+			self.total_repair_cost += item.total_value
 
 	def on_submit(self):
 		self.check_repair_status()

From 2ceeb8138d03076e7192b9a97bd14741976ce5fc Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 01:54:44 +0530
Subject: [PATCH 051/106] fix(Asset Repair): Set company when creating Stock
 Entry

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index ad0792e3ea..c39b20c50d 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -75,7 +75,8 @@ class AssetRepair(Document):
 	def decrease_stock_quantity(self):
 		stock_entry = frappe.get_doc({
 			"doctype": "Stock Entry",
-			"stock_entry_type": "Material Issue"
+			"stock_entry_type": "Material Issue",
+			"company": frappe.get_value('Asset', self.asset, "company")
 		})
 
 		for stock_item in self.stock_items:

From 9fb47795c4b3d03ee36273959b4b844dacb35bb7 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 04:55:49 +0530
Subject: [PATCH 052/106] fix(Asset Repair): Only modify depreciation schedule
 if calculate_depreciation is checked

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index c39b20c50d..fa918b7efd 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -47,7 +47,8 @@ class AssetRepair(Document):
 		if self.capitalize_repair_cost:
 			self.check_for_purchase_invoice()
 			self.make_gl_entries()
-			self.modify_depreciation_schedule()
+			if frappe.db.get_value('Asset', self.asset, 'calculate_depreciation'):
+				self.modify_depreciation_schedule()
 
 	def check_repair_status(self):
 		if self.repair_status == "Pending":

From 034f7bde336a8bfa69787d382634c7848511584d Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 21:07:07 +0530
Subject: [PATCH 053/106] fix(Asset Repair): Remove unnecessary condition

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index fa918b7efd..80c9f09e14 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -15,9 +15,8 @@ class AssetRepair(Document):
 			self.completion_date = nowdate()
 
 		self.update_status()
-		if self.stock_consumption:
-			self.set_total_value()		# change later
-			self.calculate_total_repair_cost()
+		self.set_total_value()		# change later
+		self.calculate_total_repair_cost()
 		
 	def update_status(self):
 		if self.repair_status == 'Pending':
@@ -72,7 +71,7 @@ class AssetRepair(Document):
 		if self.capitalize_repair_cost:
 			asset_value += self.repair_cost
 		frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
-		
+
 	def decrease_stock_quantity(self):
 		stock_entry = frappe.get_doc({
 			"doctype": "Stock Entry",

From aa9bbe51bd0205918f75cc035efb9c07b055ca80 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 21:28:58 +0530
Subject: [PATCH 054/106] fix(Asset Repair): Add Company in GL Entries

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 80c9f09e14..0b4612e754 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -117,7 +117,8 @@ class AssetRepair(Document):
 			"voucher_type": self.doctype,		
 			"voucher_no": self.name,
 			"cost_center": self.cost_center,
-			"posting_date": getdate()
+			"posting_date": getdate(),
+			"company": company
 		})
 		gl_entry.insert()
 		gl_entry = frappe.get_doc({
@@ -131,7 +132,8 @@ class AssetRepair(Document):
 			"cost_center": self.cost_center,
 			"posting_date": getdate(),
 			"against_voucher_type": "Purchase Invoice",
-			"against_voucher": self.purchase_invoice
+			"against_voucher": self.purchase_invoice,
+			"company": company
 		})
 		gl_entry.insert()
 

From b55649f2ecf1b4b0f4121fe0d7ad986517d021b8 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 22:28:30 +0530
Subject: [PATCH 055/106] fix(Asset): Add depreciation schedule details in
 create_asset()

---
 erpnext/assets/doctype/asset/test_asset.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/test_asset.py b/erpnext/assets/doctype/asset/test_asset.py
index a0d76031fc..568a41039c 100644
--- a/erpnext/assets/doctype/asset/test_asset.py
+++ b/erpnext/assets/doctype/asset/test_asset.py
@@ -660,7 +660,7 @@ def create_asset(**args):
 		"item_code": args.item_code or "Macbook Pro",
 		"company": args.company or"_Test Company",
 		"purchase_date": "2015-01-01",
-		"calculate_depreciation": 0,
+		"calculate_depreciation": args.calculate_depreciation or 0,
 		"gross_purchase_amount": 100000,
 		"purchase_receipt_amount": 100000,
 		"expected_value_after_useful_life": 10000,
@@ -671,6 +671,13 @@ def create_asset(**args):
 		"is_existing_asset": args.is_existing_asset or 0
 	})
 
+	if asset.calculate_depreciation:
+		asset.append("finance_books", {
+			"depreciation_method": "Straight Line",
+			"frequency_of_depreciation": 12,
+			"total_number_of_depreciations": 5
+		})
+
 	try:
 		asset.save()
 	except frappe.DuplicateEntryError:

From 12d9e3b1e6e04dbc2eae8f2f7a588fbafbdfe59b Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 3 Jun 2021 22:28:57 +0530
Subject: [PATCH 056/106] fix(Asset Repair): Add tests

---
 .../doctype/asset_repair/asset_repair.py      |   1 -
 .../doctype/asset_repair/test_asset_repair.py | 166 +++++++++++++++++-
 2 files changed, 164 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 0b4612e754..8724d0a125 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -172,7 +172,6 @@ class AssetRepair(Document):
 		if asset.to_date > schedule_date:
 			row.total_number_of_depreciations += 1
 
-			
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):
 	downtime = time_diff_in_hours(completion_date, failure_date)
diff --git a/erpnext/assets/doctype/asset_repair/test_asset_repair.py b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
index 3d325a9683..9c9dd44971 100644
--- a/erpnext/assets/doctype/asset_repair/test_asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
@@ -2,8 +2,170 @@
 # Copyright (c) 2017, Frappe Technologies Pvt. Ltd. and Contributors
 # See license.txt
 from __future__ import unicode_literals
-
+import frappe
+from frappe.utils import nowdate, flt
 import unittest
+from erpnext.assets.doctype.asset.test_asset import create_asset_data, create_asset, set_depreciation_settings_in_company
 
 class TestAssetRepair(unittest.TestCase):
-	pass
+	def setUp(self):
+		set_depreciation_settings_in_company()
+		create_asset_data()
+		frappe.db.sql("delete from `tabTax Rule`")
+
+	def test_completion_date(self):
+		asset_repair = create_asset_repair()
+		asset_repair.repair_status = "Completed"
+		asset_repair.save()
+		self.assertTrue(asset_repair.completion_date)
+
+	def test_update_status(self):
+		asset = create_asset()
+		initial_status = asset.status
+		asset_repair = create_asset_repair(asset = asset)
+
+		if asset_repair.repair_status == "Pending":
+			asset.reload()
+			self.assertEqual(asset.status, "Out of Order")
+
+		asset_repair.repair_status = "Completed"
+		asset_repair.save()
+		asset_status = frappe.db.get_value("Asset", asset_repair.asset, "status")
+		self.assertEqual(asset_status, initial_status)
+
+	def test_stock_item_total_value(self):
+		asset_repair = create_asset_repair(stock_consumption = 1)
+
+		for item in asset_repair.stock_items:
+			total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
+			self.assertEqual(item.total_value, total_value)
+
+	def test_total_repair_cost(self):
+		asset_repair = create_asset_repair(stock_consumption = 1)
+
+		total_repair_cost = asset_repair.repair_cost
+		self.assertEqual(total_repair_cost, asset_repair.repair_cost)
+		for item in asset_repair.stock_items:
+			total_repair_cost += item.total_value
+			
+		self.assertEqual(total_repair_cost, asset_repair.total_repair_cost)
+
+	def test_repair_status_after_submit(self):
+		asset_repair = create_asset_repair(submit = 1)
+		self.assertNotEqual(asset_repair.repair_status, "Pending")
+
+	def test_stock_items(self):
+		asset_repair = create_asset_repair(stock_consumption = 1)
+		self.assertTrue(asset_repair.stock_consumption)
+		self.assertTrue(asset_repair.stock_items)
+
+	def test_warehouse(self):
+		asset_repair = create_asset_repair(stock_consumption = 1)
+		self.assertTrue(asset_repair.stock_consumption)
+		self.assertTrue(asset_repair.warehouse)
+
+	def test_decrease_stock_quantity(self):
+		asset_repair = create_asset_repair(stock_consumption = 1, submit = 1)
+		stock_entry = frappe.get_last_doc('Stock Entry')
+
+		self.assertEqual(stock_entry.stock_entry_type, "Material Issue")
+		self.assertEqual(stock_entry.items[0].s_warehouse, asset_repair.warehouse)
+		self.assertEqual(stock_entry.items[0].item_code, asset_repair.stock_items[0].item)
+		self.assertEqual(stock_entry.items[0].qty, asset_repair.stock_items[0].consumed_quantity)
+
+	def test_increase_in_asset_value_due_to_stock_consumption(self):
+		asset = create_asset()
+		initial_asset_value = asset.asset_value
+		asset_repair = create_asset_repair(asset= asset, stock_consumption = 1, submit = 1)
+		asset.reload()
+
+		increase_in_asset_value = asset.asset_value - initial_asset_value
+		self.assertEqual(asset_repair.stock_items[0].total_value, increase_in_asset_value)
+
+	def test_increase_in_asset_value_due_to_repair_cost_capitalisation(self):
+		asset = create_asset()
+		initial_asset_value = asset.asset_value
+		asset_repair = create_asset_repair(asset= asset, capitalize_repair_cost = 1, submit = 1)
+		asset.reload()
+
+		increase_in_asset_value = asset.asset_value - initial_asset_value
+		self.assertEqual(asset_repair.repair_cost, increase_in_asset_value)
+
+	def test_purchase_invoice(self):
+		asset_repair = create_asset_repair(capitalize_repair_cost = 1, submit = 1)
+		self.assertTrue(asset_repair.purchase_invoice)
+
+	def test_gl_entries(self):
+		asset_repair = create_asset_repair(capitalize_repair_cost = 1, submit = 1)
+		gl_entry = frappe.get_last_doc('GL Entry')
+		self.assertEqual(asset_repair.name, gl_entry.voucher_no)
+
+	def test_increase_in_asset_life(self):
+		asset = create_asset(calculate_depreciation = 1)
+		initial_num_of_depreciations = num_of_depreciations(asset)
+		create_asset_repair(asset= asset, capitalize_repair_cost = 1, submit = 1)
+		asset.reload()
+		self.assertEqual((initial_num_of_depreciations + 1), num_of_depreciations(asset))
+
+def num_of_depreciations(asset):
+	return asset.finance_books[0].total_number_of_depreciations
+
+def create_asset_repair(**args):
+	from erpnext.stock.doctype.warehouse.test_warehouse import create_warehouse
+	from erpnext.accounts.doctype.purchase_invoice.test_purchase_invoice import make_purchase_invoice
+
+	args = frappe._dict(args)
+
+	if args.asset:
+		asset = args.asset
+	else:
+		asset = create_asset(is_existing_asset = 1)
+	asset_repair = frappe.new_doc("Asset Repair")
+	asset_repair.update({
+		"asset": asset.name,
+		"asset_name": asset.asset_name,
+		"failure_date": nowdate(),
+		"description": "Test Description",
+		"repair_cost": 0
+	})
+
+	if args.stock_consumption:
+		asset_repair.stock_consumption = 1
+		asset_repair.warehouse = create_warehouse("Test Warehouse", company = asset.company)
+		asset_repair.append("stock_items", {
+			"item": args.item or args.item_code or "_Test Item",
+			"valuation_rate": args.rate if args.get("rate") is not None else 100,
+			"consumed_quantity": args.qty or 1
+		})
+
+	try:
+		asset_repair.save()
+	except frappe.DuplicateEntryError:
+		pass
+
+	if args.submit:
+		asset_repair.repair_status = "Completed"
+		asset_repair.cost_center = "_Test Cost Center - _TC"
+
+		if args.stock_consumption:
+			stock_entry = frappe.get_doc({
+				"doctype": "Stock Entry",
+				"stock_entry_type": "Material Receipt",
+				"company": asset.company
+			})
+			stock_entry.append('items', {
+				"t_warehouse": asset_repair.warehouse,
+				"item_code": asset_repair.stock_items[0].item,
+				"qty": asset_repair.stock_items[0].consumed_quantity
+			})
+			stock_entry.submit()
+
+		if args.capitalize_repair_cost:
+			asset_repair.capitalize_repair_cost = 1
+			asset_repair.repair_cost = 1000
+			if asset.calculate_depreciation:
+				asset_repair.increase_in_asset_life = 12
+			asset_repair.purchase_invoice = make_purchase_invoice().name
+
+		asset_repair.submit()
+	return asset_repair
\ No newline at end of file

From d4acf87feb85bf43f4096356e6c6f8d72cb47719 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 15 Jun 2021 10:35:44 +0530
Subject: [PATCH 057/106] fix(Asset Repair): Make Accounting Dimensions section
 collapsible

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 522f2874d9..b81d74bd3d 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -174,6 +174,7 @@
    "fieldtype": "Section Break"
   },
   {
+   "collapsible": 1,
    "fieldname": "accounting_dimensions_section",
    "fieldtype": "Section Break",
    "label": "Accounting Dimensions"
@@ -239,7 +240,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-05-21 10:37:35.002238",
+ "modified": "2021-06-15 10:34:00.839353",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 285bc60684c4e03e34356e990226cbe5b00edd21 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 15 Jun 2021 12:02:07 +0530
Subject: [PATCH 058/106] fix(Asset Repair): Set asset_name as title

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 1 +
 1 file changed, 1 insertion(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index b81d74bd3d..1a714a7a3c 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -241,6 +241,7 @@
  "is_submittable": 1,
  "links": [],
  "modified": "2021-06-15 10:34:00.839353",
+ "title_field": "asset_name",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 14ea9ebcb7cc34cc386c37e7f2cdc4b60a9ffd58 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 07:03:32 +0530
Subject: [PATCH 059/106] fix(Asset Repair): Display fields according to the
 state of the doc

---
 erpnext/assets/doctype/asset_repair/asset_repair.js   | 2 --
 erpnext/assets/doctype/asset_repair/asset_repair.json | 9 +++++++--
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 7633a595a2..fdb8d0af67 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -3,8 +3,6 @@
 
 frappe.ui.form.on('Asset Repair', {
 	refresh: function(frm) {
-		frm.toggle_display(['completion_date', 'repair_status', 'accounting_details', 'accounting_dimensions_section'], !(frm.doc.__islocal));
-	
 		if (frm.doc.docstatus) {
 			frm.add_custom_button("View General Ledger", function() {
 				frappe.route_options = {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 1a714a7a3c..c2780a13b9 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -71,6 +71,7 @@
   },
   {
    "allow_on_submit": 1,
+   "depends_on": "eval:!doc.__islocal",
    "fieldname": "completion_date",
    "fieldtype": "Datetime",
    "label": "Completion Date"
@@ -78,6 +79,7 @@
   {
    "allow_on_submit": 1,
    "default": "Pending",
+   "depends_on": "eval:!doc.__islocal",
    "fieldname": "repair_status",
    "fieldtype": "Select",
    "label": "Repair Status",
@@ -154,6 +156,7 @@
   },
   {
    "default": "0",
+   "depends_on": "eval:!doc.__islocal",
    "fieldname": "capitalize_repair_cost",
    "fieldtype": "Check",
    "label": "Capitalize Repair Cost"
@@ -197,6 +200,7 @@
   },
   {
    "default": "0",
+   "depends_on": "eval:!doc.__islocal",
    "fieldname": "stock_consumption",
    "fieldtype": "Check",
    "label": "Stock Consumed During Repair"
@@ -231,6 +235,7 @@
    "label": "Increase In Asset Life(Months)"
   },
   {
+   "depends_on": "eval:!doc.__islocal",
    "fieldname": "purchase_invoice",
    "fieldtype": "Link",
    "label": "Purchase Invoice",
@@ -240,8 +245,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-15 10:34:00.839353",
- "title_field": "asset_name",
+ "modified": "2021-06-16 07:01:28.217619",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
@@ -280,6 +284,7 @@
  ],
  "sort_field": "modified",
  "sort_order": "DESC",
+ "title_field": "asset_name",
  "track_changes": 1,
  "track_seen": 1
 }
\ No newline at end of file

From 695cd7099458a571f4beb086be7043a34672b8a5 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 07:50:03 +0530
Subject: [PATCH 060/106] fix(Asset Repair): Add title to error messages

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 8724d0a125..bdc21a7b7b 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -55,9 +55,9 @@ class AssetRepair(Document):
 
 	def check_for_stock_items_and_warehouse(self):
 		if not self.stock_items:
-			frappe.throw(_("Please enter Stock Items consumed during Asset Repair."))
+			frappe.throw(_("Please enter Stock Items consumed during the Repair."), title=_("Missing Items"))
 		if not self.warehouse:
-			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Repair were taken."))
+			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during the Repair were taken."), title=_("Missing Warehouse"))
 
 	def check_for_cost_center(self):
 		if not self.cost_center:

From bf52f558705153098195f01b5bd786aea3337b2a Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 07:56:40 +0530
Subject: [PATCH 061/106] fix(Asset Repair): Make Stock Items and Warehouse
 mandatory if stock_consumption is checked

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index c2780a13b9..253321ad1e 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -170,6 +170,7 @@
    "fieldname": "stock_items",
    "fieldtype": "Table",
    "label": "Stock Items",
+   "mandatory_depends_on": "stock_consumption",
    "options": "Stock Item"
   },
   {
@@ -218,6 +219,7 @@
    "label": "Total Repair Cost"
   },
   {
+   "depends_on": "stock_consumption",
    "fieldname": "warehouse",
    "fieldtype": "Link",
    "label": "Warehouse",
@@ -245,7 +247,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-16 07:01:28.217619",
+ "modified": "2021-06-16 07:52:49.438800",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 9e07b7d4a736e324ad50f7053039d10c97fabd3c Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:07:06 +0530
Subject: [PATCH 062/106] fix(Asset Repair): Add Company field

---
 .../assets/doctype/asset_repair/asset_repair.json    | 10 +++++++++-
 erpnext/assets/doctype/asset_repair/asset_repair.py  | 12 +++++-------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 253321ad1e..5cc236393f 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -11,6 +11,7 @@
   "naming_series",
   "column_break_2",
   "asset_name",
+  "company",
   "section_break_5",
   "failure_date",
   "repair_status",
@@ -242,12 +243,19 @@
    "fieldtype": "Link",
    "label": "Purchase Invoice",
    "options": "Purchase Invoice"
+  },
+  {
+   "fetch_from": "asset.company",
+   "fieldname": "company",
+   "fieldtype": "Link",
+   "label": "Company",
+   "options": "Company"
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-16 07:52:49.438800",
+ "modified": "2021-06-16 08:02:34.782990",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index bdc21a7b7b..d84810590d 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -76,7 +76,7 @@ class AssetRepair(Document):
 		stock_entry = frappe.get_doc({
 			"doctype": "Stock Entry",
 			"stock_entry_type": "Material Issue",
-			"company": frappe.get_value('Asset', self.asset, "company")
+			"company": self.company
 		})
 
 		for stock_item in self.stock_items:
@@ -103,8 +103,7 @@ class AssetRepair(Document):
 
 	def get_gl_entries(self):
 		gl_entry = []
-		company = frappe.db.get_value('Asset', self.asset, 'company')
-		repair_and_maintenance_account = frappe.db.get_value('Company', company, 'repair_and_maintenance_account')
+		repair_and_maintenance_account = frappe.db.get_value('Company', self.company, 'repair_and_maintenance_account')
 		fixed_asset_account = self.get_fixed_asset_account()
 		expense_account = frappe.get_doc('Purchase Invoice', self.purchase_invoice).items[0].expense_account	
 
@@ -118,7 +117,7 @@ class AssetRepair(Document):
 			"voucher_no": self.name,
 			"cost_center": self.cost_center,
 			"posting_date": getdate(),
-			"company": company
+			"company": self.company
 		})
 		gl_entry.insert()
 		gl_entry = frappe.get_doc({
@@ -133,15 +132,14 @@ class AssetRepair(Document):
 			"posting_date": getdate(),
 			"against_voucher_type": "Purchase Invoice",
 			"against_voucher": self.purchase_invoice,
-			"company": company
+			"company": self.company
 		})
 		gl_entry.insert()
 
 	def get_fixed_asset_account(self):
 		asset_category = frappe.get_doc('Asset Category', frappe.db.get_value('Asset', self.asset, 'asset_category'))
-		company = frappe.db.get_value('Asset', self.asset, 'company')
 		for account in asset_category.accounts:
-			if account.company_name == company:
+			if account.company_name == self.company:
 				return account.fixed_asset_account
 
 	def modify_depreciation_schedule(self):

From 71c60f75d7f45abd16cac637ab64cc8ecd0322a8 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:13:36 +0530
Subject: [PATCH 063/106] fix(Asset Repair): Filter Cost Center and Project by
 Company

---
 .../doctype/asset_repair/asset_repair.js       | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index fdb8d0af67..1e87722179 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -30,3 +30,21 @@ frappe.ui.form.on('Asset Repair', {
 		}
 	}
 });
+
+cur_frm.fields_dict.cost_center.get_query = function(doc) {
+	return{
+		filters:{
+			'is_group': 0,
+			'company': doc.company
+		}
+	}
+}
+
+cur_frm.fields_dict.project.get_query = function(doc) {
+	return{
+		filters:{
+			'is_group': 0,
+			'company': doc.company
+		}
+	}
+}
\ No newline at end of file

From 94ac52c47de45273570aa09c3c56a82084ce076b Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:18:45 +0530
Subject: [PATCH 064/106] fix(Asset Repair): Add mandatory_depends_on condition
 for Purchase Invoice

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 3 ++-
 erpnext/assets/doctype/asset_repair/asset_repair.py   | 5 -----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 5cc236393f..b2aac7a4e6 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -242,6 +242,7 @@
    "fieldname": "purchase_invoice",
    "fieldtype": "Link",
    "label": "Purchase Invoice",
+   "mandatory_depends_on": "eval: doc.repair_status == 'Completed' && doc.repair_cost > 0",
    "options": "Purchase Invoice"
   },
   {
@@ -255,7 +256,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-16 08:02:34.782990",
+ "modified": "2021-06-16 08:16:07.581813",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index d84810590d..39f7ee2153 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -44,7 +44,6 @@ class AssetRepair(Document):
 			self.check_for_stock_items_and_warehouse()
 			self.decrease_stock_quantity()
 		if self.capitalize_repair_cost:
-			self.check_for_purchase_invoice()
 			self.make_gl_entries()
 			if frappe.db.get_value('Asset', self.asset, 'calculate_depreciation'):
 				self.modify_depreciation_schedule()
@@ -89,10 +88,6 @@ class AssetRepair(Document):
 		stock_entry.insert()
 		stock_entry.submit()
 
-	def check_for_purchase_invoice(self):
-		if not self.purchase_invoice:
-			frappe.throw(_("Please link Purchase Invoice."))
-
 	def on_cancel(self):
 		self.make_gl_entries(cancel=True)
 

From c61bbc5915c02515d053534cb831afdc4e8af13a Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:26:02 +0530
Subject: [PATCH 065/106] fix(Asset Repair): Use existing function from
 asset.py for fetching fixed_asset_account

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 39f7ee2153..443c0a78f5 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -8,6 +8,7 @@ from frappe import _
 from frappe.utils import time_diff_in_hours, getdate, nowdate, add_months, flt, cint
 from frappe.model.document import Document
 from erpnext.accounts.general_ledger import make_gl_entries
+from erpnext.assets.doctype.asset.asset import get_asset_account
 
 class AssetRepair(Document):
 	def validate(self):
@@ -99,7 +100,7 @@ class AssetRepair(Document):
 	def get_gl_entries(self):
 		gl_entry = []
 		repair_and_maintenance_account = frappe.db.get_value('Company', self.company, 'repair_and_maintenance_account')
-		fixed_asset_account = self.get_fixed_asset_account()
+		fixed_asset_account = get_asset_account("fixed_asset_account", asset=self.asset, company=self.company)
 		expense_account = frappe.get_doc('Purchase Invoice', self.purchase_invoice).items[0].expense_account	
 
 		gl_entry = frappe.get_doc({
@@ -131,12 +132,6 @@ class AssetRepair(Document):
 		})
 		gl_entry.insert()
 
-	def get_fixed_asset_account(self):
-		asset_category = frappe.get_doc('Asset Category', frappe.db.get_value('Asset', self.asset, 'asset_category'))
-		for account in asset_category.accounts:
-			if account.company_name == self.company:
-				return account.fixed_asset_account
-
 	def modify_depreciation_schedule(self):
 		if self.increase_in_asset_life:
 			asset = frappe.get_doc('Asset', self.asset)

From abb0c769a4bf1a4b7c83b78f874f47e9f501ff65 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:33:05 +0530
Subject: [PATCH 066/106] fix(Asset Repair): Uncheck allow_on_submit for all
 fields

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index b2aac7a4e6..7e9587428b 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -71,14 +71,12 @@
    "fieldtype": "Column Break"
   },
   {
-   "allow_on_submit": 1,
    "depends_on": "eval:!doc.__islocal",
    "fieldname": "completion_date",
    "fieldtype": "Datetime",
    "label": "Completion Date"
   },
   {
-   "allow_on_submit": 1,
    "default": "Pending",
    "depends_on": "eval:!doc.__islocal",
    "fieldname": "repair_status",
@@ -104,13 +102,11 @@
    "fieldtype": "Column Break"
   },
   {
-   "allow_on_submit": 1,
    "fieldname": "actions_performed",
    "fieldtype": "Long Text",
    "label": "Actions performed"
   },
   {
-   "allow_on_submit": 1,
    "fieldname": "downtime",
    "fieldtype": "Data",
    "in_list_view": 1,
@@ -122,7 +118,6 @@
    "fieldtype": "Column Break"
   },
   {
-   "allow_on_submit": 1,
    "fieldname": "repair_cost",
    "fieldtype": "Currency",
    "label": "Repair Cost"
@@ -256,7 +251,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-16 08:16:07.581813",
+ "modified": "2021-06-16 08:32:06.160615",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From a33e751b0f4dcb773586c88dfd3f025cb6a62995 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:35:50 +0530
Subject: [PATCH 067/106] fix(Asset Repair): Make Cost Center non-mandatory

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 443c0a78f5..688f54aee7 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -37,7 +37,6 @@ class AssetRepair(Document):
 
 	def on_submit(self):
 		self.check_repair_status()
-		self.check_for_cost_center()
 
 		if self.stock_consumption or self.capitalize_repair_cost:
 			self.increase_asset_value()
@@ -59,10 +58,6 @@ class AssetRepair(Document):
 		if not self.warehouse:
 			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during the Repair were taken."), title=_("Missing Warehouse"))
 
-	def check_for_cost_center(self):
-		if not self.cost_center:
-			frappe.throw(_("Please enter Cost Center."))
-
 	def increase_asset_value(self):
 		asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value')
 		for item in self.stock_items:

From 491763fa277bb1af5241f62dedd1bd7374af7ac6 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 08:45:54 +0530
Subject: [PATCH 068/106] fix(Asset Repair): Fix Sider issues

---
 .../assets/doctype/asset_repair/asset_repair.js  | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 1e87722179..2319b069b0 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -32,19 +32,19 @@ frappe.ui.form.on('Asset Repair', {
 });
 
 cur_frm.fields_dict.cost_center.get_query = function(doc) {
-	return{
-		filters:{
+	return {
+		filters: {
 			'is_group': 0,
 			'company': doc.company
 		}
-	}
-}
+	};
+};
 
 cur_frm.fields_dict.project.get_query = function(doc) {
-	return{
-		filters:{
+	return {
+		filters: {
 			'is_group': 0,
 			'company': doc.company
 		}
-	}
-}
\ No newline at end of file
+	};
+};
\ No newline at end of file

From 6d6aee29e835dfa46b85b4c3a75deb5c892b9223 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 10:42:37 +0530
Subject: [PATCH 069/106] fix(Asset Repair): Fix GL Entry creation

---
 .../doctype/asset_repair/asset_repair.py      | 78 ++++++++++++-------
 1 file changed, 48 insertions(+), 30 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 688f54aee7..ccf8d5ca39 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -9,8 +9,9 @@ from frappe.utils import time_diff_in_hours, getdate, nowdate, add_months, flt,
 from frappe.model.document import Document
 from erpnext.accounts.general_ledger import make_gl_entries
 from erpnext.assets.doctype.asset.asset import get_asset_account
+from erpnext.controllers.accounts_controller import AccountsController
 
-class AssetRepair(Document):
+class AssetRepair(AccountsController):
 	def validate(self):
 		if self.repair_status == "Completed" and not self.completion_date:
 			self.completion_date = nowdate()
@@ -93,39 +94,56 @@ class AssetRepair(Document):
 			make_gl_entries(gl_entries, cancel)
 
 	def get_gl_entries(self):
-		gl_entry = []
+		gl_entries = []
 		repair_and_maintenance_account = frappe.db.get_value('Company', self.company, 'repair_and_maintenance_account')
 		fixed_asset_account = get_asset_account("fixed_asset_account", asset=self.asset, company=self.company)
 		expense_account = frappe.get_doc('Purchase Invoice', self.purchase_invoice).items[0].expense_account	
 
-		gl_entry = frappe.get_doc({
-			"doctype": "GL Entry",
-			"account": expense_account,
-			"credit": self.total_repair_cost,
-			"credit_in_account_currency": self.total_repair_cost,
-			"against": repair_and_maintenance_account,
-			"voucher_type": self.doctype,		
-			"voucher_no": self.name,
-			"cost_center": self.cost_center,
-			"posting_date": getdate(),
-			"company": self.company
-		})
-		gl_entry.insert()
-		gl_entry = frappe.get_doc({
-			"doctype": "GL Entry",
-			"account": fixed_asset_account,
-			"debit": self.total_repair_cost,
-			"debit_in_account_currency": self.total_repair_cost,
-			"against": expense_account,
-			"voucher_type": self.doctype,
-			"voucher_no": self.name,
-			"cost_center": self.cost_center,
-			"posting_date": getdate(),
-			"against_voucher_type": "Purchase Invoice",
-			"against_voucher": self.purchase_invoice,
-			"company": self.company
-		})
-		gl_entry.insert()
+		gl_entries.append(
+			self.get_gl_dict({
+				"account": expense_account,
+				"credit": self.repair_cost,
+				"credit_in_account_currency": self.repair_cost,
+				"against": repair_and_maintenance_account,
+				"voucher_type": self.doctype,		
+				"voucher_no": self.name,
+				"cost_center": self.cost_center,
+				"posting_date": getdate(),
+				"company": self.company
+			}, item=self)
+		)
+
+		gl_entries.append(
+			self.get_gl_dict({
+				"account": expense_account,
+				"credit": self.total_repair_cost - self.repair_cost,
+				"credit_in_account_currency": self.total_repair_cost - self.repair_cost,
+				"against": repair_and_maintenance_account,
+				"voucher_type": self.doctype,		
+				"voucher_no": self.name,
+				"cost_center": self.cost_center,
+				"posting_date": getdate(),
+				"company": self.company
+			}, item=self)
+		)
+
+		gl_entries.append(
+			self.get_gl_dict({
+				"account": fixed_asset_account,
+				"debit": self.total_repair_cost,
+				"debit_in_account_currency": self.total_repair_cost,
+				"against": expense_account,
+				"voucher_type": self.doctype,
+				"voucher_no": self.name,
+				"cost_center": self.cost_center,
+				"posting_date": getdate(),
+				"against_voucher_type": "Purchase Invoice",
+				"against_voucher": self.purchase_invoice,
+				"company": self.company
+			}, item=self)
+		)
+
+		return gl_entries
 
 	def modify_depreciation_schedule(self):
 		if self.increase_in_asset_life:

From 34997789cdd2e328cafa1883c6a6b0fa408c5694 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 16 Jun 2021 10:48:07 +0530
Subject: [PATCH 070/106] fix(Asset Repair): Filter Warehouse by Company

---
 erpnext/assets/doctype/asset_repair/asset_repair.js | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 2319b069b0..efa6a9d494 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -41,6 +41,14 @@ cur_frm.fields_dict.cost_center.get_query = function(doc) {
 };
 
 cur_frm.fields_dict.project.get_query = function(doc) {
+	return {
+		filters: {
+			'company': doc.company
+		}
+	};
+};
+
+cur_frm.fields_dict.warehouse.get_query = function(doc) {
 	return {
 		filters: {
 			'is_group': 0,

From 66d4e2ba51baada6ba855a04b1452d6d53f601e3 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 17 Jun 2021 08:28:19 +0530
Subject: [PATCH 071/106] fix(Asset Repair): Display value_after_depreciation
 in Finance Books

---
 .../assets/doctype/asset_finance_book/asset_finance_book.json  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json b/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json
index d9b7b695f7..ee3a2072f0 100644
--- a/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json
+++ b/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json
@@ -67,7 +67,6 @@
   {
    "fieldname": "value_after_depreciation",
    "fieldtype": "Currency",
-   "hidden": 1,
    "label": "Value After Depreciation",
    "no_copy": 1,
    "options": "Company:company:default_currency",
@@ -85,7 +84,7 @@
  "index_web_pages_for_search": 1,
  "istable": 1,
  "links": [],
- "modified": "2020-11-05 16:30:09.213479",
+ "modified": "2021-06-17 08:02:32.650738",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Finance Book",

From 05c70ac5849b15d953974f751c6adebdf6bfb51c Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 17 Jun 2021 13:02:02 +0530
Subject: [PATCH 072/106] fix(Asset): Replace asset_value with
 value_after_depreciation in Finance Books

---
 erpnext/assets/doctype/asset/asset.json          |  9 +--------
 erpnext/assets/doctype/asset/asset.py            | 11 ++++++-----
 .../asset_finance_book/asset_finance_book.json   |  2 +-
 .../assets/doctype/asset_repair/asset_repair.py  | 16 +++++++++++-----
 4 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.json b/erpnext/assets/doctype/asset/asset.json
index 8a0e3ad2a6..d55258c8f6 100644
--- a/erpnext/assets/doctype/asset/asset.json
+++ b/erpnext/assets/doctype/asset/asset.json
@@ -23,7 +23,6 @@
   "asset_name",
   "asset_category",
   "location",
-  "asset_value",
   "custodian",
   "department",
   "disposal_date",
@@ -484,12 +483,6 @@
    "fieldtype": "Section Break",
    "label": "Finance Books"
   },
-  {
-   "fieldname": "asset_value",
-   "fieldtype": "Currency",
-   "label": "Asset Value",
-   "read_only": 1
-  },
   {
    "fieldname": "to_date",
    "fieldtype": "Date",
@@ -523,7 +516,7 @@
    "link_fieldname": "asset"
   }
  ],
- "modified": "2021-05-21 12:05:29.424083",
+ "modified": "2021-06-17 12:59:39.189106",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset",
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index ade74e6055..f67266bb92 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -96,9 +96,6 @@ class Asset(AccountsController):
 			finance_books = get_item_details(self.item_code, self.asset_category)
 			self.set('finance_books', finance_books)
 
-		if not(self.asset_value):
-			self.asset_value = self.gross_purchase_amount
-
 	def validate_asset_values(self):
 		if not self.asset_category:
 			self.asset_category = frappe.get_cached_value("Item", self.item_code, "asset_category")
@@ -187,8 +184,12 @@ class Asset(AccountsController):
 					start = n
 					break
 
-			value_after_depreciation = (flt(self.asset_value) -
-				flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
+			if d.value_after_depreciation: 
+				value_after_depreciation = (flt(d.value_after_depreciation) -
+					flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
+			else:
+				value_after_depreciation = (flt(self.gross_purchase_amount) -
+					flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
 
 			d.value_after_depreciation = value_after_depreciation
 
diff --git a/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json b/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json
index ee3a2072f0..e5a5f194c1 100644
--- a/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json
+++ b/erpnext/assets/doctype/asset_finance_book/asset_finance_book.json
@@ -84,7 +84,7 @@
  "index_web_pages_for_search": 1,
  "istable": 1,
  "links": [],
- "modified": "2021-06-17 08:02:32.650738",
+ "modified": "2021-06-17 12:59:05.743683",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Finance Book",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index ccf8d5ca39..678a47e8c7 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -60,13 +60,19 @@ class AssetRepair(AccountsController):
 			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during the Repair were taken."), title=_("Missing Warehouse"))
 
 	def increase_asset_value(self):
-		asset_value = frappe.db.get_value('Asset', self.asset, 'asset_value')
+		total_value_of_stock_consumed = 0
 		for item in self.stock_items:
-			asset_value += item.total_value
+			total_value_of_stock_consumed += item.total_value
 
-		if self.capitalize_repair_cost:
-			asset_value += self.repair_cost
-		frappe.db.set_value('Asset', self.asset, 'asset_value', asset_value)
+		asset = frappe.get_doc('Asset', self.asset)
+		asset.flags.ignore_validate_update_after_submit = True
+		if asset.calculate_depreciation:
+			for row in asset.finance_books:
+				row.value_after_depreciation += total_value_of_stock_consumed
+
+				if self.capitalize_repair_cost:
+					row.value_after_depreciation += self.repair_cost
+		asset.save()
 
 	def decrease_stock_quantity(self):
 		stock_entry = frappe.get_doc({

From 354116142a856c81750942ee6f887c3d529b1731 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 18 Jun 2021 09:53:18 +0530
Subject: [PATCH 073/106] fix(Asset Repair): Create GL Entries for each item in
 Stock Items

---
 .../doctype/asset_repair/asset_repair.py      | 30 +++++++++++--------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 678a47e8c7..3bcf958db6 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -119,19 +119,23 @@ class AssetRepair(AccountsController):
 			}, item=self)
 		)
 
-		gl_entries.append(
-			self.get_gl_dict({
-				"account": expense_account,
-				"credit": self.total_repair_cost - self.repair_cost,
-				"credit_in_account_currency": self.total_repair_cost - self.repair_cost,
-				"against": repair_and_maintenance_account,
-				"voucher_type": self.doctype,		
-				"voucher_no": self.name,
-				"cost_center": self.cost_center,
-				"posting_date": getdate(),
-				"company": self.company
-			}, item=self)
-		)
+		if self.stock_consumption:
+			# creating GL Entries for each row in Stock Items based on the Stock Entry created for it
+			stock_entry = frappe.get_last_doc('Stock Entry')
+			for item in stock_entry.items:
+				gl_entries.append(
+					self.get_gl_dict({
+						"account": item.expense_account,
+						"credit": item.amount,
+						"credit_in_account_currency": item.amount,
+						"against": repair_and_maintenance_account,
+						"voucher_type": self.doctype,		
+						"voucher_no": self.name,
+						"cost_center": self.cost_center,
+						"posting_date": getdate(),
+						"company": self.company
+					}, item=self)
+				)
 
 		gl_entries.append(
 			self.get_gl_dict({

From 94dfd0e318cf13164a2b6eab5ed7d9cef369467f Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Fri, 18 Jun 2021 09:59:45 +0530
Subject: [PATCH 074/106] fix(Asset): Add function to clear old depreciation
 schedule

---
 erpnext/assets/doctype/asset/asset.py | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index f67266bb92..1cef290a7a 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -176,13 +176,8 @@ class Asset(AccountsController):
 
 		for d in self.get('finance_books'):
 			self.validate_asset_finance_books(d)
-
-			start = 0
-			for n in range(len(self.schedules)):
-				if not self.schedules[n].journal_entry:
-					del self.schedules[n:]
-					start = n
-					break
+			
+			start = self.clear_depreciation_schedule()
 
 			if d.value_after_depreciation: 
 				value_after_depreciation = (flt(d.value_after_depreciation) -
@@ -296,6 +291,15 @@ class Asset(AccountsController):
 							"finance_book_id": d.idx
 						})
 
+	def clear_depreciation_schedule(self):
+		start = 0
+		for n in range(len(self.schedules)):
+			if not self.schedules[n].journal_entry:
+				del self.schedules[n:]
+				start = n
+				break
+		return start
+
 	def check_is_pro_rata(self, row):
 		has_pro_rata = False
 

From 93b975277179481b9884215d6d15a44d76fdd3d2 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 13:06:27 +0530
Subject: [PATCH 075/106] fix: Add comments

---
 erpnext/assets/doctype/asset/asset.py         | 12 ++++++--
 .../doctype/asset_repair/asset_repair.py      | 30 +++++++++++--------
 2 files changed, 28 insertions(+), 14 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 1cef290a7a..9a8b6c9791 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -179,7 +179,8 @@ class Asset(AccountsController):
 			
 			start = self.clear_depreciation_schedule()
 
-			if d.value_after_depreciation: 
+			# value_after_depreciation - current Asset value
+			if d.value_after_depreciation:
 				value_after_depreciation = (flt(d.value_after_depreciation) -
 					flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
 			else:
@@ -291,6 +292,7 @@ class Asset(AccountsController):
 							"finance_book_id": d.idx
 						})
 
+	# used when depreciation schedule needs to be modified due to increase in asset life
 	def clear_depreciation_schedule(self):
 		start = 0
 		for n in range(len(self.schedules)):
@@ -300,10 +302,13 @@ class Asset(AccountsController):
 				break
 		return start
 
+
+	# if it returns True, depreciation_amount will not be equal for the first and last rows
 	def check_is_pro_rata(self, row):
 		has_pro_rata = False
-
 		days = date_diff(row.depreciation_start_date, self.available_for_use_date) + 1
+
+		# if frequency_of_depreciation is 12 months, total_days = 365
 		total_days = get_total_days(row.depreciation_start_date, row.frequency_of_depreciation)
 
 		if days < total_days:
@@ -783,9 +788,12 @@ def get_depreciation_amount(asset, depreciable_value, row):
 	depreciation_left = flt(row.total_number_of_depreciations) - flt(asset.number_of_depreciations_booked)
 
 	if row.depreciation_method in ("Straight Line", "Manual"):
+		# if the Depreciation Schedule is being prepared for the first time
 		if not asset.to_date:
 			depreciation_amount = (flt(row.value_after_depreciation) -
 				flt(row.expected_value_after_useful_life)) / depreciation_left
+
+		# if the Depreciation Schedule is being modified after Asset Repair
 		else:
 			depreciation_amount = (flt(row.value_after_depreciation) -
 				flt(row.expected_value_after_useful_life)) / (date_diff(asset.to_date, asset.available_for_use_date) / 365)
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 3bcf958db6..fb815a2451 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -46,7 +46,7 @@ class AssetRepair(AccountsController):
 			self.decrease_stock_quantity()
 		if self.capitalize_repair_cost:
 			self.make_gl_entries()
-			if frappe.db.get_value('Asset', self.asset, 'calculate_depreciation'):
+			if frappe.db.get_value('Asset', self.asset, 'calculate_depreciation') and self.increase_in_asset_life:
 				self.modify_depreciation_schedule()
 
 	def check_repair_status(self):
@@ -156,27 +156,33 @@ class AssetRepair(AccountsController):
 		return gl_entries
 
 	def modify_depreciation_schedule(self):
-		if self.increase_in_asset_life:
-			asset = frappe.get_doc('Asset', self.asset)
-			asset.flags.ignore_validate_update_after_submit = True
-			for row in asset.finance_books:
-				row.total_number_of_depreciations += self.increase_in_asset_life/row.frequency_of_depreciation
+		asset = frappe.get_doc('Asset', self.asset)
+		asset.flags.ignore_validate_update_after_submit = True
+		for row in asset.finance_books:
+			row.total_number_of_depreciations += self.increase_in_asset_life/row.frequency_of_depreciation
 
-				asset.edit_dates = ""
-				extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
-				if extra_months != 0:
-					self.calculate_last_schedule_date(asset, row, extra_months)
+			asset.edit_dates = ""
+			extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
+			if extra_months != 0:
+				self.calculate_last_schedule_date(asset, row, extra_months)
 
-			asset.prepare_depreciation_data()
-			asset.save()
+		asset.prepare_depreciation_data()
+		asset.save()
 
 	# to help modify depreciation schedule when increase_in_asset_life is not a multiple of frequency_of_depreciation
 	def calculate_last_schedule_date(self, asset, row, extra_months):
 		asset.edit_dates = "Don't Edit"
 		number_of_pending_depreciations = cint(row.total_number_of_depreciations) - \
 			cint(asset.number_of_depreciations_booked)
+
+		# the Schedule Date in the final row of the old Depreciation Schedule
 		last_schedule_date = asset.schedules[len(asset.schedules)-1].schedule_date
+
+		# the Schedule Date in the final row of the new Depreciation Schedule
 		asset.to_date = add_months(last_schedule_date, extra_months)
+
+		# the latest possible date at which the depreciation can occur, without increasing the Total Number of Depreciations
+		# if depreciations happen yearly and the Depreciation Posting Date is 01-01-2020, this could be 01-01-2021, 01-01-2022...
 		schedule_date = add_months(row.depreciation_start_date,
 			number_of_pending_depreciations * cint(row.frequency_of_depreciation))
 

From 2b93e54e1f43e35d2b51ea3b954ef7ad87892844 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 13:45:37 +0530
Subject: [PATCH 076/106] fix(Asset Repair): Fix depreciation_amount
 calculation

---
 erpnext/assets/doctype/asset/asset.py |  2 +-
 erpnext/regional/india/utils.py       | 12 ++++++++++--
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 9a8b6c9791..a2917fe399 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -789,7 +789,7 @@ def get_depreciation_amount(asset, depreciable_value, row):
 
 	if row.depreciation_method in ("Straight Line", "Manual"):
 		# if the Depreciation Schedule is being prepared for the first time
-		if not asset.to_date:
+		if not asset.edit_dates:
 			depreciation_amount = (flt(row.value_after_depreciation) -
 				flt(row.expected_value_after_useful_life)) / depreciation_left
 
diff --git a/erpnext/regional/india/utils.py b/erpnext/regional/india/utils.py
index 075c698fea..4d373444a6 100644
--- a/erpnext/regional/india/utils.py
+++ b/erpnext/regional/india/utils.py
@@ -838,8 +838,16 @@ def get_depreciation_amount(asset, depreciable_value, row):
 	depreciation_left = flt(row.total_number_of_depreciations) - flt(asset.number_of_depreciations_booked)
 
 	if row.depreciation_method in ("Straight Line", "Manual"):
-		depreciation_amount = (flt(row.value_after_depreciation) -
-			flt(row.expected_value_after_useful_life)) / depreciation_left
+		# if the Depreciation Schedule is being prepared for the first time
+		if not asset.edit_dates:
+			depreciation_amount = (flt(row.value_after_depreciation) -
+				flt(row.expected_value_after_useful_life)) / depreciation_left
+
+		# if the Depreciation Schedule is being modified after Asset Repair
+		else:
+			depreciation_amount = (flt(row.value_after_depreciation) -
+				flt(row.expected_value_after_useful_life)) / (date_diff(asset.to_date, asset.available_for_use_date) / 365)
+		
 	else:
 		rate_of_depreciation = row.rate_of_depreciation
 		# if its the first depreciation

From da8da9fa4e2991029d7c90e9fdd8243360b9f3f4 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 14:00:26 +0530
Subject: [PATCH 077/106] fix: Replace edit_dates with
 flags.increase_in_asset_life

---
 erpnext/assets/doctype/asset/asset.json             | 9 +--------
 erpnext/assets/doctype/asset/asset.py               | 4 ++--
 erpnext/assets/doctype/asset_repair/asset_repair.py | 4 ++--
 erpnext/regional/india/utils.py                     | 2 +-
 4 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.json b/erpnext/assets/doctype/asset/asset.json
index d55258c8f6..d77eb10418 100644
--- a/erpnext/assets/doctype/asset/asset.json
+++ b/erpnext/assets/doctype/asset/asset.json
@@ -54,7 +54,6 @@
   "section_break_14",
   "schedules",
   "to_date",
-  "edit_dates",
   "insurance_details",
   "policy_number",
   "insurer",
@@ -488,12 +487,6 @@
    "fieldtype": "Date",
    "hidden": 1,
    "label": "To Date"
-  },
-  {
-   "fieldname": "edit_dates",
-   "fieldtype": "Data",
-   "hidden": 1,
-   "label": "Edit Dates"
   }
  ],
  "idx": 72,
@@ -516,7 +509,7 @@
    "link_fieldname": "asset"
   }
  ],
- "modified": "2021-06-17 12:59:39.189106",
+ "modified": "2021-06-19 13:56:58.450182",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset",
diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index a2917fe399..27d21e2542 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -223,7 +223,7 @@ class Asset(AccountsController):
 
 				# For last row
 				elif has_pro_rata and n == cint(number_of_pending_depreciations) - 1:
-					if not self.edit_dates:
+					if not self.flags.increase_in_asset_life:
 						self.to_date = add_months(self.available_for_use_date,
 							n * cint(d.frequency_of_depreciation))
 
@@ -789,7 +789,7 @@ def get_depreciation_amount(asset, depreciable_value, row):
 
 	if row.depreciation_method in ("Straight Line", "Manual"):
 		# if the Depreciation Schedule is being prepared for the first time
-		if not asset.edit_dates:
+		if not asset.flags.increase_in_asset_life:
 			depreciation_amount = (flt(row.value_after_depreciation) -
 				flt(row.expected_value_after_useful_life)) / depreciation_left
 
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index fb815a2451..9ef6591499 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -161,7 +161,7 @@ class AssetRepair(AccountsController):
 		for row in asset.finance_books:
 			row.total_number_of_depreciations += self.increase_in_asset_life/row.frequency_of_depreciation
 
-			asset.edit_dates = ""
+			asset.flags.increase_in_asset_life = False
 			extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
 			if extra_months != 0:
 				self.calculate_last_schedule_date(asset, row, extra_months)
@@ -171,7 +171,7 @@ class AssetRepair(AccountsController):
 
 	# to help modify depreciation schedule when increase_in_asset_life is not a multiple of frequency_of_depreciation
 	def calculate_last_schedule_date(self, asset, row, extra_months):
-		asset.edit_dates = "Don't Edit"
+		asset.flags.increase_in_asset_life = True
 		number_of_pending_depreciations = cint(row.total_number_of_depreciations) - \
 			cint(asset.number_of_depreciations_booked)
 
diff --git a/erpnext/regional/india/utils.py b/erpnext/regional/india/utils.py
index 4d373444a6..0dafe01711 100644
--- a/erpnext/regional/india/utils.py
+++ b/erpnext/regional/india/utils.py
@@ -839,7 +839,7 @@ def get_depreciation_amount(asset, depreciable_value, row):
 
 	if row.depreciation_method in ("Straight Line", "Manual"):
 		# if the Depreciation Schedule is being prepared for the first time
-		if not asset.edit_dates:
+		if not asset.flags.increase_in_asset_life:
 			depreciation_amount = (flt(row.value_after_depreciation) -
 				flt(row.expected_value_after_useful_life)) / depreciation_left
 

From 09ba6f64770102520bb5c94f7ce862f9f9752597 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 14:06:45 +0530
Subject: [PATCH 078/106] fix(Asset Repair): Move Total Repair Cost to the
 Stock Consumption Details section

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 7e9587428b..f43b5d92bf 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -26,11 +26,11 @@
   "capitalize_repair_cost",
   "stock_consumption",
   "column_break_8",
-  "total_repair_cost",
   "purchase_invoice",
   "stock_consumption_details_section",
   "warehouse",
   "stock_items",
+  "total_repair_cost",
   "asset_depreciation_details_section",
   "increase_in_asset_life",
   "section_break_9",
@@ -210,6 +210,7 @@
   },
   {
    "depends_on": "stock_consumption",
+   "description": "Sum of Repair Cost and the total value of all Stock Items consumed during the repair.",
    "fieldname": "total_repair_cost",
    "fieldtype": "Currency",
    "label": "Total Repair Cost"
@@ -251,7 +252,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-16 08:32:06.160615",
+ "modified": "2021-06-19 14:04:35.423111",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 7db7988e4c08db88c1a9872db29748afa10a1a67 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 14:54:30 +0530
Subject: [PATCH 079/106] fix(Asset Repair): Add Stock Entry field

---
 .../doctype/asset_repair/asset_repair.json       | 16 +++++++++++++---
 .../assets/doctype/asset_repair/asset_repair.py  |  4 +++-
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index f43b5d92bf..6c90ca2348 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -31,6 +31,7 @@
   "warehouse",
   "stock_items",
   "total_repair_cost",
+  "stock_entry",
   "asset_depreciation_details_section",
   "increase_in_asset_life",
   "section_break_9",
@@ -118,6 +119,7 @@
    "fieldtype": "Column Break"
   },
   {
+   "default": "0",
    "fieldname": "repair_cost",
    "fieldtype": "Currency",
    "label": "Repair Cost"
@@ -209,11 +211,12 @@
    "label": "Stock Consumption Details"
   },
   {
-   "depends_on": "stock_consumption",
+   "depends_on": "eval: doc.stock_consumption && doc.total_repair_cost > 0",
    "description": "Sum of Repair Cost and the total value of all Stock Items consumed during the repair.",
    "fieldname": "total_repair_cost",
    "fieldtype": "Currency",
-   "label": "Total Repair Cost"
+   "label": "Total Repair Cost",
+   "read_only": 1
   },
   {
    "depends_on": "stock_consumption",
@@ -247,12 +250,19 @@
    "fieldtype": "Link",
    "label": "Company",
    "options": "Company"
+  },
+  {
+   "fieldname": "stock_entry",
+   "fieldtype": "Link",
+   "label": "Stock Entry",
+   "options": "Stock Entry",
+   "read_only": 1
   }
  ],
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-19 14:04:35.423111",
+ "modified": "2021-06-19 14:47:25.875814",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 9ef6591499..212af7a930 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -91,6 +91,8 @@ class AssetRepair(AccountsController):
 		stock_entry.insert()
 		stock_entry.submit()
 
+		self.stock_entry = stock_entry.name
+
 	def on_cancel(self):
 		self.make_gl_entries(cancel=True)
 
@@ -121,7 +123,7 @@ class AssetRepair(AccountsController):
 
 		if self.stock_consumption:
 			# creating GL Entries for each row in Stock Items based on the Stock Entry created for it
-			stock_entry = frappe.get_last_doc('Stock Entry')
+			stock_entry = frappe.get_doc('Stock Entry', self.stock_entry)
 			for item in stock_entry.items:
 				gl_entries.append(
 					self.get_gl_dict({

From 012b9eaeff8a2729ec1962260f5c05d8fb2fc5c3 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 15:18:54 +0530
Subject: [PATCH 080/106] fix(Asset Repair): Make Error Description
 non-mandatory

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 6c90ca2348..53d72ab68f 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -95,8 +95,7 @@
   {
    "fieldname": "description",
    "fieldtype": "Long Text",
-   "label": "Error Description",
-   "reqd": 1
+   "label": "Error Description"
   },
   {
    "fieldname": "column_break_9",
@@ -262,7 +261,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-19 14:47:25.875814",
+ "modified": "2021-06-19 15:18:10.625833",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 23876c085407866ca40e2248d9a6636d5cdff6da Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 19 Jun 2021 15:23:06 +0530
Subject: [PATCH 081/106] fix(Asset Repair): Prevent some fields from being
 copied on duplicating the doc

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 53d72ab68f..6f9b6863f7 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -75,7 +75,8 @@
    "depends_on": "eval:!doc.__islocal",
    "fieldname": "completion_date",
    "fieldtype": "Datetime",
-   "label": "Completion Date"
+   "label": "Completion Date",
+   "no_copy": 1
   },
   {
    "default": "Pending",
@@ -233,7 +234,8 @@
   {
    "fieldname": "increase_in_asset_life",
    "fieldtype": "Int",
-   "label": "Increase In Asset Life(Months)"
+   "label": "Increase In Asset Life(Months)",
+   "no_copy": 1
   },
   {
    "depends_on": "eval:!doc.__islocal",
@@ -241,6 +243,7 @@
    "fieldtype": "Link",
    "label": "Purchase Invoice",
    "mandatory_depends_on": "eval: doc.repair_status == 'Completed' && doc.repair_cost > 0",
+   "no_copy": 1,
    "options": "Purchase Invoice"
   },
   {
@@ -261,7 +264,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-19 15:18:10.625833",
+ "modified": "2021-06-19 15:20:24.056706",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 740df95c581446991b8c338454f4dc9b6e61dcee Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sun, 20 Jun 2021 17:44:35 +0530
Subject: [PATCH 082/106] fix(Asset Repair): Set completion_date on changing
 repair_status to 'Completed'

---
 erpnext/assets/doctype/asset_repair/asset_repair.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index efa6a9d494..ced3dad1e5 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -28,6 +28,10 @@ frappe.ui.form.on('Asset Repair', {
 				}
 			});
 		}
+
+		if (frm.doc.repair_status == "Completed") {
+			frm.set_value('completion_date', frappe.datetime.now_datetime());
+		}				
 	}
 });
 

From b081d7933262933dd8f31668d4d2ff1d15df5d12 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 21 Jun 2021 14:52:00 +0530
Subject: [PATCH 083/106] fix(Asset Repair): Fix tests

---
 .../doctype/asset_repair/asset_repair.json    |  2 +-
 .../doctype/asset_repair/asset_repair.py      | 58 +++++++++----------
 .../doctype/asset_repair/test_asset_repair.py |  7 ++-
 3 files changed, 32 insertions(+), 35 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 6f9b6863f7..c25216c21b 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -264,7 +264,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-19 15:20:24.056706",
+ "modified": "2021-06-20 17:35:51.075537",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 212af7a930..5fccfb76a5 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -5,74 +5,73 @@
 from __future__ import unicode_literals
 import frappe
 from frappe import _
-from frappe.utils import time_diff_in_hours, getdate, nowdate, add_months, flt, cint
-from frappe.model.document import Document
+from frappe.utils import time_diff_in_hours, getdate, add_months, flt, cint
 from erpnext.accounts.general_ledger import make_gl_entries
 from erpnext.assets.doctype.asset.asset import get_asset_account
 from erpnext.controllers.accounts_controller import AccountsController
 
 class AssetRepair(AccountsController):
 	def validate(self):
-		if self.repair_status == "Completed" and not self.completion_date:
-			self.completion_date = nowdate()
-
+		self.asset_doc = frappe.get_doc('Asset', self.asset)
 		self.update_status()
-		self.set_total_value()		# change later
+		if self.get('stock_items'):
+			self.set_total_value()		# change later
 		self.calculate_total_repair_cost()
 		
 	def update_status(self):
 		if self.repair_status == 'Pending':
 			frappe.db.set_value('Asset', self.asset, 'status', 'Out of Order')
 		else:
-			asset = frappe.get_doc('Asset', self.asset)
-			asset.set_status()
+			self.asset_doc.set_status()
 
 	def set_total_value(self):
-		for item in self.stock_items:
+		for item in self.get('stock_items'):
 			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
 
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost
-		for item in self.stock_items:
-			self.total_repair_cost += item.total_value
+		if self.get('stock_items'):
+			for item in self.get('stock_items'):
+				self.total_repair_cost += item.total_value
 
 	def on_submit(self):
 		self.check_repair_status()
 
-		if self.stock_consumption or self.capitalize_repair_cost:
+		if self.get('stock_consumption') or self.get('capitalize_repair_cost'):
 			self.increase_asset_value()
-		if self.stock_consumption:
+		if self.get('stock_consumption'):
 			self.check_for_stock_items_and_warehouse()
 			self.decrease_stock_quantity()
-		if self.capitalize_repair_cost:
+		if self.get('capitalize_repair_cost'):
 			self.make_gl_entries()
 			if frappe.db.get_value('Asset', self.asset, 'calculate_depreciation') and self.increase_in_asset_life:
 				self.modify_depreciation_schedule()
 
+		self.asset_doc.flags.ignore_validate_update_after_submit = True
+		self.asset_doc.save()
+
 	def check_repair_status(self):
 		if self.repair_status == "Pending":
 			frappe.throw(_("Please update Repair Status."))
 
 	def check_for_stock_items_and_warehouse(self):
-		if not self.stock_items:
+		if not self.get('stock_items'):
 			frappe.throw(_("Please enter Stock Items consumed during the Repair."), title=_("Missing Items"))
 		if not self.warehouse:
 			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during the Repair were taken."), title=_("Missing Warehouse"))
 
 	def increase_asset_value(self):
 		total_value_of_stock_consumed = 0
-		for item in self.stock_items:
-			total_value_of_stock_consumed += item.total_value
+		if self.get('stock_consumption'):
+			for item in self.get('stock_items'):
+				total_value_of_stock_consumed += item.total_value
 
-		asset = frappe.get_doc('Asset', self.asset)
-		asset.flags.ignore_validate_update_after_submit = True
-		if asset.calculate_depreciation:
-			for row in asset.finance_books:
+		if self.asset_doc.calculate_depreciation:
+			for row in self.asset_doc.finance_books:
 				row.value_after_depreciation += total_value_of_stock_consumed
 
 				if self.capitalize_repair_cost:
 					row.value_after_depreciation += self.repair_cost
-		asset.save()
 
 	def decrease_stock_quantity(self):
 		stock_entry = frappe.get_doc({
@@ -81,7 +80,7 @@ class AssetRepair(AccountsController):
 			"company": self.company
 		})
 
-		for stock_item in self.stock_items:
+		for stock_item in self.get('stock_items'):
 			stock_entry.append('items', {
 				"s_warehouse": self.warehouse,
 				"item_code": stock_item.item,
@@ -121,7 +120,7 @@ class AssetRepair(AccountsController):
 			}, item=self)
 		)
 
-		if self.stock_consumption:
+		if self.get('stock_consumption'):
 			# creating GL Entries for each row in Stock Items based on the Stock Entry created for it
 			stock_entry = frappe.get_doc('Stock Entry', self.stock_entry)
 			for item in stock_entry.items:
@@ -158,18 +157,13 @@ class AssetRepair(AccountsController):
 		return gl_entries
 
 	def modify_depreciation_schedule(self):
-		asset = frappe.get_doc('Asset', self.asset)
-		asset.flags.ignore_validate_update_after_submit = True
-		for row in asset.finance_books:
+		for row in self.asset_doc.finance_books:
 			row.total_number_of_depreciations += self.increase_in_asset_life/row.frequency_of_depreciation
 
-			asset.flags.increase_in_asset_life = False
+			self.asset_doc.flags.increase_in_asset_life = False
 			extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
 			if extra_months != 0:
-				self.calculate_last_schedule_date(asset, row, extra_months)
-
-		asset.prepare_depreciation_data()
-		asset.save()
+				self.calculate_last_schedule_date(self.asset_doc, row, extra_months)
 
 	# to help modify depreciation schedule when increase_in_asset_life is not a multiple of frequency_of_depreciation
 	def calculate_last_schedule_date(self, asset, row, extra_months):
diff --git a/erpnext/assets/doctype/asset_repair/test_asset_repair.py b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
index 9c9dd44971..d1b417fd38 100644
--- a/erpnext/assets/doctype/asset_repair/test_asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
@@ -105,7 +105,9 @@ class TestAssetRepair(unittest.TestCase):
 		initial_num_of_depreciations = num_of_depreciations(asset)
 		create_asset_repair(asset= asset, capitalize_repair_cost = 1, submit = 1)
 		asset.reload()
+	
 		self.assertEqual((initial_num_of_depreciations + 1), num_of_depreciations(asset))
+		self.assertEqual(asset.schedules[-1].accumulated_depreciation_amount, asset.finance_books[0].value_after_depreciation)
 
 def num_of_depreciations(asset):
 	return asset.finance_books[0].total_number_of_depreciations
@@ -126,7 +128,8 @@ def create_asset_repair(**args):
 		"asset_name": asset.asset_name,
 		"failure_date": nowdate(),
 		"description": "Test Description",
-		"repair_cost": 0
+		"repair_cost": 0,
+		"company": asset.company
 	})
 
 	if args.stock_consumption:
@@ -142,7 +145,7 @@ def create_asset_repair(**args):
 		asset_repair.save()
 	except frappe.DuplicateEntryError:
 		pass
-
+	
 	if args.submit:
 		asset_repair.repair_status = "Completed"
 		asset_repair.cost_center = "_Test Cost Center - _TC"

From a34bf5edeceff190bf7c264cfa163443ea0aa71e Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 21 Jun 2021 14:55:34 +0530
Subject: [PATCH 084/106] fix: Rename 'Stock Item' to 'Asset Repair Consumed
 Item'

---
 .../assets/doctype/asset_maintenance/asset_maintenance.json   | 4 ++--
 erpnext/assets/doctype/asset_repair/asset_repair.json         | 4 ++--
 .../{stock_item => asset_repair_consumed_item}/__init__.py    | 0
 .../asset_repair_consumed_item.json}                          | 2 +-
 .../asset_repair_consumed_item.py}                            | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)
 rename erpnext/assets/doctype/{stock_item => asset_repair_consumed_item}/__init__.py (100%)
 rename erpnext/assets/doctype/{stock_item/stock_item.json => asset_repair_consumed_item/asset_repair_consumed_item.json} (96%)
 rename erpnext/assets/doctype/{stock_item/stock_item.py => asset_repair_consumed_item/asset_repair_consumed_item.py} (81%)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
index da2fd75451..63a55389d8 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
@@ -126,11 +126,11 @@
    "fieldname": "stock_items",
    "fieldtype": "Table",
    "label": "Stock Items",
-   "options": "Stock Item"
+   "options": "Asset Repair Consumed Item"
   }
  ],
  "links": [],
- "modified": "2021-05-13 05:24:58.480132",
+ "modified": "2021-06-21 14:53:46.041123",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Maintenance",
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index c25216c21b..6a14384f3f 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -169,7 +169,7 @@
    "fieldtype": "Table",
    "label": "Stock Items",
    "mandatory_depends_on": "stock_consumption",
-   "options": "Stock Item"
+   "options": "Asset Repair Consumed Item"
   },
   {
    "fieldname": "section_break_23",
@@ -264,7 +264,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-20 17:35:51.075537",
+ "modified": "2021-06-21 14:53:46.665576",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",
diff --git a/erpnext/assets/doctype/stock_item/__init__.py b/erpnext/assets/doctype/asset_repair_consumed_item/__init__.py
similarity index 100%
rename from erpnext/assets/doctype/stock_item/__init__.py
rename to erpnext/assets/doctype/asset_repair_consumed_item/__init__.py
diff --git a/erpnext/assets/doctype/stock_item/stock_item.json b/erpnext/assets/doctype/asset_repair_consumed_item/asset_repair_consumed_item.json
similarity index 96%
rename from erpnext/assets/doctype/stock_item/stock_item.json
rename to erpnext/assets/doctype/asset_repair_consumed_item/asset_repair_consumed_item.json
index b1f05db395..528f0ec986 100644
--- a/erpnext/assets/doctype/stock_item/stock_item.json
+++ b/erpnext/assets/doctype/asset_repair_consumed_item/asset_repair_consumed_item.json
@@ -46,7 +46,7 @@
  "modified": "2021-05-12 03:19:55.006300",
  "modified_by": "Administrator",
  "module": "Assets",
- "name": "Stock Item",
+ "name": "Asset Repair Consumed Item",
  "owner": "Administrator",
  "permissions": [],
  "sort_field": "modified",
diff --git a/erpnext/assets/doctype/stock_item/stock_item.py b/erpnext/assets/doctype/asset_repair_consumed_item/asset_repair_consumed_item.py
similarity index 81%
rename from erpnext/assets/doctype/stock_item/stock_item.py
rename to erpnext/assets/doctype/asset_repair_consumed_item/asset_repair_consumed_item.py
index 0e3cc3f8ba..fa22a5712f 100644
--- a/erpnext/assets/doctype/stock_item/stock_item.py
+++ b/erpnext/assets/doctype/asset_repair_consumed_item/asset_repair_consumed_item.py
@@ -4,5 +4,5 @@
 # import frappe
 from frappe.model.document import Document
 
-class StockItem(Document):
+class AssetRepairConsumedItem(Document):
 	pass

From f88a13b292334475c6cc889eefafc2ccd556e987 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 21 Jun 2021 15:02:40 +0530
Subject: [PATCH 085/106] fix(Asset Repair): Compute total_value instantly

---
 erpnext/assets/doctype/asset_repair/asset_repair.js | 7 +++++++
 erpnext/assets/doctype/asset_repair/asset_repair.py | 6 ------
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index ced3dad1e5..91bed4fdd0 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -35,6 +35,13 @@ frappe.ui.form.on('Asset Repair', {
 	}
 });
 
+frappe.ui.form.on('Asset Repair Consumed Item', {
+	consumed_quantity: function(frm, cdt, cdn) {
+		var row = locals[cdt][cdn];
+		frappe.model.set_value(cdt, cdn, 'total_value', row.consumed_quantity * row.valuation_rate);
+	},
+});
+
 cur_frm.fields_dict.cost_center.get_query = function(doc) {
 	return {
 		filters: {
diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 5fccfb76a5..79b9a6a2b5 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -14,8 +14,6 @@ class AssetRepair(AccountsController):
 	def validate(self):
 		self.asset_doc = frappe.get_doc('Asset', self.asset)
 		self.update_status()
-		if self.get('stock_items'):
-			self.set_total_value()		# change later
 		self.calculate_total_repair_cost()
 		
 	def update_status(self):
@@ -24,10 +22,6 @@ class AssetRepair(AccountsController):
 		else:
 			self.asset_doc.set_status()
 
-	def set_total_value(self):
-		for item in self.get('stock_items'):
-			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
-
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost
 		if self.get('stock_items'):

From 91a99e0d8973b97743065aa61f62635b13e05256 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 21 Jun 2021 15:22:02 +0530
Subject: [PATCH 086/106] fix(Asset Repair): Increase stock quantity and
 decrease asset value on cancellation

---
 .../doctype/asset_repair/asset_repair.py      | 55 ++++++++++++++++---
 1 file changed, 46 insertions(+), 9 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 79b9a6a2b5..e7b8b45b7e 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -24,9 +24,9 @@ class AssetRepair(AccountsController):
 
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost
-		if self.get('stock_items'):
-			for item in self.get('stock_items'):
-				self.total_repair_cost += item.total_value
+
+		total_value_of_stock_consumed = self.get_total_value_of_stock_consumed()
+		self.total_repair_cost += total_value_of_stock_consumed
 
 	def on_submit(self):
 		self.check_repair_status()
@@ -44,6 +44,14 @@ class AssetRepair(AccountsController):
 		self.asset_doc.flags.ignore_validate_update_after_submit = True
 		self.asset_doc.save()
 
+	def on_cancel(self):
+		if self.get('stock_consumption') or self.get('capitalize_repair_cost'):
+			self.decrease_asset_value()
+		if self.get('stock_consumption'):
+			self.increase_stock_quantity()
+		if self.get('capitalize_repair_cost'):
+			self.make_gl_entries(cancel=True)
+
 	def check_repair_status(self):
 		if self.repair_status == "Pending":
 			frappe.throw(_("Please update Repair Status."))
@@ -55,10 +63,7 @@ class AssetRepair(AccountsController):
 			frappe.throw(_("Please enter Warehouse from which Stock Items consumed during the Repair were taken."), title=_("Missing Warehouse"))
 
 	def increase_asset_value(self):
-		total_value_of_stock_consumed = 0
-		if self.get('stock_consumption'):
-			for item in self.get('stock_items'):
-				total_value_of_stock_consumed += item.total_value
+		total_value_of_stock_consumed = self.get_total_value_of_stock_consumed()
 
 		if self.asset_doc.calculate_depreciation:
 			for row in self.asset_doc.finance_books:
@@ -67,6 +72,24 @@ class AssetRepair(AccountsController):
 				if self.capitalize_repair_cost:
 					row.value_after_depreciation += self.repair_cost
 
+	def decrease_asset_value(self):
+		total_value_of_stock_consumed = self.get_total_value_of_stock_consumed()
+
+		if self.asset_doc.calculate_depreciation:
+			for row in self.asset_doc.finance_books:
+				row.value_after_depreciation -= total_value_of_stock_consumed
+
+				if self.capitalize_repair_cost:
+					row.value_after_depreciation -= self.repair_cost
+		
+	def get_total_value_of_stock_consumed(self):
+		total_value_of_stock_consumed = 0
+		if self.get('stock_consumption'):
+			for item in self.get('stock_items'):
+				total_value_of_stock_consumed += item.total_value
+
+		return total_value_of_stock_consumed
+
 	def decrease_stock_quantity(self):
 		stock_entry = frappe.get_doc({
 			"doctype": "Stock Entry",
@@ -86,8 +109,22 @@ class AssetRepair(AccountsController):
 
 		self.stock_entry = stock_entry.name
 
-	def on_cancel(self):
-		self.make_gl_entries(cancel=True)
+	def increase_stock_quantity(self):
+		stock_entry = frappe.get_doc({
+			"doctype": "Stock Entry",
+			"stock_entry_type": "Material Receipt",
+			"company": self.company
+		})
+
+		for stock_item in self.get('stock_items'):
+			stock_entry.append('items', {
+				"s_warehouse": self.warehouse,
+				"item_code": stock_item.item,
+				"qty": stock_item.consumed_quantity
+			})
+
+		stock_entry.insert()
+		stock_entry.submit()
 
 	def make_gl_entries(self, cancel=False):
 		if flt(self.repair_cost) > 0:

From 5776a962b3d12f25f39c88208e8e10c9897830fd Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Mon, 21 Jun 2021 18:57:11 +0530
Subject: [PATCH 087/106] fix(Asset Repair): Return Depreciation Schedule to
 original state on cancellation

---
 .../doctype/asset_repair/asset_repair.py      | 45 ++++++++++++++++++-
 1 file changed, 43 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index e7b8b45b7e..01b36880be 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -42,15 +42,25 @@ class AssetRepair(AccountsController):
 				self.modify_depreciation_schedule()
 
 		self.asset_doc.flags.ignore_validate_update_after_submit = True
+		self.asset_doc.prepare_depreciation_data()
 		self.asset_doc.save()
 
 	def on_cancel(self):
+		self.asset_doc = frappe.get_doc('Asset', self.asset)
+
 		if self.get('stock_consumption') or self.get('capitalize_repair_cost'):
 			self.decrease_asset_value()
 		if self.get('stock_consumption'):
 			self.increase_stock_quantity()
 		if self.get('capitalize_repair_cost'):
+			self.ignore_linked_doctypes = ('GL Entry', 'Stock Ledger Entry')
 			self.make_gl_entries(cancel=True)
+			if frappe.db.get_value('Asset', self.asset, 'calculate_depreciation') and self.increase_in_asset_life:
+				self.revert_depreciation_schedule_on_cancellation()
+
+		self.asset_doc.flags.ignore_validate_update_after_submit = True
+		self.asset_doc.prepare_depreciation_data()
+		self.asset_doc.save()
 
 	def check_repair_status(self):
 		if self.repair_status == "Pending":
@@ -101,7 +111,8 @@ class AssetRepair(AccountsController):
 			stock_entry.append('items', {
 				"s_warehouse": self.warehouse,
 				"item_code": stock_item.item,
-				"qty": stock_item.consumed_quantity
+				"qty": stock_item.consumed_quantity,
+				"basic_rate": stock_item.valuation_rate
 			})
 
 		stock_entry.insert()
@@ -118,7 +129,7 @@ class AssetRepair(AccountsController):
 
 		for stock_item in self.get('stock_items'):
 			stock_entry.append('items', {
-				"s_warehouse": self.warehouse,
+				"t_warehouse": self.warehouse,
 				"item_code": stock_item.item,
 				"qty": stock_item.consumed_quantity
 			})
@@ -126,6 +137,8 @@ class AssetRepair(AccountsController):
 		stock_entry.insert()
 		stock_entry.submit()
 
+		self.stock_entry = stock_entry.name
+
 	def make_gl_entries(self, cancel=False):
 		if flt(self.repair_cost) > 0:
 			gl_entries = self.get_gl_entries()
@@ -216,6 +229,34 @@ class AssetRepair(AccountsController):
 		if asset.to_date > schedule_date:
 			row.total_number_of_depreciations += 1
 
+	def revert_depreciation_schedule_on_cancellation(self):
+		for row in self.asset_doc.finance_books:
+			row.total_number_of_depreciations -= self.increase_in_asset_life/row.frequency_of_depreciation
+
+			self.asset_doc.flags.increase_in_asset_life = False
+			extra_months = self.increase_in_asset_life % row.frequency_of_depreciation
+			if extra_months != 0:
+				self.calculate_last_schedule_date_before_modification(self.asset_doc, row, extra_months)
+
+	def calculate_last_schedule_date_before_modification(self, asset, row, extra_months):
+		asset.flags.increase_in_asset_life = True
+		number_of_pending_depreciations = cint(row.total_number_of_depreciations) - \
+			cint(asset.number_of_depreciations_booked)
+
+		# the Schedule Date in the final row of the modified Depreciation Schedule
+		last_schedule_date = asset.schedules[len(asset.schedules)-1].schedule_date
+
+		# the Schedule Date in the final row of the original Depreciation Schedule
+		asset.to_date = add_months(last_schedule_date, -extra_months)
+
+		# the latest possible date at which the depreciation can occur, without decreasing the Total Number of Depreciations
+		# if depreciations happen yearly and the Depreciation Posting Date is 01-01-2020, this could be 01-01-2021, 01-01-2022...
+		schedule_date = add_months(row.depreciation_start_date,
+			(number_of_pending_depreciations - 1) * cint(row.frequency_of_depreciation))
+
+		if asset.to_date < schedule_date:
+			row.total_number_of_depreciations -= 1
+
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):
 	downtime = time_diff_in_hours(completion_date, failure_date)

From 3b0eac79bf6b302d2b4800c2f6333828f00d4bff Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 22 Jun 2021 16:26:09 +0530
Subject: [PATCH 088/106] fix(Asset Repair): Compute total_value

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 01b36880be..342a8861e6 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -14,6 +14,9 @@ class AssetRepair(AccountsController):
 	def validate(self):
 		self.asset_doc = frappe.get_doc('Asset', self.asset)
 		self.update_status()
+
+		if self.get('stock_items'):
+ 			self.set_total_value()
 		self.calculate_total_repair_cost()
 		
 	def update_status(self):
@@ -22,6 +25,10 @@ class AssetRepair(AccountsController):
 		else:
 			self.asset_doc.set_status()
 
+	def set_total_value(self):
+ 		for item in self.get('stock_items'):
+ 			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
+
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost
 

From 3a44d888661b0d3b81e475307484dc09ba4b2c78 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 22 Jun 2021 16:28:29 +0530
Subject: [PATCH 089/106] fix(Asset): Fix tests for Asset Repair

---
 erpnext/assets/doctype/asset/test_asset.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/test_asset.py b/erpnext/assets/doctype/asset/test_asset.py
index 29fbc9f15d..f3667c7b95 100644
--- a/erpnext/assets/doctype/asset/test_asset.py
+++ b/erpnext/assets/doctype/asset/test_asset.py
@@ -707,7 +707,7 @@ def create_asset(**args):
 		"available_for_use_date": "2020-06-06",
 		"location": "Test Location",
 		"asset_owner": "Company",
-		"is_existing_asset": args.is_existing_asset or 0
+		"is_existing_asset": 1
 	})
 
 	if asset.calculate_depreciation:

From 2ea325c0c3fbdc97e2aa1427f565ee176f543a4c Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 22 Jun 2021 16:33:10 +0530
Subject: [PATCH 090/106] fix(Asset Repair): Revert Stock Entry on cancellation

---
 .../doctype/asset_repair/asset_repair.py      | 21 +++----------------
 1 file changed, 3 insertions(+), 18 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 342a8861e6..4261c535af 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -125,26 +125,11 @@ class AssetRepair(AccountsController):
 		stock_entry.insert()
 		stock_entry.submit()
 
-		self.stock_entry = stock_entry.name
+		self.db_set('stock_entry', stock_entry.name)
 
 	def increase_stock_quantity(self):
-		stock_entry = frappe.get_doc({
-			"doctype": "Stock Entry",
-			"stock_entry_type": "Material Receipt",
-			"company": self.company
-		})
-
-		for stock_item in self.get('stock_items'):
-			stock_entry.append('items', {
-				"t_warehouse": self.warehouse,
-				"item_code": stock_item.item,
-				"qty": stock_item.consumed_quantity
-			})
-
-		stock_entry.insert()
-		stock_entry.submit()
-
-		self.stock_entry = stock_entry.name
+		stock_entry = frappe.get_doc('Stock Entry', self.stock_entry)
+		stock_entry.cancel()
 
 	def make_gl_entries(self, cancel=False):
 		if flt(self.repair_cost) > 0:

From 859c8c92f7a73f4b85fd7bea244169e69fda3b24 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 22 Jun 2021 17:16:12 +0530
Subject: [PATCH 091/106] fix: Remove changes made to Asset Maintenance

---
 .../asset_maintenance/asset_maintenance.js    |  3 --
 .../asset_maintenance/asset_maintenance.json  | 31 +---------------
 .../asset_maintenance/asset_maintenance.py    | 37 +------------------
 3 files changed, 3 insertions(+), 68 deletions(-)

diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
index 19393b7e9d..70b8654509 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.js
@@ -30,10 +30,7 @@ frappe.ui.form.on('Asset Maintenance', {
 		if(!frm.is_new()) {
 			frm.trigger('make_dashboard');
 		}
-
-		frm.toggle_display(['stock_consumption_details_section'], frm.doc.stock_consumption);
 	},
-
 	make_dashboard: (frm) => {
 		if(!frm.is_new()) {
 			frappe.call({
diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
index 63a55389d8..c0c2566fe2 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.json
@@ -12,17 +12,13 @@
   "column_break_3",
   "item_code",
   "item_name",
-  "stock_consumption",
   "section_break_6",
   "maintenance_team",
   "column_break_9",
   "maintenance_manager",
   "maintenance_manager_name",
   "section_break_8",
-  "asset_maintenance_tasks",
-  "stock_consumption_details_section",
-  "warehouse",
-  "stock_items"
+  "asset_maintenance_tasks"
  ],
  "fields": [
   {
@@ -104,33 +100,10 @@
    "label": "Maintenance Tasks",
    "options": "Asset Maintenance Task",
    "reqd": 1
-  },
-  {
-   "default": "0",
-   "fieldname": "stock_consumption",
-   "fieldtype": "Check",
-   "label": "Stock Consumed During Maintenance"
-  },
-  {
-   "fieldname": "stock_consumption_details_section",
-   "fieldtype": "Section Break",
-   "label": "Stock Consumption Details"
-  },
-  {
-   "fieldname": "warehouse",
-   "fieldtype": "Link",
-   "label": "Warehouse",
-   "options": "Warehouse"
-  },
-  {
-   "fieldname": "stock_items",
-   "fieldtype": "Table",
-   "label": "Stock Items",
-   "options": "Asset Repair Consumed Item"
   }
  ],
  "links": [],
- "modified": "2021-06-21 14:53:46.041123",
+ "modified": "2020-05-28 20:28:32.993823",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Maintenance",
diff --git a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
index e3e654c398..a506deec93 100644
--- a/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
+++ b/erpnext/assets/doctype/asset_maintenance/asset_maintenance.py
@@ -19,45 +19,10 @@ class AssetMaintenance(Document):
 			if not task.assign_to and self.docstatus == 0:
 				throw(_("Row #{}: Please asign task to a member.").format(task.idx))
 
-		if self.stock_consumption:
-			self.check_for_stock_items_and_warehouse()
-			self.increase_asset_value()
-			self.decrease_stock_quantity()
-
 	def on_update(self):
 		for task in self.get('asset_maintenance_tasks'):
 			assign_tasks(self.name, task.assign_to, task.maintenance_task, task.next_due_date)
-		self.sync_maintenance_tasks()		
-
-	def check_for_stock_items_and_warehouse(self):
-		if self.stock_consumption:
-			if not self.stock_items:
-				frappe.throw(_("Please enter Stock Items consumed during Asset Maintenance."))
-			if not self.warehouse:
-				frappe.throw(_("Please enter Warehouse from which Stock Items consumed during Asset Maintenance were taken."))
-
-	def increase_asset_value(self):
-		asset_value = frappe.db.get_value('Asset', self.asset_name, 'asset_value')
-		for item in self.stock_items:
-			asset_value += item.total_value
-
-		frappe.db.set_value('Asset', self.asset_name, 'asset_value', asset_value)
-
-	def decrease_stock_quantity(self):
-		stock_entry = frappe.get_doc({
-			"doctype": "Stock Entry",
-			"stock_entry_type": "Material Issue"
-		})
-
-		for stock_item in self.stock_items:
-			stock_entry.append('items', {
-				"s_warehouse": self.warehouse,
-				"item_code": stock_item.item,
-				"qty": stock_item.consumed_quantity
-			})
-
-		stock_entry.insert()
-		stock_entry.submit()
+		self.sync_maintenance_tasks()
 
 	def sync_maintenance_tasks(self):
 		tasks_names = []

From 2e4596540516a2f053f47e0902060b7e235b5d42 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Tue, 22 Jun 2021 17:24:14 +0530
Subject: [PATCH 092/106] fix(Asset Repair): Fix Sider issues

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 4261c535af..6054258ea6 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -16,7 +16,7 @@ class AssetRepair(AccountsController):
 		self.update_status()
 
 		if self.get('stock_items'):
- 			self.set_total_value()
+			self.set_total_value()
 		self.calculate_total_repair_cost()
 		
 	def update_status(self):
@@ -26,8 +26,8 @@ class AssetRepair(AccountsController):
 			self.asset_doc.set_status()
 
 	def set_total_value(self):
- 		for item in self.get('stock_items'):
- 			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
+		for item in self.get('stock_items'):
+			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
 
 	def calculate_total_repair_cost(self):
 		self.total_repair_cost = self.repair_cost

From 80399802c62794f5002937eb291525e3934de5fc Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 23 Jun 2021 13:26:47 +0530
Subject: [PATCH 093/106] fix(Asset Repair): Replace asset_value with
 value_after_depreciation in tests

---
 .../doctype/asset_repair/test_asset_repair.py     | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/test_asset_repair.py b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
index d1b417fd38..52a960e850 100644
--- a/erpnext/assets/doctype/asset_repair/test_asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
@@ -74,21 +74,21 @@ class TestAssetRepair(unittest.TestCase):
 		self.assertEqual(stock_entry.items[0].qty, asset_repair.stock_items[0].consumed_quantity)
 
 	def test_increase_in_asset_value_due_to_stock_consumption(self):
-		asset = create_asset()
-		initial_asset_value = asset.asset_value
+		asset = create_asset(calculate_depreciation = 1)
+		initial_asset_value = get_asset_value(asset)
 		asset_repair = create_asset_repair(asset= asset, stock_consumption = 1, submit = 1)
 		asset.reload()
 
-		increase_in_asset_value = asset.asset_value - initial_asset_value
+		increase_in_asset_value = get_asset_value(asset) - initial_asset_value
 		self.assertEqual(asset_repair.stock_items[0].total_value, increase_in_asset_value)
 
 	def test_increase_in_asset_value_due_to_repair_cost_capitalisation(self):
-		asset = create_asset()
-		initial_asset_value = asset.asset_value
+		asset = create_asset(calculate_depreciation = 1)
+		initial_asset_value = get_asset_value(asset)
 		asset_repair = create_asset_repair(asset= asset, capitalize_repair_cost = 1, submit = 1)
 		asset.reload()
 
-		increase_in_asset_value = asset.asset_value - initial_asset_value
+		increase_in_asset_value = get_asset_value(asset) - initial_asset_value
 		self.assertEqual(asset_repair.repair_cost, increase_in_asset_value)
 
 	def test_purchase_invoice(self):
@@ -109,6 +109,9 @@ class TestAssetRepair(unittest.TestCase):
 		self.assertEqual((initial_num_of_depreciations + 1), num_of_depreciations(asset))
 		self.assertEqual(asset.schedules[-1].accumulated_depreciation_amount, asset.finance_books[0].value_after_depreciation)
 
+def get_asset_value(asset):
+	return asset.finance_books[0].value_after_depreciation
+
 def num_of_depreciations(asset):
 	return asset.finance_books[0].total_number_of_depreciations
 

From 2f5f9d8566a57fd3b9781cfbabfe71583dafea62 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 23 Jun 2021 22:19:05 +0530
Subject: [PATCH 094/106] fix(Asset): Fix value_after_depreciation calculation

---
 erpnext/assets/doctype/asset/asset.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 27d21e2542..29379657a1 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -182,10 +182,10 @@ class Asset(AccountsController):
 			# value_after_depreciation - current Asset value
 			if d.value_after_depreciation:
 				value_after_depreciation = (flt(d.value_after_depreciation) -
-					flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
+					flt(self.opening_accumulated_depreciation)) 
 			else:
 				value_after_depreciation = (flt(self.gross_purchase_amount) -
-					flt(self.opening_accumulated_depreciation)) - flt(d.expected_value_after_useful_life)
+					flt(self.opening_accumulated_depreciation)) 
 
 			d.value_after_depreciation = value_after_depreciation
 

From dce21137364b86e997a2d000c00c25b4d2c78684 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Wed, 23 Jun 2021 22:26:45 +0530
Subject: [PATCH 095/106] fix(Asset Repair): Remove test that's no longer
 necessary

---
 erpnext/assets/doctype/asset_repair/test_asset_repair.py | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/test_asset_repair.py b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
index 52a960e850..b3d78b3bfb 100644
--- a/erpnext/assets/doctype/asset_repair/test_asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
@@ -13,12 +13,6 @@ class TestAssetRepair(unittest.TestCase):
 		create_asset_data()
 		frappe.db.sql("delete from `tabTax Rule`")
 
-	def test_completion_date(self):
-		asset_repair = create_asset_repair()
-		asset_repair.repair_status = "Completed"
-		asset_repair.save()
-		self.assertTrue(asset_repair.completion_date)
-
 	def test_update_status(self):
 		asset = create_asset()
 		initial_status = asset.status

From 60a44ae1e63dafa8d9b15b1d11a479f74c196a73 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 12:44:13 +0530
Subject: [PATCH 096/106] fix(Asset): Fix test

---
 erpnext/assets/doctype/asset/test_asset.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/test_asset.py b/erpnext/assets/doctype/asset/test_asset.py
index f3667c7b95..32bdb5224a 100644
--- a/erpnext/assets/doctype/asset/test_asset.py
+++ b/erpnext/assets/doctype/asset/test_asset.py
@@ -125,7 +125,6 @@ class TestAsset(unittest.TestCase):
 			"frequency_of_depreciation": 12,
 			"depreciation_start_date": "2030-12-31"
 		})
-		asset.insert()
 		self.assertEqual(asset.status, "Draft")
 		asset.save()
 		expected_schedules = [

From 550fe8ae395e6bf4b5598ed01fa81229fc8955ce Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 13:22:26 +0530
Subject: [PATCH 097/106] fix(Asset): Remove redundant code

---
 erpnext/assets/doctype/asset/test_asset.py | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/erpnext/assets/doctype/asset/test_asset.py b/erpnext/assets/doctype/asset/test_asset.py
index 32bdb5224a..59fbe3b030 100644
--- a/erpnext/assets/doctype/asset/test_asset.py
+++ b/erpnext/assets/doctype/asset/test_asset.py
@@ -153,9 +153,8 @@ class TestAsset(unittest.TestCase):
 			"frequency_of_depreciation": 12,
 			"depreciation_start_date": '2030-12-31'
 		})
-		asset.insert()
-		self.assertEqual(asset.status, "Draft")
 		asset.save()
+		self.assertEqual(asset.status, "Draft")
 
 		expected_schedules = [
 			['2030-12-31', 66667.00, 66667.00],
@@ -184,7 +183,7 @@ class TestAsset(unittest.TestCase):
 			"frequency_of_depreciation": 12,
 			"depreciation_start_date": "2030-12-31"
 		})
-		asset.insert()
+		asset.save()
 		self.assertEqual(asset.status, "Draft")
 
 		expected_schedules = [
@@ -215,7 +214,6 @@ class TestAsset(unittest.TestCase):
 			"depreciation_start_date": "2030-12-31"
 		})
 
-		asset.insert()
 		asset.save()
 
 		expected_schedules = [
@@ -246,7 +244,6 @@ class TestAsset(unittest.TestCase):
 			"frequency_of_depreciation": 10,
 			"depreciation_start_date": "2020-12-31"
 		})
-		asset.insert()
 		asset.submit()
 		asset.load_from_db()
 		self.assertEqual(asset.status, "Submitted")
@@ -349,7 +346,6 @@ class TestAsset(unittest.TestCase):
 			"frequency_of_depreciation": 10,
 			"depreciation_start_date": "2020-12-31"
 		})
-		asset.insert()
 		asset.submit()
 		post_depreciation_entries(date="2021-01-01")
 
@@ -379,7 +375,6 @@ class TestAsset(unittest.TestCase):
 			"total_number_of_depreciations": 10,
 			"frequency_of_depreciation": 1
 		})
-		asset.insert()
 		asset.submit()
 
 		post_depreciation_entries(date=add_months('2020-01-01', 4))
@@ -423,7 +418,6 @@ class TestAsset(unittest.TestCase):
 			"frequency_of_depreciation": 10,
 			"depreciation_start_date": "2020-12-31"
 		})
-		asset.insert()
 		asset.submit()
 		post_depreciation_entries(date="2021-01-01")
 
@@ -467,7 +461,7 @@ class TestAsset(unittest.TestCase):
 			"total_number_of_depreciations": 3,
 			"frequency_of_depreciation": 10
 		})
-		asset.insert()
+		asset.save()
 		accumulated_depreciation_after_full_schedule = \
 			max(d.accumulated_depreciation_amount for d in asset.get("schedules"))
 

From 2fdb923953b4ed84b55771fbc07dc1556d7ac1f1 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 14:56:34 +0530
Subject: [PATCH 098/106] fix(Asset Repair): Change controller hooks

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 6054258ea6..64c51fd8c3 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -35,7 +35,7 @@ class AssetRepair(AccountsController):
 		total_value_of_stock_consumed = self.get_total_value_of_stock_consumed()
 		self.total_repair_cost += total_value_of_stock_consumed
 
-	def on_submit(self):
+	def before_submit(self):
 		self.check_repair_status()
 
 		if self.get('stock_consumption') or self.get('capitalize_repair_cost'):
@@ -52,7 +52,7 @@ class AssetRepair(AccountsController):
 		self.asset_doc.prepare_depreciation_data()
 		self.asset_doc.save()
 
-	def on_cancel(self):
+	def before_cancel(self):
 		self.asset_doc = frappe.get_doc('Asset', self.asset)
 
 		if self.get('stock_consumption') or self.get('capitalize_repair_cost'):

From e21d44c5c3b0ba8ef2252aba8fb2714e078bb159 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 15:04:44 +0530
Subject: [PATCH 099/106] fix(Asset): Remove to_date field

---
 erpnext/assets/doctype/asset/asset.json | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/erpnext/assets/doctype/asset/asset.json b/erpnext/assets/doctype/asset/asset.json
index d77eb10418..de060757e2 100644
--- a/erpnext/assets/doctype/asset/asset.json
+++ b/erpnext/assets/doctype/asset/asset.json
@@ -53,7 +53,6 @@
   "next_depreciation_date",
   "section_break_14",
   "schedules",
-  "to_date",
   "insurance_details",
   "policy_number",
   "insurer",
@@ -481,12 +480,6 @@
    "fieldname": "section_break_36",
    "fieldtype": "Section Break",
    "label": "Finance Books"
-  },
-  {
-   "fieldname": "to_date",
-   "fieldtype": "Date",
-   "hidden": 1,
-   "label": "To Date"
   }
  ],
  "idx": 72,
@@ -509,7 +502,7 @@
    "link_fieldname": "asset"
   }
  ],
- "modified": "2021-06-19 13:56:58.450182",
+ "modified": "2021-06-24 14:58:51.097908",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset",

From f4e8e7d933fc7d96d22da7e23fe22f9bb6cba200 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 22:18:59 +0530
Subject: [PATCH 100/106] fix(Asset): Remove extra tabs

---
 erpnext/assets/doctype/asset/asset.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 29379657a1..95e3c8ad71 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -196,7 +196,7 @@ class Asset(AccountsController):
 
 			if has_pro_rata:
 				number_of_pending_depreciations += 1
-			
+
 			skip_row = False
 			for n in range(start, number_of_pending_depreciations):
 				# If depreciation is already completed (for double declining balance)

From 289b449f2bf11e79fef5501bc5e012f0e589e416 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 22:21:08 +0530
Subject: [PATCH 101/106] fix(Asset): Add comment

---
 erpnext/assets/doctype/asset/asset.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/erpnext/assets/doctype/asset/asset.py b/erpnext/assets/doctype/asset/asset.py
index 95e3c8ad71..66f0bdcd58 100644
--- a/erpnext/assets/doctype/asset/asset.py
+++ b/erpnext/assets/doctype/asset/asset.py
@@ -224,6 +224,7 @@ class Asset(AccountsController):
 				# For last row
 				elif has_pro_rata and n == cint(number_of_pending_depreciations) - 1:
 					if not self.flags.increase_in_asset_life:
+						# In case of increase_in_asset_life, the self.to_date is already set on asset_repair submission
 						self.to_date = add_months(self.available_for_use_date,
 							n * cint(d.frequency_of_depreciation))
 

From b44e4121da48debd9b0481d558625afca781e150 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 22:25:45 +0530
Subject: [PATCH 102/106] fix(Asset Repair): Move filters for cost_center,
 warehouse and project to setup

---
 .../doctype/asset_repair/asset_repair.js      | 56 ++++++++++---------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js
index 91bed4fdd0..1cebfff66e 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.js
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.js
@@ -2,6 +2,34 @@
 // For license information, please see license.txt
 
 frappe.ui.form.on('Asset Repair', {
+	setup: function(frm) {
+		frm.fields_dict.cost_center.get_query = function(doc) {
+			return {
+				filters: {
+					'is_group': 0,
+					'company': doc.company
+				}
+			};
+		};
+
+		frm.fields_dict.project.get_query = function(doc) {
+			return {
+				filters: {
+					'company': doc.company
+				}
+			};
+		};
+
+		frm.fields_dict.warehouse.get_query = function(doc) {
+			return {
+				filters: {
+					'is_group': 0,
+					'company': doc.company
+				}
+			};
+		};
+	},
+
 	refresh: function(frm) {
 		if (frm.doc.docstatus) {
 			frm.add_custom_button("View General Ledger", function() {
@@ -40,30 +68,4 @@ frappe.ui.form.on('Asset Repair Consumed Item', {
 		var row = locals[cdt][cdn];
 		frappe.model.set_value(cdt, cdn, 'total_value', row.consumed_quantity * row.valuation_rate);
 	},
-});
-
-cur_frm.fields_dict.cost_center.get_query = function(doc) {
-	return {
-		filters: {
-			'is_group': 0,
-			'company': doc.company
-		}
-	};
-};
-
-cur_frm.fields_dict.project.get_query = function(doc) {
-	return {
-		filters: {
-			'company': doc.company
-		}
-	};
-};
-
-cur_frm.fields_dict.warehouse.get_query = function(doc) {
-	return {
-		filters: {
-			'is_group': 0,
-			'company': doc.company
-		}
-	};
-};
\ No newline at end of file
+});
\ No newline at end of file

From c73e137bfa934de13b2512a44243ba810f123553 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 22:27:30 +0530
Subject: [PATCH 103/106] fix(Asset Repair): Edit description for
 total_repair_cost

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 6a14384f3f..0651deb2ac 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -212,7 +212,7 @@
   },
   {
    "depends_on": "eval: doc.stock_consumption && doc.total_repair_cost > 0",
-   "description": "Sum of Repair Cost and the total value of all Stock Items consumed during the repair.",
+   "description": "Sum of Repair Cost and Value of Consumed Stock Items.",
    "fieldname": "total_repair_cost",
    "fieldtype": "Currency",
    "label": "Total Repair Cost",

From dcb8a2895cee137054656487005706007bf6691f Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Thu, 24 Jun 2021 22:30:08 +0530
Subject: [PATCH 104/106] fix(Asset Repair): Simplify code for Asset Repair
 creation in tests

---
 erpnext/assets/doctype/asset_repair/test_asset_repair.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/test_asset_repair.py b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
index b3d78b3bfb..30bbb37851 100644
--- a/erpnext/assets/doctype/asset_repair/test_asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/test_asset_repair.py
@@ -138,10 +138,7 @@ def create_asset_repair(**args):
 			"consumed_quantity": args.qty or 1
 		})
 
-	try:
-		asset_repair.save()
-	except frappe.DuplicateEntryError:
-		pass
+	asset_repair.insert(ignore_if_duplicate=True)
 	
 	if args.submit:
 		asset_repair.repair_status = "Completed"

From ca2dbcec593ba3d1845d038721790420db114189 Mon Sep 17 00:00:00 2001
From: GangaManoj <ganga.manoj98@gmail.com>
Date: Sat, 26 Jun 2021 01:32:17 +0530
Subject: [PATCH 105/106] fix(Asset Repair): Rearrange fields

---
 erpnext/assets/doctype/asset_repair/asset_repair.json | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.json b/erpnext/assets/doctype/asset_repair/asset_repair.json
index 0651deb2ac..ba3189887c 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.json
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.json
@@ -8,10 +8,10 @@
  "engine": "InnoDB",
  "field_order": [
   "asset",
-  "naming_series",
+  "company",
   "column_break_2",
   "asset_name",
-  "company",
+  "naming_series",
   "section_break_5",
   "failure_date",
   "repair_status",
@@ -264,7 +264,7 @@
  "index_web_pages_for_search": 1,
  "is_submittable": 1,
  "links": [],
- "modified": "2021-06-21 14:53:46.665576",
+ "modified": "2021-06-25 13:14:38.307723",
  "modified_by": "Administrator",
  "module": "Assets",
  "name": "Asset Repair",

From 40c90d03a4e583ff374e180b3abbf95b31f74fed Mon Sep 17 00:00:00 2001
From: Saqib <nextchamp.saqib@gmail.com>
Date: Mon, 28 Jun 2021 11:42:28 +0530
Subject: [PATCH 106/106] fix(Asset Repair): cancellation

---
 erpnext/assets/doctype/asset_repair/asset_repair.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py
index 64c51fd8c3..d32fdf7054 100644
--- a/erpnext/assets/doctype/asset_repair/asset_repair.py
+++ b/erpnext/assets/doctype/asset_repair/asset_repair.py
@@ -30,7 +30,7 @@ class AssetRepair(AccountsController):
 			item.total_value = flt(item.valuation_rate) * flt(item.consumed_quantity)
 
 	def calculate_total_repair_cost(self):
-		self.total_repair_cost = self.repair_cost
+		self.total_repair_cost = flt(self.repair_cost)
 
 		total_value_of_stock_consumed = self.get_total_value_of_stock_consumed()
 		self.total_repair_cost += total_value_of_stock_consumed
@@ -129,6 +129,7 @@ class AssetRepair(AccountsController):
 
 	def increase_stock_quantity(self):
 		stock_entry = frappe.get_doc('Stock Entry', self.stock_entry)
+		stock_entry.flags.ignore_links = True
 		stock_entry.cancel()
 
 	def make_gl_entries(self, cancel=False):
@@ -252,4 +253,4 @@ class AssetRepair(AccountsController):
 @frappe.whitelist()
 def get_downtime(failure_date, completion_date):
 	downtime = time_diff_in_hours(completion_date, failure_date)
-	return round(downtime, 2)
\ No newline at end of file
+	return round(downtime, 2)