IRC logs of #tryton-fr for Wednesday, 2013-02-20

chat.freenode.net #tryton-fr log beginning Wed Feb 20 00:00:02 CET 2013
2013-02-20 10:37 <cedk> Pilou: qu'est-ce que tu en pense de http://codereview.tryton.org/665003/
2013-02-20 10:41 <Pilou> dans quel cas l'exception est elle levée ?
2013-02-20 10:43 <cedk> Pilou: si le pays est utiliser quelque part et qu'on doit le supprimer
2013-02-20 10:44 <cedk> Pilou: j'ai testé avec le dernier export de pycountry et j'ai pas eu de problème
2013-02-20 10:45 <cedk> Pilou: du coup on a plus besoin de se tracasser des changements d'id de la norme XML
2013-02-20 10:46 <Pilou> ça ne vaudrait pas le coup d'enregistrer db_id, model, tb_s dans une table et d'avoir un bouton/wizard qui permet d'essayer à nouveau la suppression ? Le cas utilisateur serait: après une mise à jour je vérifie la vue qui contient db_id, model, tb_s. Je corrige les problèmes dans les données (ie ne plus utiliser db_id), je réessaie de supprimer en utilisant le bouton.
2013-02-20 10:46 <Pilou> En ce qui concerne le patch c'est sain de désactiver ce qui ne doit plus être utilisé.
2013-02-20 10:47 <cedk> Pilou: pour le wizard, une mise à jour du module le fait
2013-02-20 10:47 <cedk> Pilou: par contre, j'ai déjà pensé à une fonctionnalité similaire mais pour la mise à jour des records
2013-02-20 10:48 <cedk> Pilou: par fois, on ne met pas à jour car il a été modifié par l'utilisateur
2013-02-20 10:48 <cedk> Pilou: du coup ça pourrait être sympat de permet à l'utilisateur d'écraser sa modif avec les données de base
2013-02-20 10:49 <Pilou> les données des XML ne sont pas immutables ?
2013-02-20 10:51 <cedk> Pilou: par defaut si, mais tu peux changer le comportement
2013-02-20 10:52 <Pilou> il faut modifier le code de trytond pour ça ?
2013-02-20 10:53 <cedk> Pilou: surcharge la method ModelStorage.check_xml_record
2013-02-20 10:54 <cedk> Pilou: mais c'est à utiliser en dernier recour
2013-02-20 10:55 <Pilou> oui je vois ça comme ça aussi :)
2013-02-20 10:55 <Pilou> Ca vaut quand même le coup d'automatiser les changements d'id pour country non ? c'est une tâche automatisable que tous les users vont devoir faire.
2013-02-20 10:56 <cedk> Pilou: je vois pas comment automatiser
2013-02-20 10:57 <cedk> Pilou: je ne pense pas que tous les users vont de voir faire quelque chose
2013-02-20 10:58 <cedk> Pilou: de plus comme les doublons seront desactivés, ils ne seront pas génant
2013-02-20 10:59 <Pilou> Le patch incomplet ne va pas du tout ? Le problème pour l'utilisateur sera d'identifier ce qu'il doit modifier.
2013-02-20 11:00 <cedk> Pilou: mais que devra-t-il modifier ?
2013-02-20 11:02 <Pilou> ben si il a un modèle qui utilise un champ "country" qui a pour valeur un id qui correspond à un pays qui est passé inactif (parce que le code du pays a été renommé), il voudra surement pointer vers le nouveau code.
2013-02-20 11:03 <cedk> Pilou: s'il le veut
2013-02-20 11:03 <cedk> Pilou: il y a 719 changements d'id
2013-02-20 11:03 <cedk> Pilou: je vois pas comment c'est gerable pour nous
2013-02-20 11:04 <cedk> Pilou: et seulement 1 pays
2013-02-20 11:04 <Pilou> ben quand c'était possible (par exemple renommage) il me semble que le début de patch ne changeait pas l'id justement.
2013-02-20 11:05 <cedk> Pilou: et le pays disparait: Netherlands Antilles
2013-02-20 11:06 <Pilou> quand c'est pas possible modifié le champ active du pays est une bonne idée :)
2013-02-20 11:07 <cedk> Pilou: les subdivision ne sont pas si importante en plus
2013-02-20 11:08 <Pilou> mon point de vue: l'idéal c'est 1) ne pas changer les id quand c'est possible 2) passer en inactif les pays supprimer 3) rendre la liste des actions de maintenance disponible à l'utilisateur (ie "impossible de supprimer tel pays")
2013-02-20 11:11 <cedk> Pilou: 2 et 3 sont présents
2013-02-20 11:12 <cedk> Pilou: 1 je pense que vu la vollatilité (subdivision) des données +700 changement un 2 ans, c'est ingérable
2013-02-20 11:12 <Pilou> pour 3 tu veux dire que lors de la migration c'est indiqué ?
2013-02-20 11:12 <Pilou> pour 3 tu veux dire que lors de la migration c'est indiqué par le message d'erreur "Could not delete id [...]" ?
2013-02-20 11:13 <cedk> Pilou: oui
2013-02-20 11:14 <Pilou> ce serait plus user friendly si c'était stocké dans un modèle mais ok ce n'est pas obligé de faire quelque chose sur ce point
2013-02-20 11:15 <cedk> Pilou: je pense pas que ce soit nécessaire puisque c'est rejouable
2013-02-20 11:16 <Pilou> pour 1 est ce que tu veux que je mette à jour le patch en suivant tes remarques ? je peux faire ça pour demain.
2013-02-20 11:18 <Pilou> je me dis que si les données des payes sont créées par Tryton, c'est un peu à Tryton de faciliter la migration. Peut être qu'une autre solution serait de fournir le script pycountry mais de ne pas charger les données à l'installation du module.
2013-02-20 11:18 <cedk> Pilou: ça m'ennuie vraiment d'avoir de gros script de migration pour quelque chose qu'on ne maitrise pas
2013-02-20 11:19 <Pilou> je comprends et c'est pour ça que je propose ^^
2013-02-20 11:19 <cedk> Pilou: c'est pas le problème mais dans 1-2 ans quand on remettra encore à jour, on va encore avoir un script de la même taille à faire
2013-02-20 11:20 <cedk> Pilou: avec en plus le problème de conflit entre les 2 migrations
2013-02-20 11:20 <cedk> Pilou: et la complexité va croite de manière exponentiel
2013-02-20 11:24 <Pilou> je suis d'accord que c'est un script moche (Un utilisateur qui utilise beaucoup tous les codes sera obligé de faire un script). Personnellement je trouve ça logique de ne pas inclure les données par défaut si les nombreuses modifications automatisables ne sont pas inclues.
2013-02-20 11:25 <cedk> Pilou: en fait, on fait déjà un peu la même chose pour les plans comptable
2013-02-20 11:25 <cedk> Pilou: on fait du «best effort»
2013-02-20 11:26 <Pilou> je ne connais pas du tout le fonctionnement au niveau des plans comptables.
2013-02-20 11:32 <cedk> Pilou: en fait on a un peu la même problématique, des données dont on ne maitrise pas l'évolution
2013-02-20 11:33 <cedk> Pilou: donc on essaie de mettre à jour mais on supprime pas les anciens s'ils sont utilisés

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!