| chat.freenode.net #tryton-fr log beginning Sat Jan 12 00:00:02 CET 2013 | ||
| 2013-01-12 13:53 -!- rom1(~romain@2a01:e35:8b2e:be80:b993:5913:8b65:cf98) has left #tryton-fr | ||
| 2013-01-12 15:11 <cedk> pilou: dis moi si tu comprends pas mon commentaire sur le codereview | ||
| 2013-01-12 15:26 <pilou> j'ai compris la modif que tu voulais que je fasse :) | ||
| 2013-01-12 15:28 <pilou> j'avais l'impression que ce n'était pas une méthode interne pas destinée à être utilisée à l'extérieur de trytond, c'est pour ça que je n'ai rajouté que ce dont j'avais besoin | ||
| 2013-01-12 15:29 <cedk> pilou: j'ai fais un deuxième commentaire | ||
| 2013-01-12 15:29 <cedk> pilou: je pense qu'il n'y a pas besoin de changer l'API | ||
| 2013-01-12 15:29 <cedk> pilou: on peut mieux utiliser ir.model.data | ||
| 2013-01-12 15:40 <pilou> hum je n'arrive pas à voir quel test il faut faire | ||
| 2013-01-12 15:42 <pilou> avec le code actuel, au niveau du module production, la nouvelle "location" par défaut est bien ajoutée mais l'objet existant "warehouse" n'est pas modifié | ||
| 2013-01-12 15:42 <cedk> pilou: il faut qu'un record ir.model.data soit créé pour chaque record | ||
| 2013-01-12 15:43 <cedk> pilou: avec le bon module | ||
| 2013-01-12 15:43 <cedk> pilou: et pour le noupdate, il faut vérifier la présence de ce record | ||
| 2013-01-12 15:47 <pilou> par rapport à l'existant http://pastebin.ca/2301576 il faut qu'il y ait un nouveau record c'est ça ? | ||
| 2013-01-12 15:47 <pilou> (enfin ça c'est l'existant avec le patch puisque "warehouse" est modifié) | ||
| 2013-01-12 15:50 <cedk> pilou: oui | ||
| 2013-01-12 15:51 <pilou> ok :) |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!