[IMP] account_renumber:

* Increase coverage to 100%.
* Update data file tags, remove old-style "or" from wizard view.
* Skip renumber when no date range is found.

account_renumber/tests/test_account_renumber.py

@@ -3,7 +3,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 from datetime import date
-from openerp import fields
+from openerp import exceptions, fields
 from openerp.tests.common import TransactionCase
 
 
@@ -115,3 +115,15 @@ class AccountRenumberCase(TransactionCase):
             self.assertEqual(
                 fields.Date.from_string(move.date).month,
                 expected_month[n])
+
+    def test_renumber_all_no_date_ranges_in_sequence(self):
+        """Works fine using a sequence without date ranges."""
+        self.sequence.use_date_range = False
+        self.test_renumber_all()
+
+    def test_failure_when_no_results(self):
+        """Ensure an exception is raised when no results are found."""
+        new_journal = self.journal.copy()
+        self.moves.write({"journal_id": new_journal.id})
+        with self.assertRaises(exceptions.MissingError):
+            self.test_renumber_all()

account_renumber/wizard/wizard_renumber.py

@@ -83,7 +83,7 @@ class WizardRenumber(models.TransientModel):
                          ("date_from", "<=", move.date),
                          ("date_to", ">=", move.date)]
                     )
-                    if date_range not in reset_ranges:
+                    if date_range and date_range not in reset_ranges:
                         date_range.number_next = self.number_next
                         reset_ranges |= date_range
                 else:

account_renumber/wizard/wizard_renumber_view.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!-- © 2016 Jairo Llopis <jairo.llopis@tecnativa.com>
      License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). -->
-<openerp>
+<odoo>
 
     <record id="view_account_renumber_form" model="ir.ui.view">
         <field name="name">account_renumber.form</field>
@@ -27,7 +27,6 @@
                             string="Renumber"
                             type="object"
                             class="oe_highlight"/>
-                    or
                     <button special="cancel"
                             string="Cancel"
                             class="oe_link"/>
@@ -50,4 +49,4 @@
               action="action_account_renumber"
               groups="account.group_account_manager"/>
 
-</openerp>
+</odoo>