[IMP] account: add more visibility about the sent status of invoices

In the account module, we want to help customers visualize which invoice
has been sent or not.

task-5231288

Author: jedel-odoo

addons/account/models/account_move.py

@@ -614,13 +614,13 @@ def _sequence_year_range_monthly_regex(self):
         tracking=True,
         help="It indicates that the invoice/payment has been sent or the PDF has been generated.",
     )
-    is_move_sent_string = fields.Selection(
+    is_move_sent_value = fields.Selection(
         selection=[
-            ("sent", "Sent"),
-            ("not_sent", "Not Sent")
+            ('sent', "Sent"),
+            ('not_sent', "Not Sent")
         ],
         copy=False,
-        compute="_compute_is_move_sent_string"
+        compute='_compute_is_move_sent_value'
     )
     is_being_sent = fields.Boolean(
         help="Is the move being sent asynchronously",
@@ -849,12 +849,9 @@ def _compute_journal_id(self):
             move.journal_id = move._search_default_journal()
 
     @api.depends('is_move_sent')
-    def _compute_is_move_sent_string(self):
+    def _compute_is_move_sent_value(self):
         for move in self:
-            if move.is_move_sent:
-                move.is_move_sent_string = "sent" 
-            else:
-                move.is_move_sent_string = "not_sent"
+            move.is_move_sent_value = 'sent' if move.is_move_sent else 'not_sent'
 
     def _get_valid_journal_types(self):
         if self.is_sale_document(include_receipts=True):

addons/account/views/account_move_views.xml

@@ -549,11 +549,11 @@
                            invisible="payment_state == 'invoicing_legacy' or move_type == 'entry'"
                            optional="show"
                     />
-                    <field name="is_move_sent_string"
+                    <field name="is_move_sent_value"
                            string="Sent"
                            widget="badge"
-                           decoration-danger="is_move_sent_string == 'not_sent'"
-                           decoration-success="is_move_sent_string == 'sent'"
+                           decoration-danger="is_move_sent_value == 'not_sent'"
+                           decoration-success="is_move_sent_value == 'sent'"
                            optional="hide"
                     />
                     <field name="move_type" column_invisible="context.get('default_move_type', True)"/>