From 92f8894164c5f784ba1f8b3eae390daaf6a7dfc1 Mon Sep 17 00:00:00 2001
From: Luis Ernesto Portillo Zaldivar <luis_portillo@siman.com>
Date: Tue, 15 Jul 2025 11:29:41 -0600
Subject: [PATCH] feat(#51): Task 2 completada - Crear modelo
 product.template.parameter
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Creado modelo product.template.parameter para asociar parámetros a análisis
- Campos: sequence, required, instructions
- Relación Many2one con analysis.parameter y product.template
- Agregadas vistas embebidas en product.template
- Actualizado analysis.parameter con relación One2many
- Configurados permisos de seguridad

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

Co-Authored-By: Claude <noreply@anthropic.com>
---
 lims_management/__manifest__.py               |   1 +
 lims_management/models/__init__.py            |   1 +
 .../__pycache__/__init__.cpython-312.pyc      | Bin 473 -> 523 bytes
 .../__pycache__/product.cpython-312.pyc       | Bin 2974 -> 3188 bytes
 lims_management/models/analysis_parameter.py  |  24 +++-
 lims_management/models/product.py             |   7 ++
 .../models/product_template_parameter.py      | 109 ++++++++++++++++++
 lims_management/security/ir.model.access.csv  |   2 +
 .../views/analysis_parameter_views.xml        |  20 +++-
 lims_management/views/analysis_views.xml      |  15 +++
 .../product_template_parameter_views.xml      |  93 +++++++++++++++
 11 files changed, 265 insertions(+), 7 deletions(-)
 create mode 100644 lims_management/models/product_template_parameter.py
 create mode 100644 lims_management/views/product_template_parameter_views.xml

diff --git a/lims_management/__manifest__.py b/lims_management/__manifest__.py
index 761494f..68fe505 100644
--- a/lims_management/__manifest__.py
+++ b/lims_management/__manifest__.py
@@ -36,6 +36,7 @@
         'views/lims_test_views.xml',
         'views/res_config_settings_views.xml',
         'views/menus.xml',
+        'views/product_template_parameter_views.xml',
         'views/analysis_parameter_views.xml',
     ],
     'demo': [
diff --git a/lims_management/models/__init__.py b/lims_management/models/__init__.py
index 4e5f5d0..f50d315 100644
--- a/lims_management/models/__init__.py
+++ b/lims_management/models/__init__.py
@@ -1,5 +1,6 @@
 # -*- coding: utf-8 -*-
 from . import analysis_parameter
+from . import product_template_parameter
 from . import analysis_range
 from . import product
 from . import partner
diff --git a/lims_management/models/__pycache__/__init__.cpython-312.pyc b/lims_management/models/__pycache__/__init__.cpython-312.pyc
index e5d6de68932cf682ebf3132c4f53bf3c2e6e50f0..7f68a96b13adb7f79d499848b2ad63836b1d84cb 100644
GIT binary patch
delta 114
zcmcb~+|9y!nwOW00SHcYmt`1C<dtNsnW$bUkiw9{mBWzB6~)EKP|2;yGjWcbpj1Io
zeoAR_Nqk9aZb43BN$SM4O^n==4H)&Aig+f+FnUV}0?lCr;$meW@qw9<k?|>m!d(W9
M$$J<jS@?ls0Ka$}^8f$<

delta 78
zcmeBXxyj6XnwOW00SJ0p$}-p|@=7umOjIvqspQh+p14A8;*(NFuE|l1`jh(@Jwye7
c${2yTSOG|UU}j`we9E9O`5mJq3m;Gb0RQO`od5s;

diff --git a/lims_management/models/__pycache__/product.cpython-312.pyc b/lims_management/models/__pycache__/product.cpython-312.pyc
index a2b65f234d246ac31c8174ae2efa74aff3fdba11..6f3e4b1f27867da07a60ef80550919edb9541f90 100644
GIT binary patch
delta 599
zcmbOy{zZcCG%qg~0}$NrF3ZT~oXEG4@xa6byCw%PiU{+h@TTyk@ULN6%>q)vz!1ed
z@uH-tK#CwxhzTmh$Hb7zk|j4ekWpBYF-0gvI7MWQC|rX8SXK!nD=L;Ej-*crC?lMu
z0g{oINRfn@mm<}|uo_}CLzE~&jjS}18W|)tVwJL*a+@QWwldaB6%^&ClqQ$xm89kt
z<Rq4)>J=mwCFZ7<q!!)chswp50HxwHQ%WU)mINdg9bO2OFUl`gNJ-66aLhZrFekG(
zv$#|jO}4N!RiQXlAvZN86R0ImAr**<OHvgQVLGaWTsI4_WHYge0(BONO>Sgc&S<__
zliie2Mh+|jBG^EL1du2Og`R>!k@Vy$4sS-=$;UXH*kyr2ewyNw`8jPF#U}f5Mso4O
zoj-XYr@TAJ<Xfz6Ir)htMPM^oK>}RK`FX`9MTwbt#YHkefg%MUv67((WL%Lnh~R(`
zAoaycli9h7WJOgbxJ~z;=zoJl<pzi34SuZ~yf$ArSw-0zd_V9_uHc%^C_7nz+gZvG
pq!(_~FAkerU<l`=+7+2j&gGU7wPEyNnvga<XJXD*1`r#p6aW;RmYM(n

delta 429
zcmew&F;ATDG%qg~0}vQ}D9xD2F_CX0<CKX9c1^q{BFdA(yM|>o6G)PQA&QrYA(bUd
zW^y2-uq0y&UkZPUz#75TEKpf~u&e?|R#YfO7)hTXP(~<A4J0Ekk|GK-FGZ|{VKqcE
zLzD<YjjTA58VMvdqLq@GQkz?uwlZ$s&XUW-CIZx4Bsy7(eL17S=1c6Rj54xd5fH%!
zBE*42G061_3Pn<rl{vi`%_iq_I<d<Dh5R(dChy?1WfYzKjWcp`9G8?k$hcdqZaMjh
zB}HI^SV00@$@zK3B}IvudBsK2K!GB8AhD962xLZ)6o}w}5g_%&ER*+h6;1Z%p2jFM
z`5Cvfw?0S~uJ{*+O>TZlX-=wLkugw$5u`%_NPJ*sWMq8KpmLEx<t~Hv7Zy`S52gue
P({m=~d}RQ!!6pCz^!ZsV

diff --git a/lims_management/models/analysis_parameter.py b/lims_management/models/analysis_parameter.py
index e469789..26300d4 100644
--- a/lims_management/models/analysis_parameter.py
+++ b/lims_management/models/analysis_parameter.py
@@ -54,12 +54,12 @@ class LimsAnalysisParameter(models.Model):
         help='Si está desmarcado, el parámetro no estará disponible para nuevas configuraciones'
     )
     
-    # Relaciones - Se agregarán cuando se creen los modelos relacionados
-    # template_parameter_ids = fields.One2many(
-    #     'product.template.parameter',
-    #     'parameter_id',
-    #     string='Análisis que usan este parámetro'
-    # )
+    # Relaciones
+    template_parameter_ids = fields.One2many(
+        'product.template.parameter',
+        'parameter_id',
+        string='Análisis que usan este parámetro'
+    )
     
     # range_ids = fields.One2many(
     #     'lims.parameter.range',
@@ -67,6 +67,18 @@ class LimsAnalysisParameter(models.Model):
     #     string='Rangos de Referencia'
     # )
     
+    # Campos computados
+    analysis_count = fields.Integer(
+        string='Cantidad de Análisis',
+        compute='_compute_analysis_count',
+        store=True
+    )
+    
+    @api.depends('template_parameter_ids')
+    def _compute_analysis_count(self):
+        for record in self:
+            record.analysis_count = len(record.template_parameter_ids)
+    
     @api.constrains('code')
     def _check_code_unique(self):
         for record in self:
diff --git a/lims_management/models/product.py b/lims_management/models/product.py
index 0f23f1c..a60c044 100644
--- a/lims_management/models/product.py
+++ b/lims_management/models/product.py
@@ -27,6 +27,13 @@ class ProductTemplate(models.Model):
         'analysis_id',
         string="Rangos de Referencia"
     )
+    
+    parameter_ids = fields.One2many(
+        'product.template.parameter',
+        'product_tmpl_id',
+        string="Parámetros del Análisis",
+        help="Parámetros que se medirán en este análisis"
+    )
 
     is_sample_type = fields.Boolean(
         string="Es Tipo de Muestra",
diff --git a/lims_management/models/product_template_parameter.py b/lims_management/models/product_template_parameter.py
new file mode 100644
index 0000000..2646998
--- /dev/null
+++ b/lims_management/models/product_template_parameter.py
@@ -0,0 +1,109 @@
+# -*- coding: utf-8 -*-
+from odoo import models, fields, api
+from odoo.exceptions import ValidationError
+
+
+class ProductTemplateParameter(models.Model):
+    _name = 'product.template.parameter'
+    _description = 'Parámetros por Análisis'
+    _order = 'product_tmpl_id, sequence, id'
+    _rec_name = 'parameter_id'
+
+    product_tmpl_id = fields.Many2one(
+        'product.template',
+        string='Análisis',
+        required=True,
+        ondelete='cascade',
+        domain=[('is_analysis', '=', True)],
+        help='Análisis al que pertenece este parámetro'
+    )
+    
+    parameter_id = fields.Many2one(
+        'lims.analysis.parameter',
+        string='Parámetro',
+        required=True,
+        ondelete='restrict',
+        help='Parámetro de laboratorio'
+    )
+    
+    sequence = fields.Integer(
+        string='Secuencia',
+        default=10,
+        help='Orden en que aparecerá el parámetro en los resultados'
+    )
+    
+    required = fields.Boolean(
+        string='Obligatorio',
+        default=True,
+        help='Si está marcado, este parámetro debe tener un valor en los resultados'
+    )
+    
+    instructions = fields.Text(
+        string='Instrucciones específicas',
+        help='Instrucciones especiales para este parámetro en este análisis'
+    )
+    
+    # Campos relacionados para facilitar búsquedas y vistas
+    parameter_name = fields.Char(
+        related='parameter_id.name',
+        string='Nombre del Parámetro',
+        store=True,
+        readonly=True
+    )
+    
+    parameter_code = fields.Char(
+        related='parameter_id.code',
+        string='Código',
+        store=True,
+        readonly=True
+    )
+    
+    parameter_value_type = fields.Selection(
+        related='parameter_id.value_type',
+        string='Tipo de Valor',
+        store=True,
+        readonly=True
+    )
+    
+    parameter_unit = fields.Char(
+        related='parameter_id.unit',
+        string='Unidad',
+        readonly=True
+    )
+    
+    _sql_constraints = [
+        ('unique_param_per_analysis', 
+         'UNIQUE(product_tmpl_id, parameter_id)', 
+         'El parámetro ya está configurado para este análisis. Cada parámetro solo puede aparecer una vez por análisis.')
+    ]
+    
+    @api.constrains('sequence')
+    def _check_sequence(self):
+        for record in self:
+            if record.sequence < 0:
+                raise ValidationError('La secuencia debe ser un número positivo.')
+    
+    def name_get(self):
+        result = []
+        for record in self:
+            name = f"{record.product_tmpl_id.name} - [{record.parameter_code}] {record.parameter_name}"
+            if record.parameter_unit:
+                name += f" ({record.parameter_unit})"
+            result.append((record.id, name))
+        return result
+    
+    @api.model
+    def create(self, vals):
+        # Si no se especifica secuencia, asignar la siguiente disponible
+        if 'sequence' not in vals and 'product_tmpl_id' in vals:
+            max_sequence = self.search([
+                ('product_tmpl_id', '=', vals['product_tmpl_id'])
+            ], order='sequence desc', limit=1).sequence
+            vals['sequence'] = (max_sequence or 0) + 10
+        return super(ProductTemplateParameter, self).create(vals)
+    
+    def copy_data(self, default=None):
+        default = dict(default or {})
+        # Al duplicar, incrementar la secuencia
+        default['sequence'] = self.sequence + 10
+        return super(ProductTemplateParameter, self).copy_data(default)
\ No newline at end of file
diff --git a/lims_management/security/ir.model.access.csv b/lims_management/security/ir.model.access.csv
index 41f33a9..cbd70a9 100644
--- a/lims_management/security/ir.model.access.csv
+++ b/lims_management/security/ir.model.access.csv
@@ -1,6 +1,8 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 access_lims_analysis_parameter_user,lims.analysis.parameter.user,model_lims_analysis_parameter,base.group_user,1,0,0,0
 access_lims_analysis_parameter_manager,lims.analysis.parameter.manager,model_lims_analysis_parameter,group_lims_admin,1,1,1,1
+access_product_template_parameter_user,product.template.parameter.user,model_product_template_parameter,base.group_user,1,0,0,0
+access_product_template_parameter_manager,product.template.parameter.manager,model_product_template_parameter,group_lims_admin,1,1,1,1
 access_lims_analysis_range_user,lims.analysis.range.user,model_lims_analysis_range,base.group_user,1,1,1,1
 access_sale_order_receptionist,sale.order.receptionist,sale.model_sale_order,group_lims_receptionist,1,1,1,0
 access_stock_lot_user,stock.lot.user,stock.model_stock_lot,base.group_user,1,1,1,1
diff --git a/lims_management/views/analysis_parameter_views.xml b/lims_management/views/analysis_parameter_views.xml
index 3eb8f09..defa737 100644
--- a/lims_management/views/analysis_parameter_views.xml
+++ b/lims_management/views/analysis_parameter_views.xml
@@ -8,6 +8,13 @@
             <form string="Parámetro de Análisis">
                 <sheet>
                     <div class="oe_button_box" name="button_box">
+                        <button name="%(lims_management.action_product_template_parameter)d" 
+                                type="action" 
+                                class="oe_stat_button" 
+                                icon="fa-flask"
+                                context="{'search_default_parameter_id': id}">
+                            <field name="analysis_count" widget="statinfo" string="Análisis"/>
+                        </button>
                         <button name="toggle_active" 
                                 type="object" 
                                 class="oe_stat_button" 
@@ -38,7 +45,17 @@
                             <field name="description" widget="text" nolabel="1" colspan="2"/>
                         </group>
                     </group>
-                    <!-- Notebook se agregará cuando se creen los modelos relacionados -->
+                    <notebook>
+                        <page string="Análisis Configurados" name="analysis">
+                            <field name="template_parameter_ids">
+                                <list>
+                                    <field name="product_tmpl_id"/>
+                                    <field name="sequence"/>
+                                    <field name="required"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>
@@ -54,6 +71,7 @@
                 <field name="name"/>
                 <field name="value_type"/>
                 <field name="unit" optional="show"/>
+                <field name="analysis_count" optional="show"/>
                 <field name="active" invisible="1"/>
             </list>
         </field>
diff --git a/lims_management/views/analysis_views.xml b/lims_management/views/analysis_views.xml
index 4c51171..d1a6e26 100644
--- a/lims_management/views/analysis_views.xml
+++ b/lims_management/views/analysis_views.xml
@@ -39,6 +39,21 @@
                         <separator string="Rangos de Referencia"/>
                         <field name="value_range_ids"
                                view_id="lims_management.view_lims_analysis_range_tree"/>
+                        <separator string="Parámetros del Análisis"/>
+                        <field name="parameter_ids"
+                               context="{'default_product_tmpl_id': id}">
+                            <list editable="bottom">
+                                <field name="sequence" widget="handle"/>
+                                <field name="parameter_id" 
+                                       options="{'no_create': True}"
+                                       domain="[('active', '=', True)]"/>
+                                <field name="parameter_code"/>
+                                <field name="parameter_value_type"/>
+                                <field name="parameter_unit"/>
+                                <field name="required"/>
+                                <field name="instructions" optional="show"/>
+                            </list>
+                        </field>
                     </page>
                 </xpath>
                 <!-- Añade el campo is_analysis cerca del nombre del producto para fácil acceso -->
diff --git a/lims_management/views/product_template_parameter_views.xml b/lims_management/views/product_template_parameter_views.xml
new file mode 100644
index 0000000..cc84322
--- /dev/null
+++ b/lims_management/views/product_template_parameter_views.xml
@@ -0,0 +1,93 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <!-- Form View -->
+    <record id="view_product_template_parameter_form" model="ir.ui.view">
+        <field name="name">product.template.parameter.form</field>
+        <field name="model">product.template.parameter</field>
+        <field name="arch" type="xml">
+            <form string="Parámetro del Análisis">
+                <sheet>
+                    <group>
+                        <group string="Información General">
+                            <field name="product_tmpl_id" readonly="1"/>
+                            <field name="parameter_id" 
+                                   options="{'no_create': True}"
+                                   context="{'form_view_ref': 'lims_management.view_lims_analysis_parameter_form'}"/>
+                            <field name="sequence"/>
+                            <field name="required"/>
+                        </group>
+                        <group string="Detalles del Parámetro">
+                            <field name="parameter_code"/>
+                            <field name="parameter_value_type"/>
+                            <field name="parameter_unit"/>
+                        </group>
+                    </group>
+                    <group string="Instrucciones Específicas">
+                        <field name="instructions" nolabel="1" placeholder="Ingrese instrucciones especiales para este parámetro en este análisis..."/>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <!-- List View -->
+    <record id="view_product_template_parameter_list" model="ir.ui.view">
+        <field name="name">product.template.parameter.list</field>
+        <field name="model">product.template.parameter</field>
+        <field name="arch" type="xml">
+            <list string="Parámetros por Análisis" editable="bottom">
+                <field name="sequence" widget="handle"/>
+                <field name="parameter_id" 
+                       options="{'no_create': True}"
+                       domain="[('active', '=', True)]"/>
+                <field name="parameter_code"/>
+                <field name="parameter_value_type"/>
+                <field name="parameter_unit"/>
+                <field name="required"/>
+                <field name="instructions" optional="show"/>
+            </list>
+        </field>
+    </record>
+
+    <!-- Search View -->
+    <record id="view_product_template_parameter_search" model="ir.ui.view">
+        <field name="name">product.template.parameter.search</field>
+        <field name="model">product.template.parameter</field>
+        <field name="arch" type="xml">
+            <search string="Buscar Parámetros">
+                <field name="product_tmpl_id"/>
+                <field name="parameter_id"/>
+                <field name="parameter_name"/>
+                <field name="parameter_code"/>
+                <filter string="Obligatorios" name="required" domain="[('required', '=', True)]"/>
+                <filter string="Opcionales" name="optional" domain="[('required', '=', False)]"/>
+                <separator/>
+                <filter string="Numéricos" name="numeric" domain="[('parameter_value_type', '=', 'numeric')]"/>
+                <filter string="Texto" name="text" domain="[('parameter_value_type', '=', 'text')]"/>
+                <group expand="0" string="Agrupar por">
+                    <filter string="Análisis" name="group_product" context="{'group_by': 'product_tmpl_id'}"/>
+                    <filter string="Parámetro" name="group_parameter" context="{'group_by': 'parameter_id'}"/>
+                    <filter string="Tipo de Valor" name="group_value_type" context="{'group_by': 'parameter_value_type'}"/>
+                    <filter string="Obligatorio" name="group_required" context="{'group_by': 'required'}"/>
+                </group>
+            </search>
+        </field>
+    </record>
+
+    <!-- Action -->
+    <record id="action_product_template_parameter" model="ir.actions.act_window">
+        <field name="name">Parámetros por Análisis</field>
+        <field name="res_model">product.template.parameter</field>
+        <field name="view_mode">list,form</field>
+        <field name="search_view_id" ref="view_product_template_parameter_search"/>
+        <field name="help" type="html">
+            <p class="o_view_nocontent_smiling_face">
+                Configurar parámetros para análisis
+            </p>
+            <p>
+                Aquí puede ver y configurar qué parámetros se miden en cada análisis,
+                su orden de aparición y si son obligatorios u opcionales.
+            </p>
+        </field>
+    </record>
+</odoo>
\ No newline at end of file