[15.0][ADD] heldespk_mgmt: Helpdesk stages per teams

TT45710

Author: carolinafernandez-tecnativa

Diff for: helpdesk_mgmt/controllers/myaccount.py

@@ -65,7 +65,8 @@ def portal_my_tickets(
         searchbar_filters = {
             "all": {"label": _("All"), "domain": []},
         }
-        for stage in request.env["helpdesk.ticket.stage"].search([]):
+        stages = self._get_applicable_stages(HelpdeskTicket.team_id, False)
+        for stage in stages:
             searchbar_filters[str(stage.id)] = {
                 "label": stage.name,
                 "domain": [("stage_id", "=", stage.id)],
@@ -182,9 +183,7 @@ def portal_my_ticket(self, ticket_id, access_token=None, **kw):
         return request.render("helpdesk_mgmt.portal_helpdesk_ticket_page", values)
 
     def _ticket_get_page_view_values(self, ticket, access_token, **kwargs):
-        closed_stages = request.env["helpdesk.ticket.stage"].search(
-            [("close_from_portal", "=", True)]
-        )
+        closed_stages = self._get_applicable_stages(ticket.team_id, True)
         values = {
             "closed_stages": closed_stages,  # used to display close buttons
             "page_name": "ticket",
@@ -259,3 +258,13 @@ def _ticket_get_order(self, order, groupby):
         if not field_name:
             return order
         return "%s, %s" % (field_name, order)
+
+    def _get_applicable_stages(self, team, close_from_portal=False):
+        domain = [
+            ("team_ids", "in", [False] + team.ids),
+        ]
+        if close_from_portal:
+            domain = ["|"] + domain
+            domain.append(("close_from_portal", "=", True))
+
+        return request.env["helpdesk.ticket.stage"].search(domain)

Diff for: helpdesk_mgmt/models/helpdesk_ticket.py

@@ -11,12 +11,48 @@ class HelpdeskTicket(models.Model):
     _inherit = ["mail.thread.cc", "mail.activity.mixin", "portal.mixin"]
 
     def _get_default_stage_id(self):
-        return self.env["helpdesk.ticket.stage"].search([], limit=1).id
+        if self.team_id:
+            return self.team_id._determine_stage()[self.team_id.id].id
+        else:
+            return (
+                self.env["helpdesk.ticket.stage"]
+                .search([("team_ids", "=", False)], limit=1)
+                .id
+            )
+
+    @api.depends("team_id")
+    def _compute_stage_ids(self):
+        for ticket in self:
+            if ticket.team_id:
+                ticket.stage_id = ticket.team_id._determine_stage()[ticket.team_id.id]
+            else:
+                ticket.stage_id = (
+                    self.env["helpdesk.ticket.stage"]
+                    .search([("team_ids", "=", False)], limit=1)
+                    .id
+                )
+
+    def _default_team_id(self):
+        team_id = (
+            self.env["helpdesk.ticket.team"]
+            .search([("user_ids", "in", self.env.uid)], limit=1)
+            .id
+        )
+        if not team_id:
+            team_id = self.env["helpdesk.ticket.team"].search([], limit=1).id
+        return team_id
 
     @api.model
     def _read_group_stage_ids(self, stages, domain, order):
-        stage_ids = self.env["helpdesk.ticket.stage"].search([])
-        return stage_ids
+        search_domain = [("id", "in", stages.ids)]
+        if self.env.context.get("default_team_id"):
+            search_domain = [
+                "|",
+                "|",
+                ("team_ids", "=", False),
+                ("team_ids", "in", self.env.context["default_team_id"]),
+            ] + search_domain
+        return stages.search(search_domain, order=order)
 
     number = fields.Char(string="Ticket number", default="/", readonly=True)
     name = fields.Char(string="Title", required=True)
@@ -31,16 +67,22 @@ def _read_group_stage_ids(self, stages, domain, order):
     user_ids = fields.Many2many(
         comodel_name="res.users", related="team_id.user_ids", string="Users"
     )
+
     stage_id = fields.Many2one(
         comodel_name="helpdesk.ticket.stage",
         string="Stage",
-        group_expand="_read_group_stage_ids",
+        compute="_compute_stage_ids",
         default=_get_default_stage_id,
-        tracking=True,
+        store=True,
+        readonly=False,
         ondelete="restrict",
-        index=True,
+        tracking=True,
+        group_expand="_read_group_stage_ids",
         copy=False,
+        index=True,
+        domain="['|',('team_ids', '=', team_id),('team_ids','=',False)]",
     )
+
     partner_id = fields.Many2one(comodel_name="res.partner", string="Contact")
     partner_name = fields.Char()
     partner_email = fields.Char(string="Email")
@@ -70,6 +112,8 @@ def _read_group_stage_ids(self, stages, domain, order):
     team_id = fields.Many2one(
         comodel_name="helpdesk.ticket.team",
         string="Team",
+        default=_default_team_id,
+        index=True,
     )
     priority = fields.Selection(
         selection=[
@@ -130,6 +174,9 @@ def create(self, vals_list):
                 vals["number"] = self._prepare_ticket_number(vals)
             if vals.get("user_id") and not vals.get("assigned_date"):
                 vals["assigned_date"] = fields.Datetime.now()
+            if vals.get("team_id"):
+                team = self.env["helpdesk.ticket.team"].browse(vals["team_id"])
+                vals["stage_id"] = team._determine_stage()[team.id].id
         return super().create(vals_list)
 
     def copy(self, default=None):

Diff for: helpdesk_mgmt/models/helpdesk_ticket_stage.py

@@ -35,6 +35,12 @@ class HelpdeskTicketStage(models.Model):
         string="Company",
         default=lambda self: self.env.company,
     )
+    team_ids = fields.Many2many(
+        comodel_name="helpdesk.ticket.team",
+        relation="team_stage_rel",
+        string="Helpdesk Teams",
+        help="Specific team that uses this stage. If it is empty all teams could uses",
+    )
 
     @api.onchange("closed")
     def _onchange_closed(self):

Diff for: helpdesk_mgmt/models/helpdesk_ticket_team.py

@@ -67,6 +67,20 @@ class HelpdeskTeam(models.Model):
         help="Allow to select this team when creating a new ticket in the portal.",
     )
 
+    def _determine_stage(self):
+        """Get a dict with the stage per team that should be set as first to a created ticket
+        :returns a mapping of team identifier with the stage.
+        :rtype : dict (key=team_id, value=record of helpdesk.ticket.stage)
+        """
+        result = dict.fromkeys(self.ids, self.env["helpdesk.ticket.stage"])
+        for team in self:
+            result[team.id] = self.env["helpdesk.ticket.stage"].search(
+                ["|", ("team_ids", "=", False), ("team_ids", "in", team.id)],
+                order="sequence",
+                limit=1,
+            )
+        return result
+
     @api.depends("ticket_ids", "ticket_ids.stage_id")
     def _compute_todo_tickets(self):
         ticket_model = self.env["helpdesk.ticket"]

Diff for: helpdesk_mgmt/readme/CONTRIBUTORS.rst

@@ -27,6 +27,7 @@
 
   * Pedro M. Baeza
   * Víctor Martínez
+  * Carolina Fernandez
 
 * `ID42 Sistemas <https://www.id42.com.br>`_:
 

Diff for: helpdesk_mgmt/static/description/index.html

@@ -568,6 +568,7 @@ <h2><a class="toc-backref" href="#toc-entry-13">Contributors</a></h2>
 <li><a class="reference external" href="https://www.tecnativa.com">Tecnativa</a>:<ul>
 <li>Pedro M. Baeza</li>
 <li>Víctor Martínez</li>
+<li>Carolina Fernandez</li>
 </ul>
 </li>
 <li><a class="reference external" href="https://www.id42.com.br">ID42 Sistemas</a>:<ul>

Diff for: helpdesk_mgmt/tests/common.py

@@ -39,6 +39,7 @@ def setUpClass(cls):
         cls.team_b = helpdesk_ticket_team.create(
             {"name": "Team B", "user_ids": [(6, 0, [cls.user_team.id])]}
         )
+        cls.new_stage = cls.env.ref("helpdesk_mgmt.helpdesk_ticket_stage_new")
         cls.ticket_a_unassigned = cls._create_ticket(cls, cls.team_a)
         cls.ticket_a_unassigned.priority = "3"
         cls.ticket_a_user_own = cls._create_ticket(cls, cls.team_a, cls.user_own)

Diff for: helpdesk_mgmt/tests/test_helpdesk_ticket.py

@@ -97,3 +97,45 @@ def test_helpdesk_ticket_message_new(self):
         self.assertEqual(
             t.name, title, "The ticket should have the correct (new) title."
         )
+
+    def test_ticket_with_team_stage(self):
+        self.new_stage.team_ids = [(6, 0, [self.team_a.id, self.team_b.id])]
+        in_progress_stage = self.env.ref(
+            "helpdesk_mgmt.helpdesk_ticket_stage_in_progress"
+        )
+        in_progress_stage.team_ids = [(6, 0, [self.team_b.id])]
+        new_ticket = self.env["helpdesk.ticket"].create(
+            {
+                "name": "New Ticket",
+                "description": "Description",
+                "team_id": self.team_a.id,
+                "user_id": self.user.id,
+                "priority": "1",
+            }
+        )
+        self.assertEqual(new_ticket.stage_id, self.new_stage)
+        self.new_stage.team_ids = [(6, 0, [self.team_a.id])]
+        new_ticket.team_id = self.team_b
+        self.assertEqual(new_ticket.stage_id, in_progress_stage)
+        self.new_stage.team_ids = False
+        new_ticket.team_id = False
+        self.assertEqual(new_ticket.stage_id, self.new_stage)
+
+    def test_ticket_without_team_stage(self):
+        self.new_stage.team_ids = False
+        self.assertEqual(
+            self.ticket.stage_id, self.team_a._determine_stage()[self.team_a.id]
+        )
+        self.assertEqual(self.ticket.stage_id, self.new_stage)
+
+    def test_ticket_without_team(self):
+        new_ticket = self.env["helpdesk.ticket"].create(
+            {
+                "name": "New Ticket",
+                "description": "Description",
+                "team_id": False,
+                "user_id": self.user.id,
+                "priority": "1",
+            }
+        )
+        self.assertEqual(self.new_stage, new_ticket.stage_id)

Diff for: helpdesk_mgmt/views/helpdesk_ticket_stage_views.xml

@@ -12,6 +12,7 @@
                 />
                 <separator />
                 <field name="name" filter_domain="[('name', 'ilike', self)]" />
+                <field name="team_ids" />
                 <field name="company_id" groups="base.group_multi_company" />
                 <group>
                     <filter
@@ -49,6 +50,7 @@
                         <group name="main_left">
                             <field name="sequence" />
                             <field name="mail_template_id" />
+                            <field name="team_ids" widget="many2many_tags" />
                             <field
                                 name="company_id"
                                 groups="base.group_multi_company"
@@ -77,6 +79,7 @@
                 <field name="sequence" widget="handle" />
                 <field name="name" />
                 <field name="mail_template_id" />
+                <field name="team_ids" widget="many2many_tags" />
                 <field
                     name="company_id"
                     optional="hide"