IRC logs of #tryton for Saturday, 2010-12-04 #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> -- 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@ 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@ has joined #tryton
2010-12-04 02:02 -!- zodman(~zodman@foresight/developer/zodman) has joined #tryton
2010-12-04 02:56 -!- sharoon(~sharoon@ has joined #tryton
2010-12-04 03:10 -!- pepeu(~manuel@ has joined #tryton
2010-12-04 05:04 -!- sharoon(~sharoon@ has joined #tryton
2010-12-04 05:18 -!- yangoon( has joined #tryton
2010-12-04 07:01 -!- sharoon(~sharoon@ has joined #tryton
2010-12-04 07:11 -!- zodman(~zodman@foresight/developer/zodman) has joined #tryton
2010-12-04 08:20 -!- GasbaKid(~GasbaKid@ has joined #tryton
2010-12-04 08:59 -!- paepke( has joined #tryton
2010-12-04 09:04 -!- GasbaKid(~GasbaKid@ has joined #tryton
2010-12-04 09:45 -!- chrue( has joined #tryton
2010-12-04 10:00 -!- paepke( has left #tryton
2010-12-04 10:17 -!- paepke( has joined #tryton
2010-12-04 11:09 -!- Timitos(~kp@ has joined #tryton
2010-12-04 11:11 -!- enlightx(~enlightx@ 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@ has joined #tryton
2010-12-04 14:12 -!- enlightx(~enlightx@ has joined #tryton
2010-12-04 14:46 -!- sharoon(~sharoon@ has joined #tryton
2010-12-04 14:55 -!- paepke( has joined #tryton
2010-12-04 14:56 -!- sharoon_(~sharoon@ has joined #tryton
2010-12-04 15:58 -!- GasbaKid(~GasbaKid@ has joined #tryton
2010-12-04 16:04 -!- pjstevns( has joined #tryton
2010-12-04 16:37 -!- udono( has joined #tryton
2010-12-04 17:00 -!- vladimir_(~vladimir@ has joined #tryton
2010-12-04 17:17 -!- paepke( has left #tryton
2010-12-04 18:07 -!- enlightx(~enlightx@ has joined #tryton
2010-12-04 18:13 -!- gremly(~gremly@ has joined #tryton
2010-12-04 18:19 -!- GasbaKid(~GasbaKid@ has joined #tryton
2010-12-04 18:22 -!- GasbaKid(~GasbaKid@ has joined #tryton
2010-12-04 19:24 -!- sharoon(~sharoon@ has joined #tryton
2010-12-04 19:28 -!- pepeu(~manuel@ 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( has joined #tryton
2010-12-04 21:06 -!- GasbaKid(~GasbaKid@ 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( 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
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 2.17.3 by Marius Gedminas - find it at!