Produits Entr'ouvert » Calebasse

Il vaudrait mieux interroger la base de données pour connaître les id à traiter différemment.

Surtout qu'il peut y avoir des surprises, genre :

 # select id, name from ressources_acttype where name ilike 'courriel';
 id  |   name   
-----+----------
 212 | COURRIEL
 213 | COURRIEL
 215 | Courriel

En cas d'oubli de la durée, le message va désormais être "Veuillez saisir un entier", le clean_duration() pourrait renvoyer un "Ce champ est obligatoire" quand il n'y a pas eu de valeur renseignée, et "Le champ doit contenir uniquement des chiffres" autrement (et le test permet d'entrer un nombre négatif, en passant).

(dans mon monde rêvé heure de début et durée auraient leur astérique de champ obligatoire affichée selon le contenu du champ "type d'acte", mais oublions).

Pour les "Ce champs est obligatoire", tu ferais mieux de _("This field is required.") (ça a l'avantage aussi de ne pas mettre de "s" à champ).

"Le champ doit contenir un chiffre" (et autres messages), il faut parler de nombre, pas de chiffre. (ou alors en parlant des caractères individuels, comme ma proposition "Le champ doit contenir uniquement des chiffres", qui marche aussi bien comme message d'erreur quand un nombre négatif est entré).

Restreindre par service, ça n'a pas d'utilité particulière (sans ça on obtient éventuellement des identifiants de type d'acte qui n'ont aucune chance d'exister pour le service, mais ce n'est pas grave).

+                raise forms.ValidationError(_(u'Le champ doit contenir uniquement des chiffres'))

Dernier (nouveau) truc, il n'y a pas à appeler gettext sur une chaîne en français.

(à part ça je ne suis pas fan da la forme du clean_duration, mais c'est une histoire de goût, à ignorer).

Le souci est donc le fait qu'on affiche un message d'erreur quand l'heure de début est au format invalide? Je corrige cela

Si je comprend bien du ticket #5086 les champs time et duration doivent rester obligatoires même si on choisi COURRIEL ou TELEPHONE comme type d'acte, sauf qu'à la sauvegarde ils sont modifié à 8h et 10 minutes respectivement?

Une petite remarque, je pense que ça serait mieux de pré-remplir l'heure de début (8h00) et la durée (10 min) lors de la sélection d'un type d'acte téléphone ou courrier plutôt que de mettre ces valeurs par défaut après la soumission.

Pour le reste c'est OK pour moi.