[BUG] website_sale_checkout_skip_payment not compatible with delivery methods about e-commerce

[victor-champonnois]

The issue happens at the last step

Here, when changing the delivery method, the method _update_website_sale_delivery should be triggered by this JS code.

But when the customer is allowed to skip_payment, the JS code throws an error at this line :

Uncaught TypeError: 
``$pay_button.data(...) is undefined
_onCarrierClick http://localhost:8012/web/content/985-5287018/1/web.assets_frontend.js:295

at this line

so the method is not tiggered, the carrier is not updated, and the first one on the list is used.

I think this might be because the payment route is skipped and we go directly to the confirmation route (like below).

e-commerce/website_sale_checkout_skip_payment/views/website_sale_template.xml

Lines 14 to 18 in d049af5

	<form target="_self" action="/shop/confirmation" method="post" class="float-right">
	<input type="hidden" name="csrf_token" t-att-value="request.csrf_token()" />
	<a role="button" class="btn btn-primary a-submit" href="#">
	<span>Confirm Order <span class="fa fa-chevron-right"></span></span>
	</a>

However I don't have a clearer idea.
@sergio-teruel Do you know a way to fix this issue ?

[sergio-teruel]

Sorry @victor-champonnois The website_sale_checkout_skip_payment module has not include any JS code, so I think that this error is related to other module...

[chienandalu]

@sergio-teruel I took a look and it seems to be related to the absence of the payment button element. https://github.com/odoo/odoo/blob/c53081f10befd4f1c98e46a450ed3bc71a6246ed/addons/website_sale_delivery/static/src/js/website_sale_delivery.js#L12

Anyway, I couldn't reproduce the bug in the versions we're currently supporting for our customers (>13).

So @victor-champonnois feel free to propose a PR patching de issue for 12.0

[github-actions]

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.