[DEL]partner_skills:disabled skill level for now

partner_skills/models/partner_skill_line.py

@@ -1,5 +1,5 @@
 from odoo import api, fields, models, _
-from odoo.exceptions import ValidationError
+from odoo.exceptions import UserError, ValidationError
 
 
 class PartnerSkillLine(models.Model):
@@ -13,10 +13,15 @@ class PartnerSkillLine(models.Model):
         'partner.skill', compute='_compute_skill_id', store=True,
         domain="[('skill_type_id', '=', skill_type_id)]",
         readonly=False, required=True, ondelete='cascade')
+    # Skill levels are hidden from the UI for now (client request). The field is
+    # kept so existing data is preserved and the feature can be re-enabled later.
     skill_level_id = fields.Many2one(
-        'partner.skill.level', compute='_compute_skill_level_id',
+        'partner.skill.level',
+        # compute='_compute_skill_level_id',  # re-enable with the method below
         domain="[('skill_type_id', '=', skill_type_id)]",
-        store=True, readonly=False, required=True, ondelete='cascade')
+        store=True, readonly=False,
+        # required=True,  # re-enable when skill levels come back
+        ondelete='cascade')
     skill_type_id = fields.Many2one(
         'partner.skill.type',
         default=lambda self: self.env['partner.skill.type'].search([], limit=1),
@@ -39,15 +44,17 @@ class PartnerSkillLine(models.Model):
                     type=record.skill_type_id.name,
                 ))
 
-    @api.constrains('skill_type_id', 'skill_level_id')
-    def _check_skill_level(self):
-        for record in self:
-            if record.skill_level_id not in record.skill_type_id.skill_level_ids:
-                raise ValidationError(_(
-                    "The skill level %(level)s is not valid for skill type: %(type)s",
-                    level=record.skill_level_id.name,
-                    type=record.skill_type_id.name,
-                ))
+    # Temporarily disabled: references skill_type_id.skill_level_ids which is
+    # commented out on partner.skill.type while levels are hidden.
+    # @api.constrains('skill_type_id', 'skill_level_id')
+    # def _check_skill_level(self):
+    #     for record in self:
+    #         if record.skill_level_id not in record.skill_type_id.skill_level_ids:
+    #             raise ValidationError(_(
+    #                 "The skill level %(level)s is not valid for skill type: %(type)s",
+    #                 level=record.skill_level_id.name,
+    #                 type=record.skill_type_id.name,
+    #             ))
 
     @api.depends('skill_type_id')
     def _compute_skill_id(self):
@@ -57,16 +64,42 @@ class PartnerSkillLine(models.Model):
             else:
                 record.skill_id = False
 
-    @api.depends('skill_id')
-    def _compute_skill_level_id(self):
-        for record in self:
-            if not record.skill_id:
-                record.skill_level_id = False
-            else:
-                skill_levels = record.skill_type_id.skill_level_ids
-                record.skill_level_id = skill_levels.filtered('default_level') or skill_levels[0] if skill_levels else False
+    # Temporarily disabled: references skill_type_id.skill_level_ids which is
+    # commented out on partner.skill.type while levels are hidden.
+    # @api.depends('skill_id')
+    # def _compute_skill_level_id(self):
+    #     for record in self:
+    #         if not record.skill_id:
+    #             record.skill_level_id = False
+    #         else:
+    #             skill_levels = record.skill_type_id.skill_level_ids
+    #             record.skill_level_id = skill_levels.filtered('default_level') or skill_levels[0] if skill_levels else False
+
+    def action_add_all_skills_of_type(self):
+        self.ensure_one()
+        if not self.skill_type_id:
+            raise UserError(_("Please select a skill type first."))
+        if not isinstance(self.partner_id.id, int):
+            raise UserError(_("Please save the contact before adding skills in bulk."))
+        existing_skills = self.partner_id.partner_skill_ids.filtered(
+            lambda l: l.skill_type_id == self.skill_type_id
+        ).skill_id
+        skills_to_add = self.skill_type_id.skill_ids - existing_skills
+        if skills_to_add:
+            self.create([{
+                'partner_id': self.partner_id.id,
+                'skill_type_id': self.skill_type_id.id,
+                'skill_id': skill.id,
+            } for skill in skills_to_add])
+        return {
+            'type': 'ir.actions.client',
+            'tag': 'soft_reload',
+        }
 
     @api.depends('skill_id', 'skill_level_id')
     def _compute_display_name(self):
         for record in self:
-            record.display_name = f"{record.skill_id.name}: {record.skill_level_id.name}"
+            if record.skill_level_id:
+                record.display_name = f"{record.skill_id.name}: {record.skill_level_id.name}"
+            else:
+                record.display_name = record.skill_id.name or ""

partner_skills/models/partner_skill_type.py

@@ -13,7 +13,7 @@ class PartnerSkillType(models.Model):
     active = fields.Boolean('Active', default=True)
     name = fields.Char(required=True, translate=True)
     skill_ids = fields.One2many('partner.skill', 'skill_type_id', string="Skills")
-    skill_level_ids = fields.One2many('partner.skill.level', 'skill_type_id', string="Levels", copy=True)
+    #skill_level_ids = fields.One2many('partner.skill.level', 'skill_type_id', string="Levels", copy=True)
     color = fields.Integer('Color', default=_get_default_color)
 
     def copy_data(self, default=None):

partner_skills/models/res_partner.py

@@ -13,4 +13,4 @@ class ResPartner(models.Model):
     @api.depends('partner_skill_ids.skill_id')
     def _compute_skill_ids(self):
         for partner in self:
-            partner.skill_ids = partner.partner_skill_ids.skill_id
+            partner.skill_ids = partner.partner_skill_ids.skill_id or None