add XXX notes

wcs/qommon/static/js/qommon.forms.js

@@ -802,6 +802,8 @@ const LiveValidation = (function(){
       if (json.err !== 1) {
         return
       } else {
+        // XXX: if faut plutôt retourner json.msg qui contiendra le bon message
+        // d'erreur
         let errorType
         for (const key in json) {
           if (json[key] === true) {
@@ -838,6 +840,9 @@ const LiveValidation = (function(){
           ? await hasServerError(this.name, field, field.form, this.checkUrl)
           : false
         const error = attrError ? attrError : serverError
+        // XXX: mais ici on aurait donc attrError qui serait une chaine de la
+        // forme missingValue (en cas de JS), et serverError qui serait un vrai
+        // texte.
         if (error) {
           this.showError(field, error)
         } else {
@@ -848,6 +853,10 @@ const LiveValidation = (function(){
     showError(field, error) {
       if (this.hasError === error) return
       this.widget.classList.add(this.errorClass)
+      // XXX: et ici on se trouve alors à chercher un élément sur base de
+      // 'error' qui ne correspond pas (c'est le message d'erreur, pas du
+      // texte), et dans ce cas le taf à faire est d'afficher ce message,
+      // pas celui qui serait dans un élément de la page.
       const errorElMessage = document.getElementById(`error_${this.name}_${error}`).innerHTML
       this.errorEl.innerHTML = errorElMessage
       this.widget.appendChild(this.errorEl)