[IMP] academic: when you choose partner that automatically shows you the children

Author: lef-adhoc

academic/__manifest__.py

@@ -19,7 +19,7 @@
 ##############################################################################
 {
     'name': 'Academic',
-    'version': "16.0.1.7.0",
+    'version': "16.0.1.8.0",
     'sequence': 14,
     'summary': '',
     'author': 'ADHOC SA',
@@ -59,6 +59,7 @@
         'data/res_users_data.xml',
         'wizards/portal_wizard_views.xml',
         'data/res_partner_role_data.xml',
+        'views/account_move_views.xml',
     ],
     'demo': [
         'demo/res_partner_relationship_demo.xml',

academic/models/__init__.py

@@ -17,3 +17,4 @@
 from . import res_partner_role
 from . import res_partner_link
 from . import sale_order
+from . import account_move

academic/models/account_move.py

@@ -1,20 +1,19 @@
-from odoo import _, models, fields, api
-from odoo.exceptions import ValidationError
-import logging
-
-_logger = logging.getLogger(__name__)
+from odoo import models, fields, api
 
 
 class AccountMove(models.Model):
     _inherit = 'account.move'
 
-    student_id = fields.Many2one('res.partner', string='Alumno', domain="[('parent_id', '=', partner_id), ('partner_type', '=', 'student')]")
+    student_ids = fields.Many2many('res.partner', compute="_compute_student_ids")
+    partner_shipping_id = fields.Many2one(domain="[('id', 'in', student_ids), ('partner_type', '=', 'student')]")
 
-    @api.constrains('student_id')
-    def _check_student(self):
-        invoices_wo_student = self.filtered(lambda x: x.type in ["out_invoice", "out_refund"] and not x.student_id)
-        if invoices_wo_student:
-            msg = _("Las facturas de clientes y notas de debito debe tener asociado siempre un alumno")
-            if len(invoices_wo_student) > 1:
-                msg += ".\n" + _("Los siguientes documentos no cumplen esa condición:") + "\n\n - %s" % '\n - '.join(invoices_wo_student.mapped('display_name'))
-            raise ValidationError(msg)
+    @api.depends('partner_id')
+    def _compute_student_ids(self):
+        for rec in self:
+            if rec.partner_id:
+                student_ids = self.env['res.partner.link'].search(
+                    [('partner_id', '=', rec.partner_id.id), ('role_ids', 'in', self.env.ref('academic.paying_role').id)]
+                ).mapped('student_id.id')
+                rec.student_ids = [(6, 0, student_ids)]
+            else:
+                rec.student_ids = [(5, 0, 0)]

academic/views/account_move_views.xml

@@ -0,0 +1,12 @@
+<odoo>
+    <record id="academic_view_move_form" model="ir.ui.view">
+        <field name="name">academic.account.move.form</field>
+        <field name="model">account.move</field>
+        <field name="inherit_id" ref="account.view_move_form"/>
+        <field name="arch" type="xml">
+            <field name="partner_id" position="after">
+                <field name="student_ids" invisible="1"/>
+            </field>
+        </field>
+    </record>
+</odoo>