Produits Entr'ouvert » Chrono

Ça ajoute un has_overlap(a, b) dont tu n'as que faire. (erreur lors d'un rebase)

create_timeperiod_exceptions_from_ics, si on décide de ne pas gérer la récurrence qui peut être définie dans l'ics, il faut lever une erreur quand une telle règle est définie (et si on veut gérer, dateutil.rrule fournit le nécessaire).

La gestion des erreurs : refuser le calendrier sur le moindre truc inattendu ou charger un maximum et fournir l'info sur ce qui n'a pas pu être chargé ? (

Côté UI on commence à avoir trop d'actions, il faudrait les réunir dans un menu (extra-actions-menu).

Ça relève du patch de gestion des exceptions mais autant déjà le noter ici, le modèle TimePeriodException est mauvais, ne devrait pas avoir d'attribut agenda (vu qu'on l'a via "desk").

event['start_datetime'] = vevent.dtstart.value

vevent.dtstart.value peut être un datetime.date() (cf https://www.mozilla.org/media/caldata/FrenchHolidays.ics, qui est l'exemple même de l'ics qu'on souhaiterait importer).

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

Ça ajoute un has_overlap(a, b) dont tu n'as que faire. (erreur lors d'un rebase)

Yep, je vais refaire le patch une fois les 2 branches wip validées et poussées.

create_timeperiod_exceptions_from_ics, si on décide de ne pas gérer la récurrence qui peut être définie dans l'ics, il faut lever une erreur quand une telle règle est définie (et si on veut gérer, dateutil.rrule fournit le nécessaire).

Il faudrait à cette occasion avoir des attributs dans TimePeriodException permettant de marquer la périodicité.

La gestion des erreurs : refuser le calendrier sur le moindre truc inattendu ou charger un maximum et fournir l'info sur ce qui n'a pas pu être chargé ? (

J'étais parti sur l'idée de "tout ou rien" car un .ics qui pour un event ne contient pas la date de début et de fin me semble invalide (sans lire la spec).

Côté UI on commence à avoir trop d'actions, il faudrait les réunir dans un menu (extra-actions-menu).

Ça relève du patch de gestion des exceptions mais autant déjà le noter ici, le modèle TimePeriodException est mauvais, ne devrait pas avoir d'attribut agenda (vu qu'on l'a via "desk").

Oui, j'en avais discuté avec Josué. C'était une première version se basant sur son code.

[...]

vevent.dtstart.value peut être un datetime.date() (cf https://www.mozilla.org/media/caldata/FrenchHolidays.ics, qui est l'exemple même de l'ics qu'on souhaiterait importer).

Ok.

                raise ICSError(_('Reccurent events not handled.'))

typo.

end_dt = datetime.datetime.combine(end_dt,
                                            datetime.datetime.max.time

Malheureusement non, cf l'ics avec les jours fériés :

SUMMARY:Lundi de Pâques
CATEGORIES:Jour férié
DTSTART;VALUE=DATE:20170417
DTEND;VALUE=DATE:20170418

→ la fin ne doit pas être le 18 avril à 23:59, ça ferait deux jours de congés.

J'étais parti sur l'idée de "tout ou rien" car un .ics qui pour un event ne contient pas la date de début et de fin me semble invalide (sans lire la spec).

Pourtant la spec serait pas mal. Mais même sans ça, regarde les ics https://www.data.gouv.fr/fr/datasets/le-calendrier-scolaire/ , il y a dedans un mélange d'événements (dtstart, pas de dtend, genre "Rentrée scolaire") et les vrais moments de congés (qui ont dtstart et dtend)

<li><a class="add" rel="popup" href="{% url 'chrono-manager-desk-add-import-time-period-exceptions' pk=desk.pk }">{ trans 'Import exceptions from .ics' %}</a></li>

Dans le ticket #12550 sur les exceptions, à parler d'ui, j'écrivais : « Et ouverture pour le ticket #16798, le choix d'un fichier ics se ferait dans la même boite de dialogue "Ajouter une exception". ».

@@ -37,12 +38,15 @@ AGENDA_KINDS = (
     ('meetings', _('Meetings')),
 )

-
 def is_midnight(dtime):
     dtime = localtime(dtime)
     return dtime.hour == 0 and dtime.minute == 0

Il y'a une suppression inutile de ligne.

Les lignes 397 à 403 ne sont pas couvertes par les tests, sinon ça m'a l'air ok.

Les lignes 397 à 403 ne sont pas couvertes par les tests

(que ça soit clair, c'est un prérequis)

"Recurrent events not handled", "Event "%s" without start date", ces messages vont être affichés à l'usager, faisons des phrases.

        messages.info(self.request, _('%d exceptions have been imported.') % exceptions)

Utiliser ngettext. Et afficher un message différent quand le fichier était vide.

C'est déjà géré par Django. Il affiche "Le fichier soumis est vide.".

Je voulais dire, quand il n'y a pas d'événement dedans.

+    ics_file = forms.FileField(label=_('ICS File'),
+                               help_text=_('where events will be considered as exceptions'))

ICS FILE
Browse
where events will be considered as exceptions
</rep>

Une phrase complète se lira mieux, vraiment.

Et adapter les tests.

>       assert '1 exception has been imported.' in resp.content
E       assert '1 exception has been imported.' in '<!DOCTYPE html>\n<html>\n  <head>\n    <meta charset="utf-8"/>\n    <title>\n  \n  Agendas\n</title>\n    <link rel="...  <div id="footer">\n    Chrono \xe2\x80\x94 Copyright \xc2\xa9 Entr\'ouvert\n  </div>\n\n  \n  \n  </body>\n</html>\n'

Oublié de commenter: help_text modifié et test corrigé

Ok alors; ack.