[IMP] website_sale: avoid pollute urls with #attrs if no variant

closes odoo/odoo#159474

Signed-off-by: Valentin Chevalier <vcr@odoo.com>

addons/website_sale/models/product_product.py

@@ -1,5 +1,7 @@
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
+from werkzeug.urls import url_join
+
 from odoo import _, api, fields, models
 from odoo.exceptions import ValidationError
 
@@ -46,7 +48,10 @@ class Product(models.Model):
     def _compute_product_website_url(self):
         for product in self:
             attributes = ','.join(str(x) for x in product.product_template_attribute_value_ids.ids)
-            product.website_url = "%s#attr=%s" % (product.product_tmpl_id.website_url, attributes)
+            url = product.product_tmpl_id.website_url
+            if attributes:
+                url = url_join(url, f"#attr={attributes}")
+            product.website_url = url
 
     def _prepare_variant_values(self, combination):
         variant_dict = super()._prepare_variant_values(combination)

addons/website_sale/static/src/js/website_sale.js

@@ -185,6 +185,9 @@ export const WebsiteSale = publicWidget.Widget.extend(VariantMixin, cartHandlerM
      */
     _setUrlHash: function ($parent) {
         var $attributes = $parent.find('input.js_variant_change:checked, select.js_variant_change option:selected');
+        if (!$attributes.length) {
+            return;
+        }
         var attributeIds = $attributes.toArray().map((elem) => $(elem).data("value_id"));
         window.location.replace('#attr=' + attributeIds.join(','));
     },