From d240ba5de1f3234220971acf961d89d954fd5110 Mon Sep 17 00:00:00 2001
From: Luis Ernesto Portillo Zaldivar <luis_portillo@siman.com>
Date: Mon, 14 Jul 2025 20:44:43 -0600
Subject: [PATCH] feat(#44): Add sample_type_product_id field to StockLot model
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Added sample_type_product_id Many2one field to reference sample type products
- Kept container_type field for backward compatibility (marked as legacy)
- Added @api.onchange method to synchronize fields
- Added get_container_name() method to retrieve container name from either field

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .../__pycache__/__init__.cpython-312.pyc      | Bin 320 -> 320 bytes
 .../__pycache__/product.cpython-312.pyc       | Bin 1401 -> 2960 bytes
 .../__pycache__/sale_order.cpython-312.pyc    | Bin 914 -> 914 bytes
 .../__pycache__/stock_lot.cpython-312.pyc     | Bin 1466 -> 4359 bytes
 lims_management/models/stock_lot.py           |  36 +++++++++++++++++-
 5 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/lims_management/models/__pycache__/__init__.cpython-312.pyc b/lims_management/models/__pycache__/__init__.cpython-312.pyc
index 03e48105f3c13e4c35712710173907af41ecae8d..c0fed3a2a0c9835cf86595f680a5d9766782fda5 100644
GIT binary patch
delta 19
ZcmX@WbbyKLG%qg~0}#~j*~n$a2mms#1gii5

delta 19
ZcmX@WbbyKLG%qg~0}$Ng-^gXg2mmo^1Xut7

diff --git a/lims_management/models/__pycache__/product.cpython-312.pyc b/lims_management/models/__pycache__/product.cpython-312.pyc
index 15c2049da44dbd2c864c2b1e27c3fc87c366bafe..945c75b16eac0f33cb5386ca9878493a03ba04e2 100644
GIT binary patch
literal 2960
zcmc&$&rcLr9Dg(Wg9TX!a9I#FOj~4Kt-DlHO_OSBse&4QR0xM{WHQcs3xhNB);BZM
zT`{4pNqg`BNfY#B+LH!N{0n;F*o%e(n@O9P^w9K%LNA{BeKWJLSZOa!x+IVHzVGw>
z@xDJkyFVwBO$a`he{!e4Ng(taKh!?4aI-NCn<a!177Szx1;G>xq8TYf1TGVes2MB7
z1SFyXgd=wljuK&}xn^Yh|8gYQ7aGd3Z2S*?$VR-lNii{OFJ9D%fo(6Mmh`OPNfWA}
zW7W~Ab)GTGv~4>G#lyqK_W}=$pn`x=LBv8KA|eu<@t;($fFMj9c^q9918E`d<M!^v
z@Y58+u^$B-f84NzP59WhzhiL%P!IwC6q<ZgIylv@xDhwuJ%Fq|zj@afn{l!(#-xus
zSZ^!XV*&5Q5^j0ix*QE81<A+CZ^5?VeRVcl0oOLu`xaaZx7S{b(|6Ev2q~m?)##|x
z=&aLdFYnKGdD1AO*wvhIVwMcmA(fU=AjmnP#O(wh=WW@w<a5@_4~A~*w%l*5Jh60*
zx*x#hkjgZb$+j*N+aYpjfWnC@sar}~sFcf!Ufgz=ZcTa7X=0SJaga1k6H|4lL8r<t
zZ_B~japj2$*Nqz7>!4M3TjBO|cjbwxYicbL4rvAfqu2Dn2Jf1oZ9!nANjFW`+IDi$
z3Xj2cl318(5Lkeaz{L4FNdSsbXPPiCI^r<u$1<*$sEo;Wda?p*1XZ_->)R!w=|u=t
zgCT*GjITV>Aa$x;*=Ojcol`B<C__$jOtq$nx2JYibX-YY0U5Qq@hhZA7_l^6t+ZUU
zWmO(ip<+ZHFPBJVpnsZZ*JZsZJJV2|umUn8B}QywIUMf*P^LxMP_I!2iDqTl4oqm&
za#Y<SELYiId&`guwPY%|z2OoDBVOtL;z&lfl|V<~)-vzQnKQ7AGnZtqy651vHFYyk
z&vVnnBA7A=Ca^^1k_v_y0p_Sc+xR`R)g;-7pE@s&sR=PL&JmUL1T|cfSX<Fh`G{#`
z-2u3)T|tUMr*)n{#ncV>gHl0S{=u);!Q;~z!SyeM8hCr)(Y2arA&r1vg!oW%ve+oV
zvV`W)XXq-7>}_FAm;uR~A6vNF0yiPG)!#DV?jfhf;=tZDvie$xqB6>g!)_7aW6+bb
z!JiQfDC|r=co;SsHL0B0ZAde6$t4(a!ArsTOgdr5zL_yK2l-EdGuF-*PT-G3FKQE`
z=rx3XZT<pTetj};T27wagf2O*VoWVN&-+<1Az4!ds^#Q^?~Ht?m@AdNZUx3zyS^jG
zMas4kT*;5_<fGibDD*)v_D8VHqZb_qULmn@RQztBDy7$?+^Up&`sLc`ORJ|Z{gOVs
zj!<Sq+(4*fRD^x&sQ5C*(I7#{YYF>$*6%dMH2$wrPeL^RWvPP)S=&RwOMD2NY?qjI
zz%eW<<DBJng;@`?v7<sqcd5`@u&21)9mN@}6(_SQWu7*y_4Tjz^@qh7@QX9(7iZ8f
z&Y;Nn_v&yq?Ije&Qca>LUX!9gU%3X~H!I2wS2e<$grZx(%rNYHG`)sTDK&^{`49CP
z$B02R{zJt}*2dXqF)um}P3(0#M4RTkY$#id&C#}(7_rC+=<2d3&8mh=6rZSp8f<Ux
zPD1n&LjXGgH17|Q7rS6k)d{lA=i6%nKIF8jTQ)lY0@e-7Bjmr*&pd-4cgSBj-i`v_
z+(A|8o%z8RZSq3zQf@I<m9o|3C!4XzzWDs*^#p1^yl`PH)$=^n^9nWe#@F{E>D^7K
zq5Zx2q4hQ-AG<gE;MV<HRq5DfM3m#tq6a~zqj#aWL>Fl_aj2STuS$nE8=@WY`H}Tz
zl<rxWSejg%1gF*H1h`4Z=ZDvuP<QTL@d3RLDqU6SXjST~CPz0LW8Gl!0Ql_eS(sg#
zTb!#(nLw=*E>fTuPtA{gJNeb*v()j2w^w6lUW$<z3|$1p&#wF19hlyS$<^2?jv4eZ
zr-N;fAfB7yY{H9DOey2k>ld2&sg<}eM{a7Q<d@ASFE2L4f3^l$&2t6bPkyFfCq53^
q=qx)4GEVHa49j{%5QN`R&vVrCD>}9r{ZJSbAz8+v@h9SY-@;$PcP!ih

delta 289
zcmbOr{*#OEG%qg~0}wRHlx93)p2#PmVh7|+XGmd4Va#F3WsG9XWr|{AWME=&XGmdc
zVMt+4Wz1rm7$q&sQpuvp`Vyp9lj#;)ZhlH?PVp_aw9M3;l;Y%xf}%`!7$zRP!&1qo
z$-g<0DUy+qee!1Jt;~L!e3KbDZFp~SXBNjN<|XD-7H1YucH&HBU&-(pWZLA(Y^H)m
zY(N&6U;#3UTNoz);%v!p^ed8wiTi1?-eSp5$<Hrh1_^<b-D33xxdNhB4kQXztpH^G
w;;;d_urw#tu1FckWd!142_W%-nURt4IfMKc7FkBN32D=FCgyx)05ZWk0WgI{OaK4?

diff --git a/lims_management/models/__pycache__/sale_order.cpython-312.pyc b/lims_management/models/__pycache__/sale_order.cpython-312.pyc
index 82b79cf563130e9f256a8880177c93d266636f1e..deaa7817a707854ba86d92bce727d0e4ddd50134 100644
GIT binary patch
delta 20
acmbQlK8c<CG%qg~0}#~jDc#83&I|xENChJR

delta 20
acmbQlK8c<CG%qg~0}yzxDA~x}&I|xCUj*6!

diff --git a/lims_management/models/__pycache__/stock_lot.cpython-312.pyc b/lims_management/models/__pycache__/stock_lot.cpython-312.pyc
index 3c5a0bbb56e07aa17b87a3a74e4f5aee9cd1c650..35ea7ee9a8647edcee88c33414e4783800702fd4 100644
GIT binary patch
literal 4359
zcmb^#OKcn0@h!PrE`OpV>Q7r*|0Jy<QB7PM1?<?7E!k-yD?ubX4r^nv;(L--CU=>)
zOS@dC01FtXbBcVDd~(r~1MA>(FYUb-GLB1j(Sv{<iUK{TmWu*8b>8k$Un|YQ?=U-W
zX5PGcGxO#R`R7C;ieUWrkENxvafJTPCe0D@k?kvhtRajrmq!II%M}9IKp~h7awveV
zBOG{$aFB4zVsqi{_j%C6v!OCChyKYXa?lY91}1sa5f(I($EFihi&}1X5ki7L+5Q8^
zb|xsxVU!JEE*lIW!Y{iv?_GevijClNkY*z*xKb_NY%jz-4q;&<yknPmq@;)=IJ!qN
z;!1Yx)#l+iwrA&<E7fyYY7yc1?>U^nEgQ+5-4jRd-iq6BJ5FtMJm)=K*_I<^J8>88
zeuy@DcJwBHz=~e5q7SF>v5n(9x?6GoLzHd90|48x1Tck90PMgg0d|76uI0?(GvQNB
zkK@x1(Q{uQ+k?-*s^0RTJmiS8mXW(RZCG~1WLPE-XxGn8NtLG6d4sB!LCeytS}5j;
z%sGN-QB7a8W0aVgqDn2DP$&GUYH38b?DQ{(hcr_udefm%Y3M@$bF@U{yK;cC{Bs0s
z6jV*Oqo$fC8G~X%?HFt&eMY`45!12<JSBN`UZEaozal$%)yU^b&e9BBnp7<!hva}0
zH3=;h6st5(?C31Rr8&TX6Cv1AJw{||a&DqY=gkM|yu;4|1gTI7&PD9xP4<zl8oC8W
z63RG+wFHL@eZoF=_YO;%D;J40JWUqWTv@h%I!TI@<iH**xz}b?Dw#wwUC${N$SOr@
z;8M;~G%RVlX%Q7Sox(4Xe9><81u_-^cY`K7Ij0p32@~lCa}8CS&e@~&fOF1$>TP03
z;7y4#n@k9eDv`Wo`O0!|I-(P7(D8z&R}Q+tq3qt$b^vyS1sD!MO*`)U8xA&K&iPl^
z%_WAj0<z6=Zw;-WWtY4kEC=?_*=!cc+zj|ONc8&-&$ju7b9j^F7fhy8`u^N_LAS=q
zui(sMD#nIxj_0+4sT5RQT_goK(YRNm#@(z{AZs(l^7rD!yjqyY>Lt6anb(;nNe0Cx
zD>)UkKKJzYliN@4KDk>#+dx6hID|1-P)m7B7FgosygVb=iG4<30y(E*6*wXUnrQb4
zw!49$&1)F^tM6VI%>bdOrn89Wh-uo<DP4lV0M0lf93o$~L2qONFwOR~!rd|`5Tg_M
zf|?_`zZcfbqG2-ONeE?b3CNBxWr8UNJqEVVHG`Lii6er?a`)hh?h6#FQEs3{0C5Ag
zH;h1I19Lc_am|Xsp)x!LGfoJ0vIy-50f<rP0o7pE1v4*Fl$BDaMRl(uTv*pHcK43=
z_cP9c@OBD71vSL@*U_({tJl}F)zt7SQKo%Be@9kR7Hm4e06Xy^dpmuz;B3@V+;Eth
zY4;yAdq4kYpvpXG{hexR<dyi|ySH9C()u3X`kcY4vPE_eaN^(r4vAb~TE`f8=k7jv
zq`~nWcSC~Pod=CHS>9iQ)rXvWyP6t&B@VrNTTdTpYcrMoYj-%6eZ~aSu;WOlYfnku
zQ|R~QGb@DdE&LZCN0m#u^yACua<c;jea+E)<$uj#ZiTy#Odbc=mx?&Jf)uvDJIXQ4
z`}EKOy5O%1`~2_$zUcEKK0k7RAKgDM$K<$uX|}BAmZ+g?Hj&r^<UX(NyC^+ag5vph
z|4C5q^hIJq87dKKz=u8UGzLd-c=oVUXo{h!Skg?RXwU?kgbaAYRhk5xwgQ-ud3v5P
zcwH?JCzLlHz<b`Z-w1dCx|vMee}fe!YqE$*l2_v-#efqqYm~R!YRfx`{r^NKL9h80
zfC}0YP`tGs8~iplxS1ZPr_a~Y=j-VUwe*F0dZLz|c$vQXXu6T?S|6<?2cHeSxKkUM
zXbgU^%D-wo)oANpzgTM<Y7D;b&wJPJ)Y{}{w_dE&#;!I7M}2b7`gE;r_}O(Jj{<qj
zC2#uQtzH!Wp)2+mu=B`l?fUWAI?Eaa9+Vvz^nw*`l0f1uzfv&Q3j%NZRndH{mRaNv
zjJNOBTrrEv0Xb-oUnkaXK1wX=(gHOKlAi~XL8ZLg;(1*iy$riJ;hXwB-FTqOVK=kr
zhd}f=Jl1lS6IT3RP7&y7yjIl-E)wg|L(~pLWj7Pq_I7$FjSe$!!bT<o6|@;kdTG;0
zb=OmwS}Iddovo$LR^w+IiMDz|`Zgi`E!DlwKaD?$SI?ZQ^_;Kwd{pcC=w)i6o|>wq
zrv8NM*S@G-`{HFPTa9NM$sV`P&d7a^sJQvBC{9#SpwTMj3E*)>`Ld+unUOFP6-|eZ
zShFaLsT00pK!>F2P6B=bu;$&nB#wAP)yp3kI&q@@?*iz5ov7d1GhAd%yrsd_O!^%c
zboL4cq7#{Iy3h5lca7t|ltE#&_tK9t|Bo~9J_BbNV7&%q1=10l7niz4`(vPe<Gp|G
z&895-T)+-u_Cjm)5H*s0m5(<&POe^Cn|VA_9l7#d$JNS>%{FPZYi;21Ktt>Ve>IY$
zZ^C@LQ2E7H9JQZlvN{^cV~ylbc3Itns|#z!W1~8D>ASAWl}|Q1&#d06cb=(ro_URg
zApuq=hpQR<CLwe!aFv;@R@6VXe(S07L}`e9jidzH8i~m_kx;)-<@>f;P{*m&^J|wM
zUv7x0M)LSyTSs8C4xus)ni9t!U8~2B)#Arqqj0|f!pZL6#a`ey$#CLgW$JZ17aDmj
zaG_BKTx{6Ipb!b|mD%6k`OTeb`^d}Cd;bo!gfiPfB#dskm?VUH-3)PkH6sfSZ(zfq
z>`8HR!1H$K2K)2CbL!`e#TpIQ(a@fIxpawUfyP=;^EiO5Ajfh4K&NZy^c((jj_ZAN
P|Lc{nR{nz+?k@iyGt44a

delta 406
zcmZoy+QrRxnwOW00SK-smuAdhp2#PmVh7|+XGmd4Va#F3WsG9XWr|{AWME=&XGmdc
zVMt+4Wz1rm=qD}9Qpuvp`Vyp9lj#;)ZhlH?PVp_aw9M3;l;Y%xf})HZ6Bk}&S;M@V
zX>tXlqBvg)e~Lg0!y3WW%pg4s3{iZQLYl&p6PR)t*(aZ4I?T`KoS#>cn3<Paq!3bB
zkUF`7rI1l`@++1^My1I<ta1YEKux#UQ&Q6sOLI#6H2Ei&@lCeA#g`1z6<-3>b&DrC
zKPM+Oxg@_RJ~L${Ly<U8SCJ@)V1p4XKt}O2hRLP;rS<Y(c-h1h8vH&8Fo-BmFq>{a
z(f$U9@C|<1n*v&2*jYr_8hs!}fOPw5vfg6JPsz_O0{Ni`WX>&CUy#oshO2`_;THbl
yumOfjX-=wLkrq&b5r~T=fW!x8Mn=Zx4Dw%C<QWCoI?DQMI%~c%$TPBmwE_U7&}UWv

diff --git a/lims_management/models/stock_lot.py b/lims_management/models/stock_lot.py
index 5bbc29d..e24cbf6 100644
--- a/lims_management/models/stock_lot.py
+++ b/lims_management/models/stock_lot.py
@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
-from odoo import models, fields
+from odoo import models, fields, api
 
 class StockLot(models.Model):
     _inherit = 'stock.lot'
@@ -26,7 +26,14 @@ class StockLot(models.Model):
         ('swab', 'Swab'),
         ('urine', 'Urine Container'),
         ('other', 'Other')
-    ], string='Container Type')
+    ], string='Container Type (Legacy)', help='Deprecated field, use sample_type_product_id instead')
+    
+    sample_type_product_id = fields.Many2one(
+        'product.template',
+        string='Tipo de Muestra',
+        domain="[('is_sample_type', '=', True)]",
+        help="Producto que representa el tipo de contenedor/muestra"
+    )
 
     collector_id = fields.Many2one(
         'res.users', 
@@ -57,3 +64,28 @@ class StockLot(models.Model):
 
     def action_dispose(self):
         self.write({'state': 'disposed'})
+    
+    @api.onchange('sample_type_product_id')
+    def _onchange_sample_type_product_id(self):
+        """Synchronize container_type when sample_type_product_id changes"""
+        if self.sample_type_product_id:
+            # Try to map product name to legacy container type
+            product_name = self.sample_type_product_id.name.lower()
+            if 'suero' in product_name or 'serum' in product_name:
+                self.container_type = 'serum_tube'
+            elif 'edta' in product_name:
+                self.container_type = 'edta_tube'
+            elif 'hisopo' in product_name or 'swab' in product_name:
+                self.container_type = 'swab'
+            elif 'orina' in product_name or 'urine' in product_name:
+                self.container_type = 'urine'
+            else:
+                self.container_type = 'other'
+    
+    def get_container_name(self):
+        """Get container name from product or legacy field"""
+        if self.sample_type_product_id:
+            return self.sample_type_product_id.name
+        elif self.container_type:
+            return dict(self._fields['container_type'].selection).get(self.container_type)
+        return 'Unknown'