Skip to content

Commit bd17e10

Browse files
sjai-odoovlst-odoo
authored andcommitted
[FIX] point_of_sale: fix tour
- removed `remove_from_ui` which is unnecessarry. - fixed lot refund tour
1 parent a9ea1e3 commit bd17e10

File tree

2 files changed

+1
-17
lines changed

2 files changed

+1
-17
lines changed

addons/point_of_sale/models/pos_order.py

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1125,23 +1125,6 @@ def _get_mail_attachments(self, name, ticket, basic_ticket):
11251125

11261126
return attachments
11271127

1128-
1129-
@api.model
1130-
def remove_from_ui(self, server_ids):
1131-
""" Remove orders from the frontend PoS application
1132-
1133-
Remove orders from the server by id.
1134-
:param server_ids: list of the id's of orders to remove from the server.
1135-
:type server_ids: list.
1136-
:returns: list -- list of db-ids for the removed orders.
1137-
"""
1138-
orders = self.search([('id', 'in', server_ids), ('state', '=', 'draft')])
1139-
orders.write({'state': 'cancel'})
1140-
# TODO Looks like delete cascade is a better solution.
1141-
orders.mapped('payment_ids').sudo().unlink()
1142-
orders.sudo().unlink()
1143-
return orders.ids
1144-
11451128
@api.model
11461129
def search_paid_order_ids(self, config_id, domain, limit, offset):
11471130
"""Search for 'paid' orders that satisfy the given domain, limit and offset."""

addons/point_of_sale/static/tests/pos/tours/ticket_screen_tour.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -201,6 +201,7 @@ registry.category("web_tour.tours").add("LotRefundTour", {
201201
ReceiptScreen.clickNextOrder(),
202202
...ProductScreen.clickRefund(),
203203
TicketScreen.selectOrder("001"),
204+
ProductScreen.clickLine("Product A"),
204205
ProductScreen.clickNumpad("1"),
205206
TicketScreen.toRefundTextContains("To Refund: 1"),
206207
TicketScreen.confirmRefund(),

0 commit comments

Comments
 (0)