IRC logs of #tryton-fr for Tuesday, 2013-04-09

chat.freenode.net #tryton-fr log beginning Tue Apr 9 00:00:08 CEST 2013
Piloucedk: http://codereview.tryton.org/605004/patch/28001/22008 le test sur sequence ligne 174 corrige en partie un bug (http://pastebin.com/r3qyydHw), est ce qu'il faudrait le fixer completement dans la branche 2.6 ?11:06
cedkPilou: je ne pense pas car de toute manière il faut toutes les sequences sur la periode11:10
cedkPilou: donc oui sur trunk, le check est plus parresseux mais ça n'enleve pas la contrainte11:11
Pilouen 2.6 même en rajoutant un test sur sequence on ne peut pas supprimer une séquence inutilisée, je testerai en trunk. Pour 2.6 je ne sais pas si il faut faire une correction (est ce que des corrections de ce type sont acceptées en 2.6?).11:14
cedkPilou: je ne comprend pas11:16
Pilousi j'ai bien compris l'exception est fixée dans le trunk parce qu'une feature a été rajoutée (issue2916). Les features ne sont pas backportées dans les versions précédentes. La question est: un patch en 2.6 serait il accepté ?11:17
cedkPilou: oui s'il corrige un bug11:18
cedkPilou: mais pour moi, il n'y a pas vraiment de bug sauf si je comprend mal le problème11:18
Pilouben y'a une exception déjà11:18
cedkPilou: pas suffisant si c'est du a une mauvais configuration11:21
cedkPilou: pour moi, il n'y a pas de sequence définit sur la périod or elles sont requises11:21
Piloujustement en l'occurrence j'ai créé de nouvelles séquences, celles que j'essaie de supprimer ne sont plus utilisées11:21
cedkPilou: je ne comprend pas ce que tu fais11:25
Pilouj'ai créé des séquences, le nom ne me convenait pas alors j'en ai créé des nouvelles. Les premières ne sont plus utilisées. Quand j'essaie de supprimer celles inutilisées il y a une exception.11:26
cedkPilou: mais les sequences n'ont rien avoir avec les periodes11:28
Pilou?11:29
cedkPilou: mais bon, c'est vrai qu'on pourrait backporter juste les lignes 169 et 17011:30
Piloupour issue3134 il faut faire un patch qui empeche la modification des séquences c'est ça ? Est ce que la modification des séquences doit être possible si il n'y a pas de période définie ?12:54
cedkPilou: non, il n'y a rien à faire13:01
cedkPilou: tu essaie de supprimer une sequence qui est utilisée sur des periods, donc le system les mets à None et tu a l'exception13:03
cedkc'est vrai que le message pourrait être plus explicit mais ça c'est fixé dans trunk13:03
Piloupourquoi la méthode 'types' des classes héritant de 'PYSON' renvoient t elles un set de type et pas un seul type ?17:55
cedkPilou: comme ça c'est flexible18:05
Pilouje n'ai pas trouvé ou c'était utilisé. On pourrait pas utiliser un seul type au lieu d'un set et avoir un type 'unknown' pour Eval ?18:07
cedkPilou: non18:08
cedkPilou: on veut que PYSON soit statiquement typé18:08
Piloulà c'est vraiment pas un fonctionnement cohérent: on définit une valeur par défaut qui ne sera pas utilisé18:09
Pilouautant rajouter un paramètre "type" qui indique le type18:09
cedkPilou: ben non, la valeur par défaut est utilisée par defaut18:10
Pilouelle *peut* être utilisée18:11
Pilouet c'est pas parce que la valeur par défaut a un type que le Eval renverra une valeur de ce type18:12
Pilouil n'y a pas de vérification du type à l'évaluation du coup on peut écrire Eval('champ_de_type_char', 0)18:14
Pilouje mantiens que le assert sert à rien avec les Eval ;)18:15
cedkPilou: oui évidement, car Python a dynamiquement typé mais si on implemente PYSON dans un language statiquement typé ça plantera18:17
Pilouautant appliqué le patch en rajoutant un commentaire " # langage with static type system would not need this" non ?18:21
cedkPilou: non, PYSON doit être indépendant du language d'implémentation18:23
Pilousi c'était statiquement typé il n'y aurait pas besoin de la valeur par défaut pour indiquer le type18:26
cedkPilou: ben le defaut donne le type18:31
Piloualors pourquoi le defaut ne serait il pas obligatoire ?18:32
cedkPilou: il y a une valeur par defaut: ''18:33
cedkPilou: mais oui elle est obligatoire18:33
Pilouok, le seul patch accepté sera un patch de doc18:34
PilouEst ce que c'est possible de faire référence à un autre champ que id dans une expression PYSON dans un domain?22:35
Pilouceci fonctionne: domain=[('id', If(In(Eval('autre_champ'), [1,2]), '!=', '='), -1)]22:35
Piloumais est ce que ceci est possible: domain=[('id', If(In(Eval('autre_champ.description'), ['desc1','desc2']), '!=', '='), -1)] ?22:36
Pilou(autre_champ est de type Many2One)22:37
cedkPilou: non22:48
cedkPilou: le mieux est de faire un champ fonction22:51

Generated by irclog2html.py 2.11.0 by Marius Gedminas - find it at mg.pov.lt!