Produits Entr'ouvert » w.c.s.

L'import d'un formdef erroné contenant plusieurs fois le même id va échouer avec l'exception FormdefImportError.

Dans la modification à test_duplicated_field_ids, il faut selon moi qu'il reste deux modes, celui qui lève FormdefImportError mais également un mode qui assure l'import et la renumérotation.

Et cette partie doit être suivie côté UI, si l'import échoue de manière récupérable (avoir FormdefImportRecoverableError ?), alors relancer l'import avec le paramètre qui assure la renuméroration.

Genre :

        try:
            formdef = FormDef.import_from_xml(fp)
+     except FormdefImportRecoverableError, e:
+          fp.seek(0) # sans doute
+          formdef = FormDef.import_from_xml(fp, fsck_on_error=True)
+          get_session().message('info', _('Form contained errors and has been automatically fixed, you should nevertheless check everything is ok.'))
        except FormdefImportError, e:
            error = True
            reason = _(e.msg)
            if e.details:
                reason += ' [%s]' % e.details
        except ValueError:
            error = True

(ou si on imagine que même avec fsck ça peut échouer, construire ça sous forme de boucle,

        try:
+          for fsck in (False, True):
+                try:
+                     formdef = FormDef.import_from_xml(fp, fsck=fsck)
+                     break
+                except FormdefImportRecoverableError, e:
+                     fp.seek(0) # sans doute
+          else:
+              get_session().message('info', _('Form contained errors and has been automatically fixed, you should nevertheless check everything is ok.'))
        except FormdefImportError, e:
            error = True
            reason = _(e.msg)

Ça me semble ok.