From 682dd595313f111f36194cbd276d71df95060084 Mon Sep 17 00:00:00 2001
From: Dario Lodeiros <dario@aloxa.eu>
Date: Mon, 8 Apr 2019 18:31:18 +0200
Subject: [PATCH] [FIX] field in attrs and compute count field based on folios
 count

---
 hotel/wizard/folio_make_invoice_advance.py        | 11 ++++++-----
 hotel/wizard/folio_make_invoice_advance_views.xml |  3 +--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/hotel/wizard/folio_make_invoice_advance.py b/hotel/wizard/folio_make_invoice_advance.py
index 0e4a00058..6fe4cb900 100644
--- a/hotel/wizard/folio_make_invoice_advance.py
+++ b/hotel/wizard/folio_make_invoice_advance.py
@@ -12,10 +12,6 @@ class FolioAdvancePaymentInv(models.TransientModel):
     _name = "folio.advance.payment.inv"
     _description = "Folios Advance Payment Invoice"
 
-    @api.model
-    def _count(self):
-        return len(self._context.get('active_ids', []))
-
     @api.model
     def _get_advance_payment_method(self):
         return 'all'
@@ -69,7 +65,7 @@ class FolioAdvancePaymentInv(models.TransientModel):
     auto_invoice = fields.Boolean('Auto Payment Invoice',
                                   default=True,
                                   help='Automatic validation and link payment to invoice')
-    count = fields.Integer(default=_count, string='# of Orders')
+    count = fields.Integer(compute='_count', store=True, string='# of Orders')
     folio_ids  = fields.Many2many("hotel.folio", string="Folios",
                                   help="Folios grouped",
                                   default=_get_default_folio)
@@ -98,6 +94,11 @@ class FolioAdvancePaymentInv(models.TransientModel):
                                         help="Taxes used for deposits",
                                         default=_default_deposit_taxes_id)
 
+    @api.depends('folio_ids')
+    def _count(self):
+        for record in self:
+            record.update({'count': len(self.folio_ids)})
+
     @api.onchange('advance_payment_method')
     def onchange_advance_payment_method(self):
         if self.advance_payment_method == 'percentage':
diff --git a/hotel/wizard/folio_make_invoice_advance_views.xml b/hotel/wizard/folio_make_invoice_advance_views.xml
index b1522dd1d..c4dd1b092 100644
--- a/hotel/wizard/folio_make_invoice_advance_views.xml
+++ b/hotel/wizard/folio_make_invoice_advance_views.xml
@@ -37,8 +37,7 @@
                                context="{'search_default_partner_invoice_id': partner_invoice_id}"
                                >
                             <tree string="Folios" editable="bottom"
-                                    decoration-danger="partner_invoice_id != parent.partner_invoice_id"
-                                    attrs="{'invisible': [('advance_payment_method', 'in', ('fixed','percentage'))]}">
+                                    decoration-danger="partner_invoice_id != parent.partner_invoice_id">
                                 <field name="partner_invoice_id" />
                                 <field name="name" />
                                 <field name="state" />