Project

General

Profile

Bug #5015

Rendez-vous du type COURRIEL et TELEPHONE

Added by Mikaël Ates (de retour le 5 mars) over 9 years ago. Updated over 9 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Start date:
20 June 2014
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Pour l'ajout de rendez-vous du type COURRIEL et TELEPHONE on accepte les rendez-vous sans heures et durée et à l'enregistrement on mettra 8h et 10m.


Files

Associated revisions

Revision 8c0910ce (diff)
Added by Serghei Mihai over 9 years ago

agenda: allow empty time and duration for events of type "TELEPHONE" and "COURRIEL"

Closes #5015

Revision ad0780e6 (diff)
Added by Serghei Mihai over 9 years ago

agenda: when adding events of type COURRIEL or TELEPHONE the start time and
duration are taken in account and default to 8h and 10 minutes if not defined

Closes #5015

History

#1

Updated by Serghei Mihai over 9 years ago

  • Assignee set to Serghei Mihai
  • Patch proposed set to No
#2

Updated by Serghei Mihai over 9 years ago

En passant les champs time et duration en optionnels et en surchargeant les clean_<field> ça fonctionne

#3

Updated by Serghei Mihai over 9 years ago

  • Status changed from Nouveau to En cours
#4

Updated by Frédéric Péters over 9 years ago

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).

#5

Updated by Serghei Mihai over 9 years ago

Frédéric Péters a écrit :

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 :

[...]

Il y a des type d'acte par service, j'en tiens compte maintenant.

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).

C'est corrigé, merci.

(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).

Je le garde pour plus tard

#6

Updated by Frédéric Péters over 9 years ago

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).

#8

Updated by Frédéric Péters over 9 years ago

+                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).

#9

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:8c0910cef5d0de5f2a202d1a056324d54219f5f0.

#10

Updated by Mikaël Ates (de retour le 5 mars) over 9 years ago

  • Status changed from Résolu (à déployer) to En cours
  • % Done changed from 100 to 50

Lors de la saisie d'un rdv de ce type, l'heure de début et la durée ne sont pas pris en compte. La durée est automatiquement initialisée à 10 min et l'heure de début à 8h.
Les modifications d'heure de début et de durée pour ces types ne sont pas pris en compte, la durée reste inchangée à 10 min et l'heure de début à 8h.
Lorsqu'il n'est pas saisie une valeur numérique dans la durée, aucun message d'erreur n'est affiché et le formulaire est validé.
L'erreur de saisie est bien indiquée pour l'heure de début lorsqu'une valeur invalide est saisie.

#11

Updated by Serghei Mihai over 9 years ago

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

#12

Updated by Serghei Mihai over 9 years ago

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?

#14

Updated by Serghei Mihai over 9 years ago

Si l'heure et la durée sont saisies, elles sont prises en compte, sinon les valeurs 8h et 10 minutes sont appliquées par défaut

#15

Updated by Jérôme Schneider over 9 years ago

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.

#16

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:ad0780e6859d60347e3b919c7e933cd6e722aff8.

#17

Updated by Mikaël Ates (de retour le 5 mars) over 9 years ago

  • Status changed from Résolu (à déployer) to Fermé

Ok

Also available in: Atom PDF