[FIX] validate national number on website form

* validate belgian national register number on website form.
* fix style of belgian national register number field.
* only display belgian national register number field if
  display_national_number is true.

l10n_be_cooperator_website_national_number/controllers/main.py

@@ -1,3 +1,4 @@
+from odoo.exceptions import ValidationError
 from odoo.http import request
 
 from odoo.addons.cooperator_website.controllers.main import WebsiteSubscription
@@ -21,6 +22,28 @@ def get_values_from_user(self, values, is_company):
 
     def fill_values(self, values, is_company, logged, load_from_user=False):
         values = super().fill_values(values, is_company, logged, load_from_user)
-        if not is_company and request.env.company.require_national_number:
-            values["national_number_required"] = True
+        display_national_number = False
+        national_number_required = False
+        if not is_company:
+            display_national_number = request.env.company.display_national_number
+            national_number_required = request.env.company.require_national_number
+        values["display_national_number"] = display_national_number
+        values["national_number_required"] = national_number_required
         return values
+
+    def _additional_validate(self, kwargs, logged, values, post_file):
+        result = super()._additional_validate(kwargs, logged, values, post_file)
+        if result is not True:
+            return result
+        try:
+            # sudo is required to allow access to res.partner.id_category.
+            request.env[
+                "subscription.request"
+            ].sudo().check_be_national_register_number(values["national_number"])
+        except ValidationError as ve:
+            values["error_msg"] = str(ve)
+            error = values.get("error", set())
+            error.add("national_number")
+            values["error"] = error
+            return False
+        return True

l10n_be_cooperator_website_national_number/views/subscription_template.xml

@@ -1,20 +1,18 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
+
     <template id="national_number_template" name="national_number_template">
-        <div
-            class="form-group field-national_number"
-            t-attf-class="form-group #{error and 'national_number' in error and 'has-error' or ''}"
-            name="national_number_container"
-        >
-            <label for="national_number">National Number</label>
+        <div class="mb-3 field-national_number" name="national_number_container">
+            <label for="national_number" class="form-label">National Number</label>
             <input
-                class="form-control form-control-sm"
-                id="national_number"
-                name="national_number"
+                t-attf-class="form-control form-control-sm{{ error and 'national_number' in error and ' is-invalid' or '' }}"
                 type="text"
-                t-att-readonly="logged"
+                name="national_number"
+                id="national_number"
                 t-att-required="national_number_required"
-                t-attf-value="#{national_number or ''}"
+                t-att-readonly="logged"
+                t-att-value="national_number"
+                placeholder="YY.MM.DD-NNN.MM"
             />
         </div>
     </template>
@@ -29,6 +27,7 @@
             position="after"
         >
             <t
+                t-if="display_national_number"
                 t-call="l10n_be_cooperator_website_national_number.national_number_template"
             />
         </xpath>