From 3dfe8541087d56ca191f29201ff842387612e214 Mon Sep 17 00:00:00 2001
From: Nabin Hait <nabinhait@gmail.com>
Date: Tue, 11 Mar 2014 17:54:33 +0530
Subject: [PATCH] Rename duplicate account

---
 .../chart_of_accounts/chart_of_accounts.py    | 58 +++++++++++++------
 1 file changed, 40 insertions(+), 18 deletions(-)

diff --git a/erpnext/accounts/doctype/chart_of_accounts/chart_of_accounts.py b/erpnext/accounts/doctype/chart_of_accounts/chart_of_accounts.py
index faf193a107..819adb489f 100644
--- a/erpnext/accounts/doctype/chart_of_accounts/chart_of_accounts.py
+++ b/erpnext/accounts/doctype/chart_of_accounts/chart_of_accounts.py
@@ -3,32 +3,54 @@
 
 from __future__ import unicode_literals
 import frappe, os, json
+from frappe.utils import cstr
+from unidecode import unidecode
+
 
 class DocType:
 	def __init__(self, d, dl):
 		self.doc, self.doclist = d, dl
 		
 	def create_accounts(self, company):
+		chart = {}
 		with open(os.path.join(os.path.dirname(__file__), "charts", 
 			self.doc.source_file), "r") as f:
 			chart = json.loads(f.read())
-
-		def _import_accounts(children, parent):
-			for child in children:
-				print child.get("name"), parent
-				account = frappe.bean({
-					"doctype": "Account",
-					"account_name": child.get("name"),
-					"company": company,
-					"parent_account": parent,
-					"group_or_ledger": "Group" if child.get("children") else "Ledger",
-					"root_type": child.get("root_type"),
-					"is_pl_account": "Yes" if child.get("root_type") in ["Expense", "Income"] \
-						else "No",
-					"account_type": child.get("account_type")
-				}).insert()
 			
-				if child.get("children"):
-					_import_accounts(child.get("children"), account.doc.name)
+		if chart:
+			accounts = []
+			def _import_accounts(children, parent):
+				for child in children:
+					account_name = child.get("name")
+					account_name_in_db = unidecode(account_name.strip().lower())
+					
+					if account_name_in_db in accounts:
+						count = accounts.count(account_name_in_db)
+						account_name = account_name + " " + cstr(count)
+											
+					account = frappe.bean({
+						"doctype": "Account",
+						"account_name": account_name,
+						"company": company,
+						"parent_account": parent,
+						"group_or_ledger": "Group" if child.get("children") else "Ledger",
+						"root_type": child.get("root_type"),
+						"is_pl_account": "Yes" if child.get("root_type") in ["Expense", "Income"] \
+							else "No",
+						"account_type": child.get("account_type")
+					}).insert()
+				
+					accounts.append(account_name_in_db)
+					# print account.doc.lft, account.doc.rgt, account.doc.root_type
 			
-		_import_accounts(chart.get("root").get("children"), None)
\ No newline at end of file
+					if child.get("children"):
+						_import_accounts(child.get("children"), account.doc.name)
+			
+			_import_accounts(chart.get("root").get("children"), None)
+		
+			# set root_type from parent or child if not set
+			# root_types = frappe.db.sql("""select lft, rgt, distinct root_type from tabAccount 
+			# 	where ifnull(root_type, '') != '' order by lft desc""")
+			# print root_types
+		
+		
\ No newline at end of file