Development #57424
les catégories ne sont pas gérées dans l'import/export
0%
Description
En exportant un Chrono (depuis l'UI) avec des catégories d'agenda, lors de l'import sur un Chrono vierge j'ai une erreur « Catégorie « formations-internes » manquante »
De fait, en regardant le code : on n'exporte pas les catégories... et donc on ne le importe pas non plus.
Fichiers
Révisions associées
agendas: ignore missing category during agenda import (#57424)
Historique
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0001-manager-import-export-categories-57424.patch 0001-manager-import-export-categories-57424.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- pas d'export seul des catégories (qui me paraît peu utile), les inclure seulement avec l'export du site
- revenir sur ce comportement qui fait planter l'import : si la catégorie manque, l'ignorer et importer tout de même l'agenda
Mis à jour par Thomas Noël il y a plus de 2 ans
Valentin Deniaud a écrit :
Je ne suis pas sûr de l'approche ici, je préférerais le même fonctionnement que dans wcs :
- pas d'export seul des catégories (qui me paraît peu utile), les inclure seulement avec l'export du site
- revenir sur ce comportement qui fait planter l'import : si la catégorie manque, l'ignorer et importer tout de même l'agenda
Mais donc on ne pourrait jamais exporter/importer les catégories ? Moi je les voyais juste exportées lors de « l'export complet » du site, et créées à l'import... c'est surtout ça qui m'a «manqué».
Mis à jour par Frédéric Péters il y a plus de 2 ans
Moi je les voyais juste exportées lors de « l'export complet » du site, et créées à l'import... c'est surtout ça qui m'a «manqué».
C'est ce que Valentin suggère, il me semble :
les inclure seulement avec l'export du site
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Frédéric Péters a écrit :
C'est ce que Valentin suggère, il me semble :
les inclure seulement avec l'export du site
Voui on est bien tous d'accord !
Mis à jour par Thomas Noël il y a plus de 2 ans
Je réagissait à l'import :
si la catégorie manque, l'ignorer et importer tout de même l'agenda
Non, si elle manque, il faut la créer.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
On pourrait, lorsqu'on exporte un agenda, au lieu d'y retrouver juste le slug de la catégorie, y trouver un dict 'category': {'slug': 'foo', 'label': 'bar'}; on a tout ce qu'il faut pour créer la catégorie si elle n'existe pas, la retrouver sinon.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
(un avis ?)
Mis à jour par Valentin Deniaud il y a plus de 2 ans
J'ai l'impression que Thomas ne parlait que de l'export du site quand il demandait de créer la catégorie.
Pour l'export/import d'un agenda seul, je reste d'avis qu'il faut ignorer la catégorie si elle n'existe pas, pour moi ce serait inattendu qu'un agenda crée une catégorie.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0002-agendas-ignore-missing-category-during-agenda-import.patch 0002-agendas-ignore-missing-category-during-agenda-import.patch ajouté
- Fichier 0001-manager-import-export-categories-57424.patch 0001-manager-import-export-categories-57424.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de En cours à Solution proposée
Je reprends ça en allégeant le patch de la gestion de l'export seul des catégories, et j'en rajoute un pour ignorer une catégorie manquante à l'import.
(je réaffirme que ça correspond au comportement de wcs, ce qui d'habitude s'avère être un argument d'autorité suffisant)
Mis à jour par Thomas Noël il y a plus de 2 ans
Valentin Deniaud a écrit :
Je reprends ça en allégeant le patch de la gestion de l'export seul des catégories, et j'en rajoute un pour ignorer une catégorie manquante à l'import.
(je réaffirme que ça correspond au comportement de wcs, ce qui d'habitude s'avère être un argument d'autorité suffisant)
Sauf que dans wcs on peut exporte/importer les catégorie (à part).
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Thomas Noël a écrit :
Sauf que dans wcs on peut exporte/importer les catégorie (à part).
J'avoue j'avais pas vu, m'enfin il n'y a pas de vue/bouton spécifique, je suis chaud pour attendre que quelqu'un les demande, ce ticket demandait la correction d'un bug et pas de la feature en plus.
Mis à jour par Thomas Noël il y a plus de 2 ans
Valentin Deniaud a écrit :
Thomas Noël a écrit :
Sauf que dans wcs on peut exporte/importer les catégorie (à part).
J'avoue j'avais pas vu, m'enfin il n'y a pas de vue/bouton spécifique, je suis chaud pour attendre que quelqu'un les demande, ce ticket demandait la correction d'un bug et pas de la feature en plus.
Je vais râler une dernière fois mais le ticket s'appelle « les catégories ne sont pas gérées dans l'import/export » et le patch, effectivement, ne les gère pas. Je comprends rien (et je passe mon tour).
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Assigné à
Valentin Deniaudsupprimé
Elles sont gérées, tout se passe comme si la case « Catégorie » du formulaire d'export wcs était toujours cochée.
Je ne sais pas ce qui a justifié la granularité de l'export wcs mais elle ne me paraît pas justifiée dans chrono, si elle l'est il faut le dire.
Si quelqu'un trouve une utilité à avoir un bouton « Exporter » sur la page des catégories il faut le dire, moi je pense que personne jamais n'ira cliquer dessus, donc je n'ai pas envie de l'ajouter.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud a écrit :
Elles sont gérées, tout se passe comme si la case « Catégorie » du formulaire d'export wcs était toujours cochée.
Ok. Je vais me soigner et arrêter d'être pénible :) Go !
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d81e159827795c706ded07dfe36f47d039e04419
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Dec 8 18:44:59 2021 +0100
agendas: ignore missing category during agenda import (#57424)
commit f974cfe604c13b8441dd8fd88201b8c05113fd10
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Wed Dec 8 17:58:20 2021 +0100
manager: import/export categories (#57424)Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: import/export categories (#57424)