From acf489c5ef5138ce792fc6cb08db437dd1944fc0 Mon Sep 17 00:00:00 2001
From: Rushabh Mehta <rmehta@gmail.com>
Date: Thu, 16 Apr 2015 15:35:49 +0530
Subject: [PATCH] [hot] [fix] journal entry

---
 .../doctype/journal_entry/journal_entry.js    | 60 ++++++++++---------
 1 file changed, 31 insertions(+), 29 deletions(-)

diff --git a/erpnext/accounts/doctype/journal_entry/journal_entry.js b/erpnext/accounts/doctype/journal_entry/journal_entry.js
index ec632ecc957..d7e5ee0b27d 100644
--- a/erpnext/accounts/doctype/journal_entry/journal_entry.js
+++ b/erpnext/accounts/doctype/journal_entry/journal_entry.js
@@ -138,6 +138,8 @@ cur_frm.script_manager.make(erpnext.accounts.JournalVoucher);
 cur_frm.cscript.refresh = function(doc) {
 	cur_frm.cscript.is_opening(doc)
 	erpnext.toggle_naming_series();
+	cur_frm.cscript.voucher_type();
+
 	if(doc.docstatus==1) {
 		cur_frm.add_custom_button(__('View Ledger'), function() {
 			frappe.route_options = {
@@ -235,37 +237,37 @@ cur_frm.cscript.voucher_type = function(doc, cdt, cdn) {
 		refresh_field("accounts");
 	}
 
-	if(in_list(["Bank Entry", "Cash Entry"], doc.voucher_type)) {
-		cur_frm.clear_table("accounts");
-		return frappe.call({
-			type: "GET",
-			method: "erpnext.accounts.doctype.journal_entry.journal_entry.get_default_bank_cash_account",
-			args: {
-				"voucher_type": doc.voucher_type,
-				"company": doc.company
-			},
-			callback: function(r) {
-				if(r.message) {
-					update_jv_details(doc, [r.message]);
+	if(!(doc.accounts || []).length) {
+		if(in_list(["Bank Entry", "Cash Entry"], doc.voucher_type)) {
+			return frappe.call({
+				type: "GET",
+				method: "erpnext.accounts.doctype.journal_entry.journal_entry.get_default_bank_cash_account",
+				args: {
+					"voucher_type": doc.voucher_type,
+					"company": doc.company
+				},
+				callback: function(r) {
+					if(r.message) {
+						update_jv_details(doc, [r.message]);
+					}
 				}
-			}
-		})
-	} else if(doc.voucher_type=="Opening Entry") {
-		cur_frm.clear_table("accounts");
-		return frappe.call({
-			type:"GET",
-			method: "erpnext.accounts.doctype.journal_entry.journal_entry.get_opening_accounts",
-			args: {
-				"company": doc.company
-			},
-			callback: function(r) {
-				frappe.model.clear_table(doc, "accounts");
-				if(r.message) {
-					update_jv_details(doc, r.message);
+			})
+		} else if(doc.voucher_type=="Opening Entry") {
+			return frappe.call({
+				type:"GET",
+				method: "erpnext.accounts.doctype.journal_entry.journal_entry.get_opening_accounts",
+				args: {
+					"company": doc.company
+				},
+				callback: function(r) {
+					frappe.model.clear_table(doc, "accounts");
+					if(r.message) {
+						update_jv_details(doc, r.message);
+					}
+					cur_frm.set_value("is_opening", "Yes")
 				}
-				cur_frm.set_value("is_opening", "Yes")
-			}
-		})
+			})
+		}
 	}
 }