[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,4 +1,6 @@
+from odoo.exceptions import ValidationError
 from odoo.http import request
+from odoo.tools.translate import _
 
 from odoo.addons.cooperator_website.controllers.main import WebsiteSubscription
 
@@ -19,8 +21,44 @@ def get_values_from_user(self, values, is_company):
                 values["national_number"] = national_number
         return values
 
+    def _get_display_national_number(self, is_company):
+        if is_company:
+            return False
+        return request.env.company.display_national_number
+
+    def _get_require_national_number(self, is_company):
+        if is_company:
+            return False
+        return request.env.company.require_national_number
+
     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
+        values["display_national_number"] = self._get_display_national_number(
+            is_company
+        )
+        values["national_number_required"] = self._get_require_national_number(
+            is_company
+        )
         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
+        is_company = kwargs.get("is_company") == "on"
+        national_number = values.get("national_number")
+        if national_number:
+            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"])
+                return True
+            except ValidationError as ve:
+                values["error_msg"] = str(ve)
+        else:
+            if not self._get_require_national_number(is_company):
+                return True
+            values["error_msg"] = _("Some mandatory fields have not been filled.")
+        values["error"] = {"national_number"}
+        return False

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>