irc.libera.chat #tryton log beginning Tue Jan 11 12:00:01 AM CET 2022 | ||
-!- irclog(~irclog@moretus.b2ck.com) has joined #tryton | 23:49 | |
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton | 23:59 | |
-!- ChanServ changed mode/#tryton -> +o cedk | 23:59 | |
-!- mrichez(~Maxime@2a02:a03f:c2e8:f900:ed77:85ea:af2b:ba6e) has joined #tryton | 06:04 | |
-!- springwurm(~springwur@5.104.149.54) has joined #tryton | 06:38 | |
-!- timitos(~kpreisler@host-88-217-184-172.customer.m-online.net) has joined #tryton | 07:32 | |
-!- rpit(~rpit@p200300c88f3e0900e532e79838eb182a.dip0.t-ipconnect.de) has joined #tryton | 07:43 | |
-!- acaubet(~Thunderbi@74.126.185.52) has joined #tryton | 08:07 | |
-!- nicoe(~nicoe@2a02:578:852a:c00:18c2:1aff:fef9:2b7f) has joined #tryton | 08:41 | |
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton | 08:43 | |
-!- ChanServ changed mode/#tryton -> +o cedk | 08:43 | |
-!- doc_duc(~usuario@185.164.101.222) has joined #tryton | 09:49 | |
LordVan | cedk, thanks that's what I gathered from reading the source. no real issue though (except that some people do not like missing numbers but well there is a simple solution: make less mistakes have less missing numbers XD) | 10:03 |
---|---|---|
LordVan | If I want to copy the sale reference to a shipment reference on creation I should do that in Sale.create_shipment - correct? | 12:12 |
pokoli | LordVan: _get_shipment_sale | 12:17 |
pokoli | LordVan: but with sale_shipment_grouping it is possible to group multiple sales into a single shipment | 12:17 |
LordVan | pokoli, we do not intend to group shipments for simplicity reasons | 12:19 |
LordVan | pokoli, that seems simple enough .. so I just call Sale._get_shipment_sale(Shipment, {'reference': self.reference }) (not in this shortened form ofc ^^ | 12:30 |
pokoli | LordVan: you get the shipment from super call and set the new values needed (reference in your case) | 12:31 |
LordVan | pokoli, yeah prolly better that way around i guess ^^ will try. thanks | 12:32 |
cedk | LordVan: shipment reference is used for the carrier shipping reference | 12:36 |
cedk | LordVan: copying such data is most of the time a bad idea | 12:36 |
LordVan | cedk, 90% of the time customers pick up their goods or we deliver them ourselves so in our case this should be just fine | 12:37 |
LordVan | cedk, or do you mean it would be better if I made an extra field instead in case we may in the future want to use it for things like UPS reference,.. ? | 12:38 |
cedk | LordVan: I mean it is often a bad idea to copy static data | 12:43 |
cedk | LordVan: on shipment you have the sale list so you can add a Function field that compute all the references | 12:43 |
LordVan | cedk, true .. well to be honest I only was considering copying it to show it in the shipment list so that it is easy to find certain orders shipments there | 12:45 |
LordVan | cedk, but what you say makes a lot more sense of course .. I'm going to have to look into how a Function field like that would work as I haven't really usedthem yet (i read some code but never tried it myself) | 12:45 |
LordVan | assuming i can addd a function field to shipment_out_tree | 12:47 |
cedk | LordVan: the origins field on Invoice could be an inspiration | 12:47 |
LordVan | cedk, thanks | 12:48 |
cedk | LordVan: also do not forget that you have the relate on shipment that point to the sales | 12:48 |
cedk | and vis-versa from the sale to shipments | 12:48 |
LordVan | cedk, I do know about that, but we just want it shown in the shipment_out_tree list | 12:49 |
cedk | LordVan: maybe but I do not get the point, such reference is meaningless. The information are on the related document | 12:51 |
cedk | for me this sounds like frequent requests from unexperimented users who are pointless once you know the software | 12:51 |
LordVan | cedk, i fully agree, but that doesn'T mean that the people who have been doing stuff in spreadsheets for many decades don'T want to have a similar list as they used to | 12:51 |
cedk | when a customer requests to me such thing, I really do all I can to not implement it | 12:52 |
LordVan | and since showing an extra column is not hard i do not really want to spend 10 times more time to explain it | 12:52 |
LordVan | cedk, there are other things that are more of an issue that i am being more insistent on but this is something simple so I am willing to humor them for now | 12:52 |
cedk | I do because customization will need to be maintained forever so I prevent to spend once some time than a recurring cost | 12:53 |
LordVan | cedk, well it is our internal system not for a customer so my "customer" is my boss .. ^^ | 12:53 |
-!- springwurm(~springwur@5.104.149.54) has joined #tryton | 13:18 | |
-!- udono(~udono@027-131-067-156.ip-addr.inexio.net) has joined #tryton | 13:20 | |
LordVan | how/where are those relates defined btw? | 13:31 |
LordVan | ah nm in the xml | 13:33 |
LordVan | hmm is there a method already to get all sales associated with a shipment? or do I need to just get all sales related to the moves? | 13:34 |
LordVan | since from what I can see i'd need to get all the outgoing moves for the shipment, then the corresponding sale lines and in turn the sales for them correct? | 13:36 |
LordVan | although .. if i do that in a function field that is displayed in the shipment tree .. and there are a lot of entries that would be quite a performance issue i think ... | 13:40 |
LordVan | i'D be potentially be checking thousands of moves, sale lines,.. | 13:40 |
LordVan | especially if done in python code with some loops | 13:41 |
LordVan | i think i will actually just make a char field that gets filled on creation of the shipment to avoid performance issues in the future. but thanks anyway | 13:55 |
-!- Pegasus_RPG(~Thunderbi@72.89.118.36) has joined #tryton | 16:13 | |
-!- rpit(~rpit@p4ffb7220.dip0.t-ipconnect.de) has joined #tryton | 17:10 | |
-!- rpit(~rpit@p200300c88f3e0900e532e79838eb182a.dip0.t-ipconnect.de) has joined #tryton | 17:15 | |
-!- Pegasus_RPG(~Thunderbi@pool-108-30-108-198.nycmny.fios.verizon.net) has joined #tryton | 20:19 | |
-!- Pegasus_RPG(~Thunderbi@pool-108-6-41-103.nycmny.fios.verizon.net) has joined #tryton | 22:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!