| chat.freenode.net #tryton-fr log beginning Tue Apr 9 00:00:08 CEST 2013 | ||
| 2013-04-09 11:06 <Pilou> cedk: 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 ? | ||
| 2013-04-09 11:10 <cedk> Pilou: je ne pense pas car de toute manière il faut toutes les sequences sur la periode | ||
| 2013-04-09 11:11 <cedk> Pilou: donc oui sur trunk, le check est plus parresseux mais ça n'enleve pas la contrainte | ||
| 2013-04-09 11:14 <Pilou> en 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?). | ||
| 2013-04-09 11:16 <cedk> Pilou: je ne comprend pas | ||
| 2013-04-09 11:17 <Pilou> si 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é ? | ||
| 2013-04-09 11:18 <cedk> Pilou: oui s'il corrige un bug | ||
| 2013-04-09 11:18 <cedk> Pilou: mais pour moi, il n'y a pas vraiment de bug sauf si je comprend mal le problème | ||
| 2013-04-09 11:18 <Pilou> ben y'a une exception déjà | ||
| 2013-04-09 11:21 <cedk> Pilou: pas suffisant si c'est du a une mauvais configuration | ||
| 2013-04-09 11:21 <cedk> Pilou: pour moi, il n'y a pas de sequence définit sur la périod or elles sont requises | ||
| 2013-04-09 11:21 <Pilou> justement en l'occurrence j'ai créé de nouvelles séquences, celles que j'essaie de supprimer ne sont plus utilisées | ||
| 2013-04-09 11:25 <cedk> Pilou: je ne comprend pas ce que tu fais | ||
| 2013-04-09 11:26 <Pilou> j'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. | ||
| 2013-04-09 11:28 <cedk> Pilou: mais les sequences n'ont rien avoir avec les periodes | ||
| 2013-04-09 11:29 <Pilou> ? | ||
| 2013-04-09 11:30 <cedk> Pilou: mais bon, c'est vrai qu'on pourrait backporter juste les lignes 169 et 170 | ||
| 2013-04-09 12:54 <Pilou> pour 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 ? | ||
| 2013-04-09 13:01 <cedk> Pilou: non, il n'y a rien à faire | ||
| 2013-04-09 13:03 <cedk> Pilou: tu essaie de supprimer une sequence qui est utilisée sur des periods, donc le system les mets à None et tu a l'exception | ||
| 2013-04-09 13:03 <cedk> c'est vrai que le message pourrait être plus explicit mais ça c'est fixé dans trunk | ||
| 2013-04-09 17:55 <Pilou> pourquoi la méthode 'types' des classes héritant de 'PYSON' renvoient t elles un set de type et pas un seul type ? | ||
| 2013-04-09 18:05 <cedk> Pilou: comme ça c'est flexible | ||
| 2013-04-09 18:07 <Pilou> je 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 ? | ||
| 2013-04-09 18:08 <cedk> Pilou: non | ||
| 2013-04-09 18:08 <cedk> Pilou: on veut que PYSON soit statiquement typé | ||
| 2013-04-09 18:09 <Pilou> là c'est vraiment pas un fonctionnement cohérent: on définit une valeur par défaut qui ne sera pas utilisé | ||
| 2013-04-09 18:09 <Pilou> autant rajouter un paramètre "type" qui indique le type | ||
| 2013-04-09 18:10 <cedk> Pilou: ben non, la valeur par défaut est utilisée par defaut | ||
| 2013-04-09 18:11 <Pilou> elle *peut* être utilisée | ||
| 2013-04-09 18:12 <Pilou> et c'est pas parce que la valeur par défaut a un type que le Eval renverra une valeur de ce type | ||
| 2013-04-09 18:14 <Pilou> il n'y a pas de vérification du type à l'évaluation du coup on peut écrire Eval('champ_de_type_char', 0) | ||
| 2013-04-09 18:15 <Pilou> je mantiens que le assert sert à rien avec les Eval ;) | ||
| 2013-04-09 18:17 <cedk> Pilou: oui évidement, car Python a dynamiquement typé mais si on implemente PYSON dans un language statiquement typé ça plantera | ||
| 2013-04-09 18:21 <Pilou> autant appliqué le patch en rajoutant un commentaire " # langage with static type system would not need this" non ? | ||
| 2013-04-09 18:23 <cedk> Pilou: non, PYSON doit être indépendant du language d'implémentation | ||
| 2013-04-09 18:26 <Pilou> si c'était statiquement typé il n'y aurait pas besoin de la valeur par défaut pour indiquer le type | ||
| 2013-04-09 18:31 <cedk> Pilou: ben le defaut donne le type | ||
| 2013-04-09 18:32 <Pilou> alors pourquoi le defaut ne serait il pas obligatoire ? | ||
| 2013-04-09 18:33 <cedk> Pilou: il y a une valeur par defaut: '' | ||
| 2013-04-09 18:33 <cedk> Pilou: mais oui elle est obligatoire | ||
| 2013-04-09 18:34 <Pilou> ok, le seul patch accepté sera un patch de doc | ||
| 2013-04-09 22:35 <Pilou> Est ce que c'est possible de faire référence à un autre champ que id dans une expression PYSON dans un domain? | ||
| 2013-04-09 22:35 <Pilou> ceci fonctionne: domain=[('id', If(In(Eval('autre_champ'), [1,2]), '!=', '='), -1)] | ||
| 2013-04-09 22:36 <Pilou> mais est ce que ceci est possible: domain=[('id', If(In(Eval('autre_champ.description'), ['desc1','desc2']), '!=', '='), -1)] ? | ||
| 2013-04-09 22:37 <Pilou> (autre_champ est de type Many2One) | ||
| 2013-04-09 22:48 <cedk> Pilou: non | ||
| 2013-04-09 22:51 <cedk> Pilou: le mieux est de faire un champ fonction |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!