[14.0][FIX] pms_helpdesk_mgmt: to list active properties

pms_helpdesk_mgmt/controllers/myaccount.py

@@ -1,89 +1,124 @@
-from odoo.http import request
-from odoo.addons.helpdesk_mgmt.controllers.myaccount import CustomerPortalHelpdesk
-from odoo.osv.expression import AND, OR
-from odoo.addons.portal.controllers.portal import CustomerPortal, pager as portal_pager
-from collections import OrderedDict
 from operator import itemgetter
-from odoo import _, http
-from odoo.exceptions import AccessError, MissingError
+
+from odoo import http
+from odoo.http import request
 from odoo.tools import groupby as groupbyelem
 
+from odoo.addons.helpdesk_mgmt.controllers.myaccount import CustomerPortalHelpdesk
+from odoo.addons.portal.controllers.portal import pager as portal_pager
+
 
 class CustomCustomerPortalHelpdesk(CustomerPortalHelpdesk):
+    def _prepare_home_portal_values(self, counters):
+        values = super(CustomCustomerPortalHelpdesk, self)._prepare_home_portal_values(
+            counters
+        )
+
+        if "ticket_count" in counters:
+            helpdesk_model = request.env["helpdesk.ticket"].sudo()
+            active_property_ids = request.env.user.get_active_property_ids()
+            domain = []
+            if active_property_ids:
+                domain.append(("pms_property_id", "in", active_property_ids))
+
+            ticket_count = (
+                helpdesk_model.search_count(domain)
+                if helpdesk_model.check_access_rights("read", raise_exception=False)
+                else 0
+            )
+            values["ticket_count"] = ticket_count
+
+        return values
+
     @http.route(
         ["/my/tickets", "/my/tickets/page/<int:page>"],
         type="http",
         auth="user",
         website=True,
     )
-    def portal_my_tickets(self, **kw):
-        response = super(CustomCustomerPortalHelpdesk, self).portal_my_tickets(**kw)
-        page = response.qcontext.get('page', 1)
-        HelpdeskTicket = request.env["helpdesk.ticket"].sudo() 
-
-        # Llamamos al método del padre para obtener los valores de la vista
-        values = super(CustomCustomerPortalHelpdesk, self)._prepare_portal_layout_values()
-        if not HelpdeskTicket.check_access_rights("read", raise_exception=False):
-            return request.redirect("/my")
+    def portal_my_tickets(self, page=1, **kw):
+        response = super(CustomCustomerPortalHelpdesk, self).portal_my_tickets(
+            page=page, **kw
+        )
+        domain = response.qcontext.get("domain", [])
+        response.qcontext.get("sortby")
+        response.qcontext.get("groupby")
+        order = response.qcontext.get("order")
 
-        domain = response.qcontext.get('domain', [])
-        order = response.qcontext.get('order', '')
-        sortby = response.qcontext.get('sortby', {})
-        groupby = response.qcontext.get('groupby', '')
-
-        active_property_ids = request.env.user.get_active_property_ids() 
+        active_property_ids = request.env.user.get_active_property_ids()
         if active_property_ids:
             domain.append(("pms_property_id", "in", active_property_ids))
-        
-        HelpdeskTicket = request.env["helpdesk.ticket"].sudo() 
+
+        HelpdeskTicket = request.env["helpdesk.ticket"].sudo()
         ticket_count = HelpdeskTicket.search_count(domain)
+
         pager = portal_pager(
             url="/my/tickets",
-            url_args={
-                "date_begin": response.qcontext.get('date_begin', {}),
-                "date_end": response.qcontext.get('date_end', {}),
-                "sortby": sortby,
-                "filterby": response.qcontext.get('filterby', ''),
-                "groupby": groupby,
-                "search": response.qcontext.get('search', ''),
-                "search_in": response.qcontext.get('search_in', ''),
-            },
+            url_args={},
             total=ticket_count,
             page=page,
             step=self._items_per_page,
         )
+
         tickets = HelpdeskTicket.search(
-            domain,
-            order=order,
-            limit=self._items_per_page,
-            offset=pager["offset"],
+            domain, order=order, limit=self._items_per_page, offset=pager["offset"]
         )
         request.session["my_tickets_history"] = tickets.ids[:100]
 
-        group = response.qcontext.get('group', None)
-        grouped_tickets = [tickets] if not group else [
-            request.env["helpdesk.ticket"].concat(*g)
-            for k, g in groupbyelem(tickets, itemgetter(group))
-        ]
-        values.update(
+        group = response.qcontext.get("group", None)
+        if group:
+            grouped_tickets = [
+                HelpdeskTicket.concat(*g)
+                for k, g in groupbyelem(tickets, itemgetter(group))
+            ]
+        else:
+            grouped_tickets = [tickets] if tickets else []
+
+        response.qcontext.update(
             {
-                "date": response.qcontext.get('date_begin', {}),
-                "date_end": response.qcontext.get('date_end', {}),
                 "grouped_tickets": grouped_tickets,
-                "page_name": "ticket",
-                "default_url": "/my/tickets",
                 "pager": pager,
-                "searchbar_sortings": response.qcontext.get('searchbar_sortings', []),
-                "searchbar_groupby": response.qcontext.get('searchbar_groupby', []),
-                "searchbar_inputs": response.qcontext.get('searchbar_inputs', []),
-                "search_in": response.qcontext.get('search_in', ''),
-                "search": response.qcontext.get('search', ''),
-                "sortby": sortby,
-                "groupby": groupby,
-                "searchbar_filters": OrderedDict(sorted(response.qcontext.get('searchbar_filters', {}).items())),
-                "filterby": response.qcontext.get('filterby', ''),
+                "ticket_count": ticket_count,
             }
         )
-        return request.render("helpdesk_mgmt.portal_my_tickets", values)
+        return request.render(
+            "pms_helpdesk_mgmt.portal_my_tickets_inherited", response.qcontext
+        )
+
+    @http.route(
+        ["/my/ticket/<int:ticket_id>"], type="http", auth="public", website=True
+    )
+    def portal_my_ticket(self, ticket_id, access_token=None, **kw):
+        HelpdeskTicket = request.env["helpdesk.ticket"].sudo()
+
+        if access_token:
+            ticket_sudo = HelpdeskTicket.search(
+                [("id", "=", ticket_id), ("access_token", "=", access_token)]
+            )
+        else:
+            ticket_sudo = HelpdeskTicket.search([("id", "=", ticket_id)])
+
+        if not ticket_sudo:
+            return request.redirect("/my")
+
+        for attachment in ticket_sudo.attachment_ids:
+            attachment.generate_access_token()
+
+        values = self._ticket_get_page_view_values(ticket_sudo, access_token, **kw)
+        return request.render("pms_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"].sudo().search([("closed", "=", True)])
+        )
 
