https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342018-11-27T13:50:10ZRedmine Entr’ouvertChrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403302018-11-27T13:50:10ZSerghei Mihai
<ul></ul><p>Pourquoi l'export d'authentic est nécessaire?</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403312018-11-27T13:51:02ZSerghei Mihai
<ul></ul><p>(ok, pigé).</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403332018-11-27T13:57:38ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul></ul><p>Je sais pas si c'est nécessaire dans ce cas précis, mais j'importe toujours a2 en premier sans me poser de questions (la doc dit ça quelque part).</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403542018-11-27T15:03:37ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/29534">0001-wip.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/29534/0001-wip.patch">0001-wip.patch</a> ajouté</li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Un embryon de patch.</p>
<p>En gros le bug, c'est qu'on fait des <code>TimePeriodException.objects.update_or_create(defaults=event, **kwargs)</code> avec quasiment rien dans kwargs (desk, label, recurrence_id), du coup c'est vite fait que ça pète une <code>MultipleObjectsReturned</code></p>
<p>J'ai commencé les boulot pour les exceptions simples, mais pour les récurrentes il y a des subtilités qui m'échappent dans l'import, je laisse la main à un spécialiste.</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403602018-11-27T15:14:49ZBenjamin Dauvergne
<ul></ul><p>C'est une utilisation anormale de l'outil, l'import manuel ne peut pas concerner des mises à jour, uniquement des ajouts. Le code est écrit ainsi, la mise à jour ne marche que si le label de l'exception est unique, ton changement va casser la mise à jour via URL (ou on veut mettre à jour start/end par contre), là si en plus on a des exceptions manuelles avec des labels identique, c'est fini, plus rien ne marche.</p>
<p>Clairement le mode actuel d'import manuel ne marche pas, surtout parce que les limitations sont trop difficiles à expliquer, il faudrait l'abandonner et fonctionner uniquement en synchronisation via uid/recurrence_id (ça veut keep_sync_by_uid=True tout le temps) comme pour la synchronisation avec un fichier distant, en plus j'ai l'impression que 100% des outils d'agenda génère un tel UID. C'est juste limitant pour les gens qui essaieraient d'en créer à la main.</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403622018-11-27T15:23:23ZLaurent Séguinlseguin@entrouvert.com
<ul></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>C'est une utilisation anormale de l'outil, l'import manuel ne peut pas concerner des mises à jour, uniquement des ajouts.</p>
</blockquote>
<p>Tant qu'on a pas mieux, c'est fonctionnellement acceptable de faire gameler proprement ce genre d'utilisation (aka: présenter un message d'erreur/refus à l'utilisateur).</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=1403852018-11-27T16:21:18ZBenjamin Dauvergne
<ul></ul><p>Le mieux qu'on puisse faire c'est "Import impossible: il y a déjà 8 exceptions avec le "Libellé optionnel pour identifier la date": "Réunion du service" » (parce qu'en plus le libellé est optionnel...).</p>
<p>Pour moi autant dégager ça et faire mieux, si un .ICS contient un évènement sans identifiant unique, on n'importe rien, on met un message d'erreur: « Ce fichier ICS ne contient pas d'identifiant unique (UID) pour l'évènement start/end/label , l'import est impossible. »</p> Chrono - Bug #28395: Crash Import exception : MultipleObjectsReturnedhttps://dev.entrouvert.org/issues/28395?journal_id=5048802023-11-14T15:41:35ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Rejeté</i></li><li><strong>Planning</strong> mis à <i>Non</i></li></ul><p>Le fichier semble s'importer, et la méthode en question n'existe plus.</p>