From 0ad6526d3ee8f806275e94425b8e6d43077e60e5 Mon Sep 17 00:00:00 2001
From: Stepan Savelyev <stepan.savelyev@rydlab.ru>
Date: Wed, 5 Feb 2020 15:41:49 +0500
Subject: [PATCH] [FIX] TypeError: unsupported operand type(s) for +: 'bool'
 and 'str'

---
 data/fields_default.xml | 76 +++++++++++++++++++++++++----------------
 1 file changed, 47 insertions(+), 29 deletions(-)

diff --git a/data/fields_default.xml b/data/fields_default.xml
index 21c35fa..f6b401f 100644
--- a/data/fields_default.xml
+++ b/data/fields_default.xml
@@ -14,12 +14,7 @@
 self = record
 
 partner = self.partner_id # res.partner
-partner_bank = partner.bank_ids and partner.bank_ids[0] # res.partner.bank
-partner_bank_name = partner_bank.bank_id.name + (" г. {city}".format(city=partner_bank.bank_id.city) if partner_bank.bank_id.city else "")
-
 seller = self.company_id
-seller_bank = seller.bank_ids and seller.bank_ids[0]
-seller_bank_name = seller_bank.bank_id.name + (" г. {city}".format(city=seller_bank.bank_id.city) if seller_bank.bank_id.city else "")
 
 # Functions
 _ = self._
@@ -77,21 +72,35 @@ ctx = {
     "partner_email": partner.email,
 }
 
-if partner_bank:
-    ctx.update({
-        "partner_bank": partner_bank_name,
-        "partner_rs": partner_bank.acc_number,
-        "partner_ks": partner_bank.bank_id.corr_account,
-        "partner_bic": partner_bank.bank_id.bic,
-    })
+seller_bank = seller.bank_ids and seller.bank_ids[0]
 if seller_bank:
+
+    bank = seller_bank.bank_id
+    bank_name = bank and bank.name or ""
+    bank_city = "г. {city}".format(city=bank.city) if bank and bank.city else ""
+    seller_bank_name = "{} {}".format(bank_name, bank_city).strip()
+
     ctx.update({
         "seller_bank": seller_bank_name,
         "seller_rs": seller_bank.acc_number,
-        "seller_ks": seller_bank.bank_id.corr_account,
-        "seller_bic": seller_bank.bank_id.bic,
+        "seller_ks": bank.corr_account,
+        "seller_bic": bank.bic,
     })
 
+partner_bank = partner.bank_ids and partner.bank_ids[0]
+if partner_bank:
+    
+    bank = partner_bank.bank_id
+    bank_name = bank and bank.name or ""
+    bank_city = "г. {city}".format(city=bank.city) if bank and bank.city else ""
+    partner_bank_name = "{} {}".format(bank_name, bank_city).strip()
+
+    ctx.update({
+        "partner_bank": partner_bank_name,
+        "partner_rs": partner_bank.acc_number,
+        "partner_ks": bank.corr_account,
+        "partner_bic": bank.bic,
+    })
 
 # Person
 if not partner.is_company:
@@ -158,12 +167,6 @@ self = record
 seller = self.company_id
 partner = self.partner_id
 
-seller_bank = seller.bank_ids and seller.bank_ids[0]
-seller_bank_name = seller_bank.bank_id.name + (" г. {city}".format(city=seller_bank.bank_id.city) if seller_bank.bank_id.city else "")
-
-partner_bank = partner.bank_ids and partner.bank_ids[0]
-partner_bank_name = partner_bank.bank_id.name + (" г. {city}".format(city=partner_bank.bank_id.city) if partner_bank.bank_id.city else "")
-
 # Functions
 _ = self.contract_id._
 get_date = self.contract_id.get_date
@@ -242,19 +245,34 @@ ctx = {
     "partner_email": partner.email,
 }
 
-if partner_bank:
-    ctx.update({
-        "partner_bank": partner_bank_name,
-        "partner_rs": partner_bank.acc_number,
-        "partner_ks": partner_bank.bank_id.corr_account,
-        "partner_bic": partner_bank.bank_id.bic,
-    })
+seller_bank = seller.bank_ids and seller.bank_ids[0]
 if seller_bank:
+
+    bank = seller_bank.bank_id
+    bank_name = bank and bank.name or ""
+    bank_city = "г. {city}".format(city=bank.city) if bank and bank.city else ""
+    seller_bank_name = "{} {}".format(bank_name, bank_city).strip()
+
     ctx.update({
         "seller_bank": seller_bank_name,
         "seller_rs": seller_bank.acc_number,
-        "seller_ks": seller_bank.bank_id.corr_account,
-        "seller_bic": seller_bank.bank_id.bic,
+        "seller_ks": bank.corr_account,
+        "seller_bic": bank.bic,
+    })
+
+partner_bank = partner.bank_ids and partner.bank_ids[0]
+if partner_bank:
+    
+    bank = partner_bank.bank_id
+    bank_name = bank and bank.name or ""
+    bank_city = "г. {city}".format(city=bank.city) if bank and bank.city else ""
+    partner_bank_name = "{} {}".format(bank_name, bank_city).strip()
+
+    ctx.update({
+        "partner_bank": partner_bank_name,
+        "partner_rs": partner_bank.acc_number,
+        "partner_ks": bank.corr_account,
+        "partner_bic": bank.bic,
     })