From dac89886cd419714a7a14311f0491c7e4f9da100 Mon Sep 17 00:00:00 2001
From: Suherdy Yacob <suherdy.yacob@mapan.co.id>
Date: Thu, 7 May 2026 16:57:09 +0700
Subject: [PATCH] refactor: update xpath selectors to use hasclass and correct
 component targets in POS XML templates

---
 .../src/app/screens/product_screen/action_pad/action_pad.xml  | 4 ++--
 static/src/app/screens/ticket_screen/ticket_screen.xml        | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/static/src/app/screens/product_screen/action_pad/action_pad.xml b/static/src/app/screens/product_screen/action_pad/action_pad.xml
index 21db3dc..fad4509 100644
--- a/static/src/app/screens/product_screen/action_pad/action_pad.xml
+++ b/static/src/app/screens/product_screen/action_pad/action_pad.xml
@@ -2,10 +2,10 @@
 <templates id="template" xml:space="preserve">
     <!-- Hide payment button if role is waiter -->
     <t t-name="pos_employee_role.ActionpadWidget" t-inherit="point_of_sale.ActionpadWidget" t-inherit-mode="extension">
-        <xpath expr="//button[contains(@class, 'pay-order-button')]" position="attributes">
+        <xpath expr="//button[hasclass('pay-order-button')]" position="attributes">
             <attribute name="t-if">this.pos.cashier.pos_role != 'waiter'</attribute>
         </xpath>
-        <xpath expr="//button[contains(@class, 'pay-order-button')]" position="after">
+        <xpath expr="//button[hasclass('pay-order-button')]" position="after">
             <button t-if="this.pos.cashier.pos_role == 'waiter' and this.pos.config.basic_receipt" 
                     class="button btn btn-secondary btn-lg py-3 d-flex align-items-center justify-content-center flex-fill"
                     t-att-class="{'col-3': showFastPaymentMethods}" 
diff --git a/static/src/app/screens/ticket_screen/ticket_screen.xml b/static/src/app/screens/ticket_screen/ticket_screen.xml
index b5c4d9c..d00420f 100644
--- a/static/src/app/screens/ticket_screen/ticket_screen.xml
+++ b/static/src/app/screens/ticket_screen/ticket_screen.xml
@@ -2,7 +2,7 @@
 <templates id="template" xml:space="preserve">
     <!-- Hide refund button in Ticket Screen from waiter and cashier -->
     <t t-name="pos_employee_role.TicketScreen" t-inherit="point_of_sale.TicketScreen" t-inherit-mode="extension">
-        <xpath expr="//button[@actionToTrigger.bind='onDoRefund']" position="attributes">
+        <xpath expr="//ActionpadWidget[@actionToTrigger.bind='onDoRefund']" position="attributes">
             <attribute name="t-if">getHasItemsToRefund() and (this.pos.cashier.pos_role == 'outlet_manager' or this.pos.cashier.pos_role == 'area_manager')</attribute>
         </xpath>
     </t>