CriptoMart/addons-cm
2
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

[FIX] website_sale_aplicoop: fix account.tax access error and duplicate home-delivery handler

Browse source 
Add sudo() to pricelist_item and fiscal position fallback in _get_pricing_info
so portal users can price the delivery product without triggering an AccessError
on account.tax. Remove the redundant #home-delivery-btn click handler from
website_sale.js — home_delivery.js already owns that button via
bindShopHomeDeliveryButton(), which manages the active class and localStorage cart.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
GitHub Copilot 2026-05-20 18:20:02 +02:00
parent 27097004fb
commit f983d71ea3
2 changed files with 2 additions and 24 deletions
Download patch file Download diff file Expand all files Collapse all files

4
website_sale_aplicoop/controllers/website_sale_pricing.py
View file

@ -140,7 +140,7 @@ def _get_pricing_info(
product=product, quantity=quantity, target_currency=currency product=product, quantity=quantity, target_currency=currency
) )
price_before_discount = price price_before_discount = price
pricelist_item = env["product.pricelist.item"].browse(rule_id) pricelist_item = env["product.pricelist.item"].sudo().browse(rule_id)
if pricelist_item and pricelist_item._show_discount_on_shop(): if pricelist_item and pricelist_item._show_discount_on_shop():
price_before_discount = pricelist_item._compute_price_before_discount( price_before_discount = pricelist_item._compute_price_before_discount(
product=product, product=product,
@ -155,7 +155,7 @@ def _get_pricing_info(
fiscal_position = ( fiscal_position = (
website.fiscal_position_id.sudo() website.fiscal_position_id.sudo()
if website and getattr(website, "fiscal_position_id", False) if website and getattr(website, "fiscal_position_id", False)
else env["account.fiscal.position"] else env["account.fiscal.position"].sudo()
) )
product_taxes = product.sudo().taxes_id._filter_taxes_by_company(company) product_taxes = product.sudo().taxes_id._filter_taxes_by_company(company)
taxes = fiscal_position.map_tax(product_taxes) if product_taxes else product_taxes taxes = fiscal_position.map_tax(product_taxes) if product_taxes else product_taxes

22
website_sale_aplicoop/static/src/js/website_sale.js
View file

@ -854,28 +854,6 @@
}); });
} }
// Home-delivery toggle button (shop sidebar)
var homeDeliveryBtn = document.getElementById("home-delivery-btn");
if (homeDeliveryBtn) {
// Restore persisted preference on page load
var storedDelivery = sessionStorage.getItem(
"eskaera_is_delivery_" + self.orderId
);
if (storedDelivery === "true") {
homeDeliveryBtn.classList.add("active");
}
homeDeliveryBtn.addEventListener("click", function (e) {
e.preventDefault();
var isNowActive = !homeDeliveryBtn.classList.contains("active");
homeDeliveryBtn.classList.toggle("active", isNowActive);
sessionStorage.setItem(
"eskaera_is_delivery_" + self.orderId,
isNowActive ? "true" : "false"
);
console.log("[HOME_DELIVERY_BTN] toggled to:", isNowActive);
});
}
// Buttons to clear cart (header + footer) // Buttons to clear cart (header + footer)
var clearCartBtns = [ var clearCartBtns = [
document.getElementById("clear-cart-btn"), document.getElementById("clear-cart-btn"),