From aad5fa4ff03f7b0b0d4c5e155ce2f447af8274c7 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Wed, 14 Dec 2022 09:46:19 +0100
Subject: [PATCH] account_usability_ak : update for py3o

---
 account_usability_akretion/models/account_move.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/account_usability_akretion/models/account_move.py b/account_usability_akretion/models/account_move.py
index 7117e87..a12f101 100644
--- a/account_usability_akretion/models/account_move.py
+++ b/account_usability_akretion/models/account_move.py
@@ -42,7 +42,7 @@ class AccountMove(models.Model):
         for inv in self:
             has_discount = False
             for line in inv.invoice_line_ids:
-                if not line.display_type and not float_is_zero(line.discount, precision_digits=prec):
+                if line.display_type == 'product' and not float_is_zero(line.discount, precision_digits=prec):
                     has_discount = True
                     break
             inv.has_discount = has_discount
@@ -92,7 +92,7 @@ class AccountMove(models.Model):
 
     def delete_lines_qty_zero(self):
         lines = self.env['account.move.line'].search([
-            ('display_type', '=', False),
+            ('display_type', '=', 'product'),
             ('move_id', 'in', self.ids),
             ('quantity', '=', 0)])
         lines.unlink()
@@ -108,7 +108,7 @@ class AccountMove(models.Model):
         # Warning: the order of invoice line is forced in the view
         # <tree editable="bottom" default_order="sequence, date desc, move_name desc, id"
         # it's not the same as the _order in the class AccountMoveLine
-        lines = self.env['account.move.line'].search([('exclude_from_invoice_tab', '=', False), ('move_id', '=', self.id)], order="sequence, date desc, move_name desc, id")
+        lines = self.env['account.move.line'].search([('display_type', 'in', ('product', 'line_section', 'line_note')), ('move_id', '=', self.id)], order="sequence, date desc, move_name desc, id")
         for line in lines:
             if line.display_type == 'line_section':
                 # insert line
@@ -117,7 +117,7 @@ class AccountMove(models.Model):
                 subtotal = 0.0  # reset counter
                 has_sections = True
             else:
-                if not line.display_type:
+                if line.display_type == 'product':
                     subtotal += line.price_subtotal * sign
             res.append({'line': line})
         if has_sections:  # insert last subtotal line
@@ -125,8 +125,8 @@ class AccountMove(models.Model):
         # res:
         # [
         #    {'line': account_invoice_line(1) with display_type=='line_section'},
-        #    {'line': account_invoice_line(2) without display_type},
-        #    {'line': account_invoice_line(3) without display_type},
+        #    {'line': account_invoice_line(2) with display_type=='product'},
+        #    {'line': account_invoice_line(3) with display_type=='product'},
         #    {'line': account_invoice_line(4) with display_type=='line_note'},
         #    {'subtotal': 8932.23},
         # ]