-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[FIX] website_event_sale: refined boundary check for sold-out tickets
Reproduction: 1. Select an event, then set a certain maximum amount sold on a paid ticket, set as 2 for simplicity 2. Go to the Event webpage, then buy all paid tickets 3. Fill out information and attempt to confirm the order 4. The shopping cart is cleared Reason: We check the event ticket availability in function _cart_update(). In V14 and V15, we added a new step to update the pricelist when confirming an order, which will trigger _cart_update(). In V13 update_pricelist is False by default. In V14 and V15, it is always set to True. The change is to make sure the pricelist is not changed once the customer chooses the pricelist. As a result, for each SO, it checks the ticket availability twice. First at when click register or check out. Second at when submit the address information. When selling the last ticket, the second check fails and leads to an empty cart because the seats_reaserved is updated already at the first step Fix: add new checking conditions increased_quantity = new_qty > old_qty to function _cart_update. We only check the availability again when these new conditions are satisfied. Current issue: This fix will restore the normal workflow for the last ticket. But the workflow has its original issue, e.g. it doesn’t update seats_reserved when increasing the number of tickets. A solution in SaaS-15.3 is to forbid the customer to manually raise the event ticket amount For example: 1. Register 1 ticket, fill in the attendee form and click continue 2. Click review order, increase the number of tickets, then continue 3. The seats_reserved is not updated (can refresh the event page, seats_reserved is the column confirmed) If first register 2 tickets and then change to 1, seats_reserved will change from 2 to 1 This is because seats_reserved is computed from the number of registration. No new registration is needed when increase the ticket number, but the registration will decrease when decrease the ticket number. A solution for it is popping new registration form when the ticket registration increases. opw-2784720 closes odoo#90232 X-original-commit: fc689f0 Signed-off-by: Nicolas Lempereur (nle) <nle@odoo.com> Signed-off-by: Liu Jinjiu (jili) <jili@odoo.com>
- Loading branch information
Showing
4 changed files
with
128 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
92 changes: 92 additions & 0 deletions
92
addons/website_event_sale/static/tests/tours/website_event_sale_last_ticket.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,92 @@ | ||
odoo.define('website_event_sale.tour.last_ticket', function (require) { | ||
'use strict'; | ||
|
||
var tour = require('web_tour.tour'); | ||
|
||
tour.register('event_buy_last_ticket', { | ||
test: true, | ||
url: '/event', | ||
},[{ | ||
content: "Open the Last ticket test event page", | ||
trigger: '.o_wevent_events_list a:contains("Last ticket test")', | ||
}, | ||
{ | ||
content: "Show available Tickets", | ||
trigger: '.btn-primary:contains("Register")', | ||
}, | ||
{ | ||
content: "Select 2 units of `VIP` ticket type", | ||
extra_trigger: '#wrap:not(:has(a[href*="/event"]:contains("Last ticket test")))', | ||
trigger: 'select:eq(0)', | ||
run: 'text 2', | ||
}, | ||
{ | ||
content: "Click on `Order Now` button", | ||
extra_trigger: 'select:eq(0):has(option:contains(2):propSelected)', | ||
trigger: '.a-submit:contains("Register")', | ||
}, | ||
{ | ||
content: "Fill attendees details", | ||
trigger: 'form[id="attendee_registration"] .btn:contains("Continue")', | ||
run: function () { | ||
$("input[name='1-name']").val("Att1"); | ||
$("input[name='1-phone']").val("111 111"); | ||
$("input[name='1-email']").val("att1@example.com"); | ||
$("input[name='2-name']").val("Att2"); | ||
$("input[name='2-phone']").val("222 222"); | ||
$("input[name='2-email']").val("att2@example.com"); | ||
}, | ||
}, | ||
{ | ||
content: "Validate attendees details", | ||
extra_trigger: "input[name='1-name'], input[name='2-name']", | ||
trigger: 'button:contains("Continue")', | ||
}, | ||
{ | ||
content: "Fill address", | ||
trigger: 'form.checkout_autoformat', | ||
run: function () { | ||
$("input[name='name']").val("test1"); | ||
$("input[name='email']").val("test@example.com"); | ||
$("input[name='phone']").val("111 111"); | ||
$("input[name='street']").val("street test 1"); | ||
$("input[name='city']").val("testCity"); | ||
$("input[name='zip']").val("123"); | ||
$('#country_id option:eq(1)').attr('selected', true); | ||
}, | ||
}, | ||
{ | ||
content: "Validate address", | ||
trigger: '.btn-primary:contains("Next")', | ||
}, | ||
{ | ||
// if the seats_available checking logic is not correct, | ||
// the shopping cart will be cleared when selling the last ticket | ||
// the tour test will be failed here | ||
content: "Select `Wire Transfer` payment method", | ||
trigger: '#payment_method label:contains("Wire Transfer")', | ||
}, | ||
// following steps are based on the website_sale_buy.js | ||
{ | ||
content: "Pay", | ||
//Either there are multiple payment methods, and one is checked, either there is only one, and therefore there are no radio inputs | ||
extra_trigger: '#payment_method label:contains("Wire Transfer") input:checked,#payment_method:not(:has("input:radio:visible"))', | ||
trigger: 'button[name="o_payment_submit_button"]:visible:not(:disabled)', | ||
}, | ||
{ | ||
content: "payment finish", | ||
trigger: '.oe_website_sale:contains("Please use the following transfer details")', | ||
// Leave /shop/confirmation to prevent RPC loop to /shop/payment/get_status. | ||
// The RPC could be handled in python while the tour is killed (and the session), leading to crashes | ||
run: function () { | ||
window.location.href = '/contactus'; // Redirect in JS to avoid the RPC loop (20x1sec) | ||
}, | ||
timeout: 30000, | ||
}, | ||
{ | ||
content: "wait page loaded", | ||
trigger: 'h1:contains("Contact us")', | ||
run: function () {}, // it's a check | ||
}, | ||
]); | ||
}); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<odoo> | ||
|
||
<template id="assets_tests" inherit_id="web.assets_tests" name="Website Event Sale Assets Tests"> | ||
<xpath expr="." position="inside"> | ||
<script type="text/javascript" src="/website_event_sale/static/tests/tours/website_event_sale.js"></script> | ||
<script type="text/javascript" src="/website_event_sale/static/tests/tours/website_event_sale_last_ticket.js"></script> | ||
</xpath> | ||
</template> | ||
|
||
</odoo> |