From 820c05ffa9114284298601de55c6e48f5a511499 Mon Sep 17 00:00:00 2001
From: Luis Ernesto Portillo Zaldivar <luis_portillo@siman.com>
Date: Tue, 15 Jul 2025 01:17:37 -0600
Subject: [PATCH] =?UTF-8?q?fix:=20Implementar=20generaci=C3=B3n=20autom?=
 =?UTF-8?q?=C3=A1tica=20de=20pruebas=20y=20mejorar=20selecci=C3=B3n=20de?=
 =?UTF-8?q?=20muestras?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Agregar método _generate_lab_tests() en sale.order para crear pruebas automáticamente al confirmar orden
- Agregar método _find_sample_for_analysis() para asociar muestras con análisis según tipo
- Mejorar dominio de sample_id en lims.test para filtrar por paciente y estado (collected/in_analysis)
- Agregar método _onchange_sale_order_line() para actualizar dominio de muestra dinámicamente
- Las pruebas ahora se crean automáticamente con la muestra correcta asignada

Esto resuelve el problema reportado donde las órdenes aprobadas no generaban pruebas
y las muestras no estaban disponibles para selección manual.

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

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .../__pycache__/sale_order.cpython-312.pyc    | Bin 7204 -> 10422 bytes
 lims_management/models/lims_test.py           |  32 ++++++-
 lims_management/models/sale_order.py          |  81 ++++++++++++++++--
 3 files changed, 106 insertions(+), 7 deletions(-)