+        values = {
+            "closed_stages": closed_stages,
+            "page_name": "ticket",
+            "ticket": ticket,
+            "user": request.env.user,
+        }
 
+        return self._get_page_view_values(
+            ticket, access_token, values, "my_tickets_history", False, **kwargs
+        )

pms_helpdesk_mgmt/views/helpdesk_ticket_templates.xml

@@ -8,9 +8,37 @@
             />
         </xpath>
     </template>
-
+
+    <template
+        id="portal_my_home_inherited_form"
+        name="Portal My Home : Replace New Ticket Form"
+        inherit_id="helpdesk_mgmt.portal_my_home"
+        priority="40"
+    >
+        <xpath expr="//t[@t-call='portal.portal_docs_entry']" position="attributes">
+            <t t-call="portal.portal_docs_entry">
+                <t t-set="title">Tickets</t>
+                <t t-set="url" t-value="'/my/tickets'" />
+                <t t-set="placeholder_count" t-value="ticket_count" />
+            </t>
+        </xpath>
+    </template>
+
     <template
-        id="portal_my_tickets"
+        id="portal_my_tickets_inherited"
+        inherit_id="helpdesk_mgmt.portal_my_tickets"
+    >
+        <xpath expr="//t[@t-if='not grouped_tickets']" position="attributes">
+            <t t-if="not grouped_tickets">
+                <div class="alert alert-warning mt8" role="alert">
+                    <p>No tickets found for your properties.</p>
+                </div>
+            </t>
+        </xpath>
+    </template>
+
+    <template
+        id="portal_ticket_list"
         inherit_id="helpdesk_mgmt.portal_ticket_list"
         name="Tickets List"
     >
@@ -20,32 +48,28 @@
         </xpath>
         <xpath expr="//tr/td[position()=3]" position="after">
             <td>
-                <span t-field="ticket.sudo().pms_property_id.name" />
+                <span t-field="ticket.pms_property_id.name" />
             </td>
             <td>
-                <span t-field="ticket.sudo().room_id.name" />
+                <span t-field="ticket.room_id.name" />
             </td>
         </xpath>
-            <xpath expr="//a[contains(@t-attf-href, '/my/ticket')]" position="replace">
-            <a t-attf-href="/my/ticket/#{ticket.id}?{{ keep_query() }}">
-                <span t-field="ticket.name" />
-            </a>
-        </xpath>
     </template>
+
     <template
         id="portal_helpdesk_ticket_page"
         inherit_id="helpdesk_mgmt.portal_helpdesk_ticket_page"
         name="Ticket Portal Template"
     >
         <xpath expr="//span[@t-field='ticket.create_date']" position="after">
             <div><strong>Property:</strong>
-            <span t-field="ticket.pms_property_id.name" /></div>
+            <span t-field="ticket.sudo().pms_property_id.name" /></div>
             <div><strong>Room:</strong>
-            <span t-field="ticket.room_id.name" /></div>
+            <span t-field="ticket.sudo().room_id.name" /></div>
         </xpath>
         <xpath expr="//span[@t-field='ticket.category_id']" position="after">
             <div><strong>Tag:</strong>
-            <span t-field="ticket.tag_ids" /></div>
+            <span t-field="ticket.sudo().tag_ids" /></div>
         </xpath>
     </template>
 
@@ -60,20 +84,19 @@
         >
             <div class="col-md-7 col-sm-8">
                 <label for="company_id">Company:</label>
-            <select
-                class="form-control"
-                id="company_id"
-                name="company_id"
-                required="required"
-                t-attf-domain="[('id', 'in', companies_active)]"
-            >
-                <option value="" disabled="disabled">Select a company</option>
-                <t t-foreach="companies" t-as="company">
-                    <option t-att-value="company.id">
-                        <t t-esc="company.name" />
-                    </option>
-                </t>
-            </select>
+                <select
+                    class="form-control"
+                    id="company_id"
+                    name="company_id"
+                    required="required"
+                >
+                    <option value="" disabled="disabled">Select a company</option>
+                    <t t-foreach="companies" t-as="company">
+                        <option t-att-value="company.id">
+                            <t t-esc="company.name" />
+                        </option>
+                    </t>
+                </select>
             </div>
             <div class="col-md-7 col-sm-8">
                 <label for="pms_property_id">Property</label>