From 0637f0a9e33c514d09a69989e503ff0302ce275b Mon Sep 17 00:00:00 2001
From: Luis Ernesto Portillo Zaldivar <luis_portillo@siman.com>
Date: Wed, 16 Jul 2025 21:49:50 -0600
Subject: [PATCH] =?UTF-8?q?fix(#69):=20Corregir=20error=20'Expected=20sing?=
 =?UTF-8?q?leton'=20en=20m=C3=A9todos=20de=20acci=C3=B3n=20de=20stock.lot?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Modificar todos los métodos action_* para manejar múltiples registros
- Usar bucle 'for record in self:' en lugar de acceder directamente a self
- Afecta a: action_collect, action_receive, action_start_analysis,
  action_complete_analysis, action_store, action_dispose, action_cancel
- Previene el error cuando se llaman estos métodos con múltiples muestras
---
 .../__pycache__/sale_order.cpython-312.pyc    | Bin 17447 -> 17447 bytes
 .../__pycache__/stock_lot.cpython-312.pyc     | Bin 24615 -> 24845 bytes
 lims_management/models/stock_lot.py           | 115 ++++++++++--------
 3 files changed, 61 insertions(+), 54 deletions(-)

diff --git a/lims_management/models/__pycache__/sale_order.cpython-312.pyc b/lims_management/models/__pycache__/sale_order.cpython-312.pyc
index cb430ed31c536f04b3d94e0b4d5e9c424161ad6a..61dbebfcb053b878b2570f1f351871a699ce70e7 100644
GIT binary patch
delta 21
bcmZ49!MMDGk?S-sFBbz4w0dvkl63(9La7Bt

delta 21
bcmZ49!MMDGk?S-sFBbz49O2x^CF=qJLfHjF

diff --git a/lims_management/models/__pycache__/stock_lot.cpython-312.pyc b/lims_management/models/__pycache__/stock_lot.cpython-312.pyc
index c5edc30b6b09c5b762e62200ee4158633e75475c..49bcfe06999c89c72a2bcbd441bf3e5669768d70 100644
GIT binary patch
delta 2246
zcmchYdraF^6vun$6KHuCTcFT5nBagG1_fcj6vn%d@u;N;th7JYN+0aD;2t&YrcSoF
z%yCs%Oa}gi^AAJwn>O-?f{&1BVrXQx_78)P8U4r1%_U}L#(M`tG7Pe0@t50A&OP_`
z-1EKX-rU&*IQKEceIb#=aL5-kc+_&S*B^H|UMe76F8SU3onVsV;nZ>ZTsXckjrV+%
z2k>hA007>fV3EBt%Htdp=82A{YmI1F=`z|!Y)q~z4}Xvl&f+2^`Fliw7?Ts3BhgNq
zdBEj1x)~RZt;Dm5rQXBD@^MyE({6Bt(Q)JTVOi$1_@a2cDJ)Jp-+HEXrt(wGQq5vb
zs8$!O)qPc~56XL1#0D}N5@!X)Sz&3>TwS2i-x$#PwV{&wAZbz(<%zIj&vhY3qP)f7
zN#cGJlHh0;S<3_j_dWq9d0po`ajomr{iAavC5PxtOd~q=>pFAq)0vd~CVx$hKUwX)
zM8dcX1lviPdTgd*SzfpzR&7#IwZK~vEsB;Ti;~d(jv#5$j<8I#GZptl#BQ&GOH|-H
zDM#Yl$UM|Ub6rc_BdzvcrrjN+zHeEsUJ(~<N?koy5-9hV2dey4p}hx!q)88iW#tb)
z^{-RMwxn)&q|SR~hpBJf_2Rj%fZneU^!fWjr42#Sqzz$N*+WfTv2k{iJKaviDBTuj
z(?to&4sH>pmgjpv+M%fOWqJOJxZr=TN6dyO8TO`0x%A4vmbSG#vUIigbE;&3;DIki
z=d#Wa()9;*{UMzxs56ChmLO?T%Y)P?fvkM<Kkj$thMbgIqr=45?km#CjP@->`ZlA`
zbajVCs$7<<R>bN}MN-XW2Xg(nfdYR)sJJdjnzZg;RpFDWboPY84zSAplKH*>%y=dz
z8yNPpoHBr4**m-6;erpp{rEX3Vl{i_A$^=A6%lqjh8@o59C<?IBYo6JF^*#kOz;%z
zEMNd8@YTWsK58ZdI7XEZ&1{(}g)f^R{o7F0h29`WglDr3Gq)iJG1yenl5mPfd6D8Y
z#Y_0ZlF|k<HANKl1QC(TjfUJ-JA)Y76}T89GFjHbajg#brhHze6Zlp)V|Sr3wxTo&
zU=mkpc0(s?)@1Tvnth?74Fs>#*i$%ESpl;+Y2VK<h5y{Q7v5!6RRE+@RIPz4tfg83
z(2P&lRKPsDSThUo32S}QBZ6=6PfgRYHX6iE;b32FPK87tu_MuAPTQ!R8L->X1P-)h
z3K8)hmv-QU))e-7iwd9@E43LV&(O(2(<nlW!<{EU^f<5W=^;8oo;cTl)r^ePJxGwJ
zKxRCmtq{1V$BpN-G7a4h^c}_b6hBb-D1M~4MnPAP2uz3}k0ffKo)P?)c2IDFnjSpV
zb^%85A8pz2C97yx^FhP5bzS4aAzY%j0)sE<r=gCm?x_VOEn|dNYcZm=ofm11j$_46
z2eXy|!fzSm0(w%A4=)&sQ%CO3?_!R+(GlvkQap=Ay`|l))NH3XNYP2rP4P5E55*9L
zmx50a5jh<ui_vkIL1(G&0`&>a%n0K!yO4ysCh<h?qcF<8*E<)rE1Dm7BXxVQ<Naqr
V+8GVW<74MNXFRt!)V}+o{|!6LZcP9H

delta 2168
zcmchYZERCj7{~i;U3WVf?55otOUELx+3uB>LIehbqU@#XMj6Z47E5gpwywP$r*|-L
zSdA!~34!GxTS5qYFpe2<TW;b&QPjj})WltwY3&EY6wnXG5S4`R!FZldOtZyB4e{pb
zFXuex+^7HlIp^ja{}@I-gvDQEW@c#kW1T+ei|v_MJh~(!jdxS|@7LRGnWZ4*2(bNI
zbJ*`o8rZ5exomHiQ#_Pz77ph%x@lKY*d6Q+5cYXiEGLB)@wejoP{8X7%Wj#3Q+S_!
zBw8bFyzzqOhB0rbez3k@9}^ax$QZ~N?zqx=)pgkwt!`GUo3B?pRl}}np@ok|g?v@W
zkBQ~u+DTzTn9Q8Wj8<(^d5GI$Mr$m;<W9Oqzx1v~tIwdzd2tVLJb3;x@D|SuY5Fv*
z*kXL(V|=Ly8a9*Tn{}}=*@fM3Mw`(5lc6edmcJZs;FvG`<c{I?DZ^vaf;ma+<CnHy
zt-V}(wefOebdy8nAv$8l)&ExOvwyl^6_!QwG_%(1?!}FK2CYMN39VmeZ;p&$?{Bqs
zP8lrILTQrLvJ3K9?`ZGXk<la3%B?C7acj)D_Tg%6%<Y-0Rn6TVnOIQk{B^AuH%{hF
z<V~6;Owm;~m4|4H8P_~itvQJkowZ92QBoxP$b)0;%ilTI!e{wqk@ps~&^Bc#o)${}
z=jAR)SjgV)4|j*cq$sp6*@wO0z+4|{K~>}v^wG@m507}$l(RMJY*(G_QKw6Fx}r|6
z%0u-2xBXydi#YFy=O%oJHn-#<fn*Qurlz?bE}1G?zFKfEI;ITfX~B}T7o``xW4=+}
zSa38Lt*B9Xh&6NND|Lna0N*Irmc63`4;v_60bb?p;`IPOD7Q=AOoa#=EIkeuWm(y~
zP;iVZh^OzCx}?xS$(&9jyiaQp$JlS>9@wa~RS<wd_CaO25Z6)S2~jV*Q&|Q!#Z+ZX
z%Nyi#??ZJ9eGMyV1O6^@A6D`?b=HQg7qQSwh?fyZ+3Qy84i7d3#FHFxAuQ8<vOh?u
z7rVMJaZ`_Pwp^-Tl9MF8f)mpGG6{xhk5aTQ1>iJWzP=djO8xpmEetFEjSkSA!rDXZ
z{Y@L-JiE8)M;Kx=)oWlx$=(dWJGood!9~Tf#RM?S0yP^zQATP;0In!a+xi5!#=fjS
zm(_(u0*D~T;X-!4A)lRUxF_NY#q|e5fu1071p{=DowXOje&ve24tBQVT+vC&gFS@%
zau?YsaeLkL5iHb!Z>D>CC?%3?*3qXhnOJKVj_}0|yZm0d7t?MIvx$0Gt7Ah8#Y`A+
z5fMjRN8CW*ve6mDcZly1iEX@zty_rw9C4k8Ymo`vfte7y>3CXq7@N;Ad($Au?AxXl
z@Trp8T$Kiu%G%wxQo+TR?(u_{9o=&ds+AS(HrRsO9oO1?ZaTX|LZe$cv^pe_Sq1n<
z^a;dfoT%SL0>mTxLlVV<MkDNvj@3CK9MZOsUYT}d)`#%3pF6C(@fx5G1b*4niD*Hz
zBJd`r1o1o~jUz6EB#+N6JwxaK=1yWR-AlTO<PFnIOr2)EosWX7oar1-SuUhy{9;@f
ZisX6~vh8{>^dC6UH_&%igYCpk{RiixU$OuI

diff --git a/lims_management/models/stock_lot.py b/lims_management/models/stock_lot.py
index dfe9ef6..8c7b8d9 100644
--- a/lims_management/models/stock_lot.py
+++ b/lims_management/models/stock_lot.py
@@ -145,74 +145,81 @@ class StockLot(models.Model):
     )
 
     def action_collect(self):
-        """Mark sample as collected"""
-        old_state = self.state
-        self.write({'state': 'collected', 'collection_date': fields.Datetime.now()})
-        self.message_post(
-            body='Muestra recolectada por %s' % self.env.user.name,
-            subject='Estado actualizado: Recolectada',
-            message_type='notification'
-        )
+        """Mark sample(s) as collected"""
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'collected', 'collection_date': fields.Datetime.now()})
+            record.message_post(
+                body='Muestra recolectada por %s' % self.env.user.name,
+                subject='Estado actualizado: Recolectada',
+                message_type='notification'
+            )
 
     def action_receive(self):
-        """Mark sample as received in laboratory"""
-        old_state = self.state
-        self.write({'state': 'received'})
-        self.message_post(
-            body='Muestra recibida en laboratorio por %s' % self.env.user.name,
-            subject='Estado actualizado: Recibida',
-            message_type='notification'
-        )
+        """Mark sample(s) as received in laboratory"""
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'received'})
+            record.message_post(
+                body='Muestra recibida en laboratorio por %s' % self.env.user.name,
+                subject='Estado actualizado: Recibida',
+                message_type='notification'
+            )
 
     def action_start_analysis(self):
         """Start analysis process"""
-        old_state = self.state
-        self.write({'state': 'in_process'})
-        self.message_post(
-            body='Análisis iniciado por %s' % self.env.user.name,
-            subject='Estado actualizado: En Proceso',
-            message_type='notification'
-        )
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'in_process'})
+            record.message_post(
+                body='Análisis iniciado por %s' % self.env.user.name,
+                subject='Estado actualizado: En Proceso',
+                message_type='notification'
+            )
 
     def action_complete_analysis(self):
         """Mark analysis as completed"""
-        old_state = self.state
-        self.write({'state': 'analyzed'})
-        self.message_post(
-            body='Análisis completado por %s' % self.env.user.name,
-            subject='Estado actualizado: Analizada',
-            message_type='notification'
-        )
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'analyzed'})
+            record.message_post(
+                body='Análisis completado por %s' % self.env.user.name,
+                subject='Estado actualizado: Analizada',
+                message_type='notification'
+            )
 
     def action_store(self):
-        """Store the sample"""
-        old_state = self.state
-        self.write({'state': 'stored'})
-        self.message_post(
-            body='Muestra almacenada por %s' % self.env.user.name,
-            subject='Estado actualizado: Almacenada',
-            message_type='notification'
-        )
+        """Store the sample(s)"""
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'stored'})
+            record.message_post(
+                body='Muestra almacenada por %s' % self.env.user.name,
+                subject='Estado actualizado: Almacenada',
+                message_type='notification'
+            )
 
     def action_dispose(self):
-        """Dispose of the sample"""
-        old_state = self.state
-        self.write({'state': 'disposed'})
-        self.message_post(
-            body='Muestra desechada por %s. Motivo de disposición registrado.' % self.env.user.name,
-            subject='Estado actualizado: Desechada',
-            message_type='notification'
-        )
+        """Dispose of the sample(s)"""
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'disposed'})
+            record.message_post(
+                body='Muestra desechada por %s. Motivo de disposición registrado.' % self.env.user.name,
+                subject='Estado actualizado: Desechada',
+                message_type='notification'
+            )
     
     def action_cancel(self):
-        """Cancel the sample"""
-        old_state = self.state
-        self.write({'state': 'cancelled'})
-        self.message_post(
-            body='Muestra cancelada por %s' % self.env.user.name,
-            subject='Estado actualizado: Cancelada',
-            message_type='notification'
-        )
+        """Cancel the sample(s)"""
+        for record in self:
+            old_state = record.state
+            record.write({'state': 'cancelled'})
+            record.message_post(
+                body='Muestra cancelada por %s' % self.env.user.name,
+                subject='Estado actualizado: Cancelada',
+                message_type='notification'
+            )
     
     def action_open_rejection_wizard(self):
         """Open the rejection wizard"""