From c6d17a6989a66268464433f514e95b14da4ac384 Mon Sep 17 00:00:00 2001
From: Anand Doshi <anand@erpnext.com>
Date: Thu, 5 Sep 2013 18:18:40 +0530
Subject: [PATCH] [fix] [minor] set price list currency and conversion rate

---
 controllers/accounts_controller.py | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/controllers/accounts_controller.py b/controllers/accounts_controller.py
index 4ee7e53e151..eb71f21ce73 100644
--- a/controllers/accounts_controller.py
+++ b/controllers/accounts_controller.py
@@ -56,23 +56,23 @@ class AccountsController(TransactionBase):
 	def set_price_list_currency(self, buying_or_selling):
 		company_currency = get_company_currency(self.doc.company)
 		fieldname = buying_or_selling.lower() + "_price_list"
+		
 		# TODO - change this, since price list now has only one currency allowed
-		if self.meta.get_field(fieldname) and self.doc.fields.get(fieldname) and \
-			not self.doc.price_list_currency:
+		if self.meta.get_field(fieldname) and self.doc.fields.get(fieldname):
+			if not self.doc.price_list_currency:
 				self.doc.fields.update(get_price_list_currency(self.doc.fields.get(fieldname)))
 				
-				if self.doc.price_list_currency:
-					if not self.doc.plc_conversion_rate:
-						if self.doc.price_list_currency == company_currency:
-							self.doc.plc_conversion_rate = 1.0
-						else:
-							exchange = self.doc.price_list_currency + "-" + company_currency
-							self.doc.plc_conversion_rate = flt(webnotes.conn.get_value("Currency Exchange",
-								exchange, "exchange_rate"))
-						
-					if not self.doc.currency:
-						self.doc.currency = self.doc.price_list_currency
-						self.doc.conversion_rate = self.doc.plc_conversion_rate
+			if self.doc.price_list_currency:
+				if self.doc.price_list_currency == company_currency:
+					self.doc.plc_conversion_rate = 1.0
+				elif not self.doc.plc_conversion_rate:
+					exchange = self.doc.price_list_currency + "-" + company_currency
+					self.doc.plc_conversion_rate = flt(webnotes.conn.get_value("Currency Exchange",
+						exchange, "exchange_rate"))
+					
+				if not self.doc.currency:
+					self.doc.currency = self.doc.price_list_currency
+					self.doc.conversion_rate = self.doc.plc_conversion_rate
 						
 		if self.meta.get_field("currency"):
 			if self.doc.currency and self.doc.currency != company_currency:
@@ -82,7 +82,7 @@ class AccountsController(TransactionBase):
 						exchange, "exchange_rate"))
 			else:
 				self.doc.conversion_rate = 1
-						
+				
 	def set_missing_item_details(self, get_item_details):
 		"""set missing item values"""
 		for item in self.doclist.get({"parentfield": self.fname}):