| chat.freenode.net #tryton log beginning Sat Dec 4 00:00:02 CET 2010 | ||
| 2010-12-04 00:14 <plantian> cedk: Yes, seems to fix problem in the correct way, but I don't understand when default method would expect ids. | ||
| 2010-12-04 00:15 <plantian> http://hg.tryton.org/1.6/trytond/file/1844a1a148fe/trytond/model/model.py#l110 -- maybe the code is not for defaults | ||
| 2010-12-04 00:25 <cedk> plantian: the goal is to transform ids from base model into ids of parent model | ||
| 2010-12-04 00:25 <cedk> plantian: but this doesn't work for default functions as they don't take ids as argument | ||
| 2010-12-04 00:26 <cedk> plantian: could you fill an issue on roundup? | ||
| 2010-12-04 00:26 <plantian> cedk: Yes, looking at 1.8 it does not seem affected. Cursor, user, context are all on Transaction now right ? | ||
| 2010-12-04 00:27 <cedk> plantian: yes but the issue is still there | ||
| 2010-12-04 00:27 <plantian> cedk: I don't think so, what issue ? | ||
| 2010-12-04 00:28 <cedk> plantian: the call to inherited method that doesn't take ids as parameter but something else | ||
| 2010-12-04 00:28 <plantian> The only way to better fix the issue is to treat defaults different from other fields and call them explicitly. | ||
| 2010-12-04 00:28 <plantian> Right. | ||
| 2010-12-04 00:29 <cedk> plantian: of course, default method are no more in the category of methods | ||
| 2010-12-04 00:29 <cedk> plantian: no, we should get a better way to transforms ids (first argument) only when required | ||
| 2010-12-04 00:33 <plantian> cedk: verification on bugs login seems to verify but presents traceback | ||
| 2010-12-04 00:33 <cedk> plantian: don't understand | ||
| 2010-12-04 00:34 <plantian> I will try to take screen shot. | ||
| 2010-12-04 00:38 -!- cheche(cheche@188.85.213.151) has joined #tryton | ||
| 2010-12-04 01:16 <plantian> cedk: Since I'm kind of unsure of what to do I don't think it would be effecient to correspond inside issue. Can we just discuss it here? Seems like goal would be to 1. remove the ids code all together and put it somewhere else and even better 2. don't override __getattr__ like this at all. | ||
| 2010-12-04 01:18 <plantian> If 1 and 2 cannot be done maybe the attributes whose first argument should be ids could be listed in a list somewhere on the class, like replace_ids_with_parent = ['browse', ...] and these could be checked when calling func in order to apply the ids code to the first argument. | ||
| 2010-12-04 01:19 <cedk> plantian: I think about using introspection to find if method got ids or not | ||
| 2010-12-04 01:22 <cedk> plantian: but I need to think a little bit more about that | ||
| 2010-12-04 01:22 <plantian> Yeah that seems pretty heavy handed since ids might mean something different or they might not want them replaced. | ||
| 2010-12-04 01:22 <cedk> plantian: and we can not drop the __getattr__ method because it is required for default value, on_change methods and I guess other stuffs | ||
| 2010-12-04 01:23 <plantian> cedk: When you say "parent" you mean in the case of product.product ids you would replace them with product.template ids right? | ||
| 2010-12-04 01:24 <cedk> plantian: yes | ||
| 2010-12-04 01:41 -!- ikks(~ikks@190.158.122.179) has joined #tryton | ||
| 2010-12-04 02:02 -!- zodman(~zodman@foresight/developer/zodman) has joined #tryton | ||
| 2010-12-04 02:56 -!- sharoon(~sharoon@218.248.84.81) has joined #tryton | ||
| 2010-12-04 03:10 -!- pepeu(~manuel@201.155.193.192) has joined #tryton | ||
| 2010-12-04 05:04 -!- sharoon(~sharoon@202.129.198.250) has joined #tryton | ||
| 2010-12-04 05:18 -!- yangoon(~mathiasb@p549F46B5.dip.t-dialin.net) has joined #tryton | ||
| 2010-12-04 07:01 -!- sharoon(~sharoon@202.129.198.250) has joined #tryton | ||
| 2010-12-04 07:11 -!- zodman(~zodman@foresight/developer/zodman) has joined #tryton | ||
| 2010-12-04 08:20 -!- GasbaKid(~GasbaKid@41.96.120.6) has joined #tryton | ||
| 2010-12-04 08:59 -!- paepke(~paepke@p4FEB0EBF.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-12-04 09:04 -!- GasbaKid(~GasbaKid@41.96.120.6) has joined #tryton | ||
| 2010-12-04 09:45 -!- chrue(~chrue@host-091-097-028-010.ewe-ip-backbone.de) has joined #tryton | ||
| 2010-12-04 10:00 -!- paepke(~paepke@p4FEB0EBF.dip0.t-ipconnect.de) has left #tryton | ||
| 2010-12-04 10:17 -!- paepke(~paepke@p4FEB0EBF.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-12-04 11:09 -!- Timitos(~kp@88.217.184.172) has joined #tryton | ||
| 2010-12-04 11:11 -!- enlightx(~enlightx@217.202.148.26) has joined #tryton | ||
| 2010-12-04 11:36 -!- cedk(~ced@gentoo/developer/cedk) has joined #tryton | ||
| 2010-12-04 11:46 <cedk> plantian: for issue1776, can you make a patch based on 1.6 trunk | ||
| 2010-12-04 11:46 <cedk> plantian: this will be a temporary fix for this branch | ||
| 2010-12-04 12:44 -!- sharkcz(~sharkcz@2001:15c0:6747:160::7) has joined #tryton | ||
| 2010-12-04 13:17 -!- enlightx(~enlightx@217.202.111.105) has joined #tryton | ||
| 2010-12-04 14:12 -!- enlightx(~enlightx@217.202.119.152) has joined #tryton | ||
| 2010-12-04 14:46 -!- sharoon(~sharoon@218.248.84.82) has joined #tryton | ||
| 2010-12-04 14:55 -!- paepke(~paepke@p4FEB0EBF.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-12-04 14:56 -!- sharoon_(~sharoon@115.184.99.214) has joined #tryton | ||
| 2010-12-04 15:58 -!- GasbaKid(~GasbaKid@41.96.44.198) has joined #tryton | ||
| 2010-12-04 16:04 -!- pjstevns(~pjstevns@helpoort.xs4all.nl) has joined #tryton | ||
| 2010-12-04 16:37 -!- udono(~udono@dynamic-unidsl-85-197-25-97.westend.de) has joined #tryton | ||
| 2010-12-04 17:00 -!- vladimir_(~vladimir@213.151.246.136) has joined #tryton | ||
| 2010-12-04 17:17 -!- paepke(~paepke@p4FEB0EBF.dip0.t-ipconnect.de) has left #tryton | ||
| 2010-12-04 18:07 -!- enlightx(~enlightx@217.202.114.186) has joined #tryton | ||
| 2010-12-04 18:13 -!- gremly(~gremly@190.26.156.189) has joined #tryton | ||
| 2010-12-04 18:19 -!- GasbaKid(~GasbaKid@41.96.44.198) has joined #tryton | ||
| 2010-12-04 18:22 -!- GasbaKid(~GasbaKid@41.96.44.198) has joined #tryton | ||
| 2010-12-04 19:24 -!- sharoon(~sharoon@115.184.49.30) has joined #tryton | ||
| 2010-12-04 19:28 -!- pepeu(~manuel@201.155.193.192) has joined #tryton | ||
| 2010-12-04 19:53 <plantian> cedk: I have not submitted a patch for a while but there it is. | ||
| 2010-12-04 20:50 -!- sharkcz(~sharkcz@2001:15c0:6747:160::7) has joined #tryton | ||
| 2010-12-04 20:58 -!- pjstevns(~pjstevns@helpoort.xs4all.nl) has joined #tryton | ||
| 2010-12-04 21:06 -!- GasbaKid(~GasbaKid@41.96.44.198) has joined #tryton | ||
| 2010-12-04 22:09 <plantian> Hi, has anyone ever mentioned any kind of interface for quick inventory lookup? Finding a product and clicking it and then clicking inventory and then clicking date and then navigated around in the locations seems a bit slow. | ||
| 2010-12-04 23:10 -!- pjstevns(~pjstevns@helpoort.xs4all.nl) has left #tryton | ||
| 2010-12-04 23:45 <cedk> plantian: but you need all this information | ||
| 2010-12-04 23:48 <plantian> cedk: Right but maybe the date should be not asked for unless the user wants to specify it and it would be TODAY by default. Also the price list implementation should be utilized to override product's default pricing. | ||
| 2010-12-04 23:48 <plantian> I just need to make this widget and it seems like something that people would often need. | ||
| 2010-12-04 23:49 <plantian> price and inventory check | ||
| 2010-12-04 23:49 <cedk> plantian: empty date means current | ||
| 2010-12-04 23:49 <cedk> plantian: applying price list is not possible | ||
| 2010-12-04 23:50 <plantian> It needs party right? I guess I'm proposing a different interface. | ||
| 2010-12-04 23:50 <cedk> plantian: not only that | ||
| 2010-12-04 23:50 <plantian> Do you think such things belong in tryton client or would be in some outside interface? Similarly do you think something such as point of sale should be in tryton client or outside interface? | ||
| 2010-12-04 23:50 <cedk> plantian: price list can be based on many things like quantity, party | ||
| 2010-12-04 23:51 <cedk> plantian: it is possible to extend it to depend on the date, the location or any other stuffs | ||
| 2010-12-04 23:51 <plantian> cedk: Hmm right, that makes things very difficult. | ||
| 2010-12-04 23:52 <plantian> cedk: I guess you'd have to make a mock sale. | ||
| 2010-12-04 23:53 <cedk> plantian: by the way, is this issue still on topics http://codereview.appspot.com/90057/ | ||
| 2010-12-04 23:54 <plantian> cedk: I think I gave up trying to make the native version extensible and just rewrote it all. | ||
| 2010-12-04 23:55 <plantian> Let me look over it though. | ||
| 2010-12-04 23:55 <plantian> I think the problem was that inventory in tryton is per location but I need to take inventory at varying levels of the location tree. | ||
| 2010-12-04 23:57 <cedk> plantian: but the design of Tryton is that locations are there to make inventories | ||
| 2010-12-04 23:58 <plantian> cedk: Yes but I have view locations as nodes in a tree with the leaves as storage locations, otherwise it would be impossible to find anything. | ||
| 2010-12-04 23:59 <cedk> plantian: I don't see the issue | ||
| 2010-12-04 23:59 <plantian> So warehouse(Warehouse) -> Section C(View) -> Row 1(View) -> Top of Row 1(Storage). |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!