refactor: improve validation handling in jsonforms-renderer and schemas-bar components

- Enhanced the getSchemaValidations method to better manage schema-specific validations and normalize error paths.
- Updated the getTabValidations method to utilize a recursive function for more robust validation checks.
- Added logic to handle additionalFields validations in the identity table component during error responses.

Author: tacxou

apps/web/src/components/core/jsonforms-renderer.vue

@@ -196,24 +196,61 @@ export default defineNuxtComponent({
       }
     },
     getSchemaValidations(): any[] {
-      let validationList: any = []
-      if (!this.schemaName) {
-        validationList = this.validations || []
-      } else {
-        validationList = this.validations[this.schemaName] || []
+      const rootValidations = this.validations || {}
+      const hasSchemaScopedValidation = Boolean(this.schemaName && rootValidations?.[this.schemaName])
+      const isFlatValidationMap = Object.values(rootValidations).every((value) => typeof value === 'string' || Array.isArray(value))
+
+      const scopedValidations = !this.schemaName
+        ? rootValidations
+        : hasSchemaScopedValidation
+          ? rootValidations[this.schemaName]
+          : isFlatValidationMap
+            ? rootValidations
+            : {}
+
+      const entries: Array<{ path: string; message: string }> = []
+
+      const collectValidationEntries = (value: unknown, path = ''): void => {
+        if (value == null) return
+
+        if (typeof value === 'string') {
+          const message = value.trim()
+          if (message) entries.push({ path, message })
+          return
+        }
+
+        if (Array.isArray(value)) {
+          for (const item of value) {
+            if (typeof item === 'string' && item.trim()) {
+              entries.push({ path, message: item.trim() })
+            }
+          }
+          return
+        }
+
+        if (typeof value === 'object') {
+          for (const [key, child] of Object.entries(value as Record<string, unknown>)) {
+            const nextPath = path ? `${path}.${key}` : key
+            collectValidationEntries(child, nextPath)
+          }
+        }
       }
 
-      const errorObject: ErrorObject[] = []
-      for (const key in validationList) {
-        errorObject.push({
-          message: validationList[key],
-          instancePath: `/${key}`,
+      collectValidationEntries(scopedValidations)
+
+      return entries.map((entry) => {
+        const normalizedPath = entry.path
+          .split('.')
+          .filter(Boolean)
+          .join('/')
+
+        return {
+          message: entry.message,
+          instancePath: normalizedPath ? `/${normalizedPath}` : '',
           keyword: 'type',
           params: {},
-        } as any)
-      }
-
-      return errorObject
+        } as ErrorObject
+      })
     },
     readonlyRenderer(): boolean {
       return this.readonly || /1|true|on|yes/i.test(String(this.$route.query.readonly).toLowerCase())

apps/web/src/components/pages/identities/schemas-bar.vue

@@ -163,7 +163,15 @@ export default defineNuxtComponent({
   },
   methods: {
     getTabValidations(tab: string) {
-      return this.validations?.hasOwnProperty(tab) ? 'red' : false
+      const hasValidation = (value: unknown): boolean => {
+        if (value == null) return false
+        if (typeof value === 'string') return value.trim().length > 0
+        if (Array.isArray(value)) return value.length > 0
+        if (typeof value === 'object') return Object.values(value as Record<string, unknown>).some(hasValidation)
+        return false
+      }
+
+      return hasValidation(this.validations?.[tab]) ? 'red' : false
     },
     addSchema(schema) {
       if (!this.identity.additionalFields) {

apps/web/src/pages/identities/table/[_id].vue

@@ -148,6 +148,10 @@ export default defineNuxtComponent({
           body: sanitizedIdentity,
         })
 
+        if (this.identity?.additionalFields?.validations) {
+          this.identity.additionalFields.validations = {}
+        }
+
         if (this.isNew) {
           this.$router.push('/identities/table')
         }
@@ -169,6 +173,24 @@ export default defineNuxtComponent({
           icon: 'mdi-alert-circle-outline',
         })
         console.error('Erreur lors de la sauvegarde de l identité:', error)
+
+        if (error?.response?._data?.validations) {
+          if (!this.identity.additionalFields) {
+            this.identity.additionalFields = {
+              attributes: {},
+              objectClasses: [],
+              validations: {},
+            }
+          }
+
+          if (!this.identity.additionalFields.validations) {
+            this.identity.additionalFields.validations = {}
+          }
+
+          this.identity.additionalFields.validations = {
+            ...error.response._data.validations,
+          }
+        }
       }
     },
     async sync() {

apps/web/src/pages/settings/health.vue

@@ -407,12 +407,12 @@ export default defineNuxtComponent({
         return 'Maintenance inactive, support open source sur GitHub.'
       }
       if (supportKeyVerificationStatus.value === 'valid') {
-        return 'Contrat de maintenance actif.'
+        return 'Support & maintenance actif.'
       }
       if (supportKeyVerificationStatus.value === 'checking' || supportKeyVerificationStatus.value === 'idle') {
         return 'Verification de la clé de maintenance en cours...'
       }
-      return 'Contrat de maintenance configuré, mais clé de maintenance invalide.'
+      return 'Support & maintenance configuré, mais clé de maintenance invalide.'
     })
 
     const { data, pending, error, refresh } = useHttp<HealthHttpResponse>('/core/health', {