diff --git a/lims_management/models/__pycache__/sale_order.cpython-312.pyc b/lims_management/models/__pycache__/sale_order.cpython-312.pyc
index 50afd7af0e0aa273ce2fdea2a8ef176822a247e3..b4b681664a49a97e4f3ca138d394e42d08680818 100644
GIT binary patch
delta 3444
zcmai0TTB~g7M_3X@dZ41usyzT%W#i#fh8n_+(?rEfi^&sKpO(YG1zHMjE!bYQ!Epw
z(yAg_q(bK*(O$I8s#OzZrKUpKCh9|z?XHq)ch#9f*OnEv>Avkt<&f-+eb_x`j3Hg^
zZhyh&|8mZmbI$zdJM(VquZ|b}j%5od7+?Hr>eWA0FBE-Ee!uYT$IgKJhtz3nzyrw5
zQm>QK#V09m$>Ji}PZLqj>Uv4Fd{AFUCuq5ncgT%q(`G`=)#|APIZ0JhF$>znQA3Gm
zXLPZm+=`;ci{*PJp3w3F=P>+)3g}|I{IR~;>&gpk{Kh=_c|d=Y<mMo4WVZ8j0ppDB
zqV58@K)tRTq%IJzDSlAyX1vBoK!^pUm>gvU;+5wax2qA?l5j363Q;i-6oplKuP`+s
z-(kuOjg#V3a8zDmT8+~}Zf-K{BCOnIXtUdWuLL7O(H{%?!v0a8W;Sm4IVqD%j8){f
z@&RL=*J8HKc4X;7)|Te@6t`97wk~tqwYgX2ykBx#vqq4~mdTflp3d$EwV!YKY|Dc(
zW#HxIQ$xz|$kM4Z%F8~b;_Nc#&l;#=XNs#*xvGbrk}O5aPLL-QDLF|V4xCa3PcID&
zJ=T$uVfmr4wA2mxxFH;3>3gjCa0_$KdB}OViTQOCbGT)ZGo8@2u9}XBLGV2w#O;FQ
zpO_2>zmo+0{;6==s(ybs)V8V<whODw{zjb3KQ3&p!M7peBfQp3K`<gsi9uf!a#0Ce
z8GR@M$*h(i7ygK?ML`1!8pS&Kvc+%MiaQl@)>1?EEb{EU-jFjU-z?r&*oBABBXrA;
zi`zN0C+<e5MyQoLY<o$ae92bIAB7Ex0T6JAn%NflmTfy3S^Ua&gpiKKD#y395#>6;
zoXb_tqvhT|>j2}rhK7a=<u!}P<Zf3rX_F^hw~~Euk|~(<i!q4B7Yc|&u%SIVqaTlk
zBH{sDx1y9`Ov|1YvA7Gj1%xVu8U!PP7P&#J!}(zZ^d_T!%^#M;Be?bgNGOFtnDcS$
zmEUvMlQ$Rt;0_S?*xJ<wA&SCUME{={7};w&PfSoP4igr_3`vlDt?6gz1SMBm9hP=#
zo{SX2sV<?Dn}|ccrgxYmGFO_Q#_clU?L^z=^7S*LPw1tI7%hLym+SI6M+;m;FMngM
z*#=twzYv+cp|!^yEpU{?A-;g&m&ThkxCsM@;5{=C!FVY;pwo?)X}@~-y=St!Oh2Xb
zMrN1<onYb)@Sm1@Oq+D(1S3aHPJN(Y-M&-)huKmwPv(k%TxjvlfOW>0pxddDZJ@;@
zi~%mMQw$SK0ff_I`xz5wLBKfI0BXj7J-7Pa_#AoH62^_N<PKB4hmYZ1*G6JJ03a|Y
zCa!9_Xu1e!uOcr3wD-o0A-0wgpuI|rU01jsmb)m1S6&+l!%ZSTHaosQ{p`AXxwj4J
zPkz@<&f+<G!~5J0p$4fy(Tw4uVV<5V&&{5pPgi`$(KWIYfb62cdQov&qtYB}_#^)C
zv=oxoI5ZrJ1f_T#=p>OLLy=ehyIPKM<K}Q^LTW$;jhFXE*XRf+c?#$niFw_6yg&#=
z)}Ar74_rSKcr7T#f|0S1f3JY5K>-%gNKE{6QJ9ztO39e$7oep5LNE+=KD`zWp_}!!
zQoXoAV|2zQ`NKeB*a3X#J(PdOoa059#@ReuqLV_r=s(<YcZeeRsgVK(t4X2fVviHN
zj92H*{b5Z+%=MBrztEMvLao#*HiD@51B51o9SF?;Q%E2$YOUyIVO6pLR`KFzlAp_>
z*Dmfv4UIH4O2QkoMpvU^^nw%=U%;JX2t5E9kMCS467a2s3I6)>na;89I(>@))Ioj}
zg(Ttz(IF>#O&Z5#%>9s^o@gK#&agRx3}}5(68ljJDKDeP+-Ss?G2;}n4gJ>yN63LV
za%8><cH-qH;jQ%HOKDi|w#oH&D_sW}=?qlSCF+sM@|N|ob=l;3XtKU#xolBf2W|&G
z3Ec_Z9#Z)3Wz+M}KY7oUu7$3vL#o3&Yfif>Q|`^Gd-IZe%PgBa?OW)3LRCQ1Zcp1?
zR}L;5OxYX0us5VRH=H|5Qhc+@C!24zf7JJ3-xA+G+xd;%^N@30u`XEC&b9s2&MtLl
zm(tw*phW3EmFgc+`$v?MXB3C;sg5qTVH<}}fl8IDgf^~}o?7w@D#K?}!{^oE^QqxU
z75+Sv>B`0|Llsv&rF5WTqKZ9Pg9W}KTtX`8s#jh0=`9T()ZeI2SJuDpx#oGy=)ATk
zl+JCR*Q0dBI}VkrooCTE%Yx<V%=P{c25$^rf8o}}yMg=Bd(l+qX|?mT(lMlz5C4%H
z$u>dIj}H^d;z_$JX1mkw+7*hhy<}dW<~&zp@3g<)eXaYQeb>is)g(V^{ID^#qf_0{
zsWctEJEfHOs9ev2Ijcj3M;897OSK5I-H$9>%2KXc${+Gh#dUCrf9~!cl|Pm)-wKhj
z6%ZL)^wdPLj<=3oK6bVB-6Pl8rJ5F{dbhH>>w)8Q&u5+og^H*DOUpo(2fzOQw3u>~
zuTV^qwP9H70;>qEw;i8sy0a;@r$^n>qqO!u7+pTzuME7jbo^!I#ZyYf;4*hQ%{iV)
z_j600LwEb{`|kNt9fNAe;PTKJ#dmgT$gi}Is-Dq&-v5SqUqSe1mSI@is#F4Z?Kk>E
zbw_u#(Dz%+&r{6(gNItW_c5RCV_-41&gCG7EgJuy`~u(#_Bu*diySr61Uk&Kf^duj
z+7;9}bZvTEFCTZ<<*oud4gI{&!Q(V}ely&2GS`%#$7n*Gr1aD@@zU|;PN-fX_G(b@
zPfm)_NihT;bQ|o6ytzT0AP|@ui*<VSVlO;W+>g+PfMxB~<-Sq|a0Ba9LjTu_F-eKz
zu!>hdQ;q+NIKD>R;{cZ^xapiLWo`Jv+K{$m0l7+3&PLVQxI)q9irLPzl}}l>sn%^N
zYolsyytO;kd_rwLk!l`Ln+KMx18G<J+iaGh&1?nSX^s~=z^vCMH<uh6!!NUpxyK)w
z-Z24?i>3@VMeMO+3&LImq!G>g4D0iah5eE=77T}dJ}(isz>$PsSI^}b$Bd!~NJi4z
za=Ii|ZD;H&TdJ6%l`c}xI9E;(0p{&w7xNPNYlIXs;FLUC+P8fYb&+{9jK+VOog0+q
jvSxkNw0{C#KseYYUX;HoZQ@6Dgr7HHb^e9IxhDE=$Ul{7

delta 1036
zcmYk4OK1~O6o%){D@~fDV`381nmSEF(jbW%AC#I#`|?q>(i9){6^CZ5Dbpn0nTUpp
z8wEudmJ7NPT)0pK5dxw}vAR)kQ&Fi5g18bl8lMYM@Sa4$yZGkZbN+Mgx#!N^k?UQ~
zXAXyrp(|M{AHBC^#`zY_&3t@dJI9<r`&ves*pli)lwwha{Njd@n{4AF>5CqonMR{b
zh%xG^kRE36;ZvMp2OHL8OXK@LbL=!bDUlJchvN)4dC!NChbT+k9^XGzj=zN(HjS?q
z;%J&Ef*Y+RWg$jx2!VmnAjU;3PpAbID@MLBA?wOiNmF(C2z3{6PQ`MC?Uu{?X_<yr
z(f-P@JXKb80~4RvMaIM=vXhHqn!FXSqBe5M5=3{&HA^JWY;~O*tg$wS>$T)rkd{A}
zmcNqXHG%qTt>lx%-`_ts_+;qO&|K;%ewo=dzkAP%%-;EtvHA9WFQxr8ky+$jkb=*p
z;2VEyjX_Jdpsx&S-HP698-9B1*}BghYR!=0rZ&Hq_IOk3VP~y<5#f>8>+S0h9(M?R
zk?NVI42LRYp}E~(;ZtQTO{wM66;6)J6(O|(Y~r-94nqkH<BPSJHnpIav6?N?*({?j
zlg}6ER62-izu*XgPz0d`hskfdBF4ZIAUQ`Ebyd$hF7aZc7+L08-@FlmeSlt~x&|bu
zhgSeP03mYU)x9cAO;en}#d1kEc`aX1O-l(EbLDXZZ=f+<ppaqV`e6wYcE{0yYR27<
zP*3&p;*YEaberfQ7kmK^oHzV|qpWa=M8afZRy;sveIc}tyz|{Ty_rT#TS>tNy<s+=
z!=uzuZ^+~)i}?ax3;UIz634Ns7>zmLIJnyZG9U=B0P6D)G28=p1E3p|KdxvxUJv^y
zg$^pXg!SJ&lEm+iqO;Xre-5!RiQJBCAA?p*>kg%GGC4(o*Fbt3AOJ`Jk`$&Rn;q8_
zT_0C9Et`!Y+)2GU{Pi0fQ?h~-&;)_A<Y3hBx&-%^&Y<A@vIRv2?~e?U1t;zy-=aI?
zhd>U`V+uQpIaRBt-ehnO_+jinn^IHsO*A!)HB#za<vGYA=Oyuqea8Ns0lO~!18xcZ
Ay#N3J

diff --git a/lims_management/models/lims_test.py b/lims_management/models/lims_test.py
index c047b98..a098d79 100644
--- a/lims_management/models/lims_test.py
+++ b/lims_management/models/lims_test.py
@@ -47,7 +47,7 @@ class LimsTest(models.Model):
     sample_id = fields.Many2one(
         'stock.lot',
         string='Muestra',
-        domain="[('is_lab_sample', '=', True)]",
+        domain="[('is_lab_sample', '=', True), ('patient_id', '=', patient_id), ('state', 'in', ['collected', 'in_analysis'])]",
         tracking=True
     )
     
@@ -110,6 +110,36 @@ class LimsTest(models.Model):
         for record in self:
             record.require_validation = require_validation == 'True'
     
+    @api.onchange('sale_order_line_id')
+    def _onchange_sale_order_line(self):
+        """Update sample domain when order line changes"""
+        if self.sale_order_line_id:
+            # Try to find a suitable sample from the order
+            order = self.sale_order_line_id.order_id
+            product = self.sale_order_line_id.product_id
+            
+            if order.is_lab_request and product.required_sample_type_id:
+                # Find samples for this patient with the required sample type
+                suitable_samples = self.env['stock.lot'].search([
+                    ('is_lab_sample', '=', True),
+                    ('patient_id', '=', order.partner_id.id),
+                    ('sample_type_product_id', '=', product.required_sample_type_id.id),
+                    ('state', 'in', ['collected', 'in_analysis'])
+                ])
+                
+                if suitable_samples:
+                    # If only one sample, select it automatically
+                    if len(suitable_samples) == 1:
+                        self.sample_id = suitable_samples[0]
+                    # Update domain to show only suitable samples
+                    return {
+                        'domain': {
+                            'sample_id': [
+                                ('id', 'in', suitable_samples.ids)
+                            ]
+                        }
+                    }
+    
     @api.model_create_multi
     def create(self, vals_list):
         """Genera código único al crear."""
diff --git a/lims_management/models/sale_order.py b/lims_management/models/sale_order.py
index 4f2227f..5024cb5 100644
--- a/lims_management/models/sale_order.py
+++ b/lims_management/models/sale_order.py
@@ -34,20 +34,21 @@ class SaleOrder(models.Model):
     )
     
     def action_confirm(self):
-        """Override to generate laboratory samples automatically"""
+        """Override to generate laboratory samples and tests automatically"""
         res = super(SaleOrder, self).action_confirm()
         
-        # Generate samples only for laboratory requests
+        # Generate samples and tests only for laboratory requests
         for order in self.filtered('is_lab_request'):
             try:
                 order._generate_lab_samples()
+                order._generate_lab_tests()
             except Exception as e:
-                _logger.error(f"Error generating samples for order {order.name}: {str(e)}")
-                # Continue with order confirmation even if sample generation fails
+                _logger.error(f"Error generating samples/tests for order {order.name}: {str(e)}")
+                # Continue with order confirmation even if generation fails
                 # But notify the user
                 order.message_post(
-                    body=_("Error al generar muestras automáticamente: %s. "
-                          "Por favor, genere las muestras manualmente.") % str(e),
+                    body=_("Error al generar muestras/pruebas automáticamente: %s. "
+                          "Por favor, genere las muestras y pruebas manualmente.") % str(e),
                     message_type='notification'
                 )
         
@@ -163,3 +164,71 @@ class SaleOrder(models.Model):
             raise UserError(
                 _("Error al crear muestra para %s: %s") % (sample_type.name, str(e))
             )
+    
+    def _generate_lab_tests(self):
+        """Generate laboratory tests for analysis order lines"""
+        self.ensure_one()
+        _logger.info(f"Generating laboratory tests for order {self.name}")
+        
+        # Get the test model
+        TestModel = self.env['lims.test']
+        created_tests = TestModel.browse()
+        
+        # Create a test for each analysis line
+        for line in self.order_line:
+            if not line.product_id.is_analysis:
+                continue
+            
+            # Find appropriate sample for this analysis
+            sample = self._find_sample_for_analysis(line.product_id)
+            
+            if not sample:
+                _logger.warning(
+                    f"No sample found for analysis {line.product_id.name} in order {self.name}"
+                )
+                self.message_post(
+                    body=_("Advertencia: No se encontró muestra para el análisis '%s'") % line.product_id.name,
+                    message_type='notification'
+                )
+                continue
+            
+            # Create the test
+            try:
+                test = TestModel.create({
+                    'sale_order_line_id': line.id,
+                    'sample_id': sample.id,
+                })
+                created_tests |= test
+                _logger.info(f"Created test {test.name} for analysis {line.product_id.name}")
+            except Exception as e:
+                _logger.error(f"Error creating test for {line.product_id.name}: {str(e)}")
+                self.message_post(
+                    body=_("Error al crear prueba para '%s': %s") % (line.product_id.name, str(e)),
+                    message_type='notification'
+                )
+        
+        # Post message with created tests
+        if created_tests:
+            test_list = "<ul>"
+            for test in created_tests:
+                test_list += f"<li>{test.name} - {test.product_id.name}</li>"
+            test_list += "</ul>"
+            
+            self.message_post(
+                body=_("Pruebas generadas automáticamente: %s") % test_list,
+                message_type='notification'
+            )
+            _logger.info(f"Created {len(created_tests)} tests for order {self.name}")
+    
+    def _find_sample_for_analysis(self, product):
+        """Find the appropriate sample for an analysis product"""
+        # Check if the analysis has a required sample type
+        if not product.required_sample_type_id:
+            return False
+        
+        # Find a generated sample with matching sample type
+        for sample in self.generated_sample_ids:
+            if sample.sample_type_product_id.id == product.required_sample_type_id.id:
+                return sample
+        
+        return False