Vauxoo / odoo

Fork of Odoo (formerly OpenERP). [This project is not publically mantained just born for internal usage with some little patches] go to official repository on github.com/odoo/odoo

https://www.odoo.com

Other

9 stars 9 forks source link

Draft: [FIX] website: Create the URLs the same way slug() does inside the sitemap #527

Closed sebasdrk17 closed 1 year ago

sebasdrk17 commented 1 year ago

This MR is reference to task#63157

This is a fix based on https://github.com/odoo/odoo/commit/b997684b062bb1010c52f4af49f72f2be570e392 which addresses the same problem but for v14.

Description of the issue/feature this PR addresses:

The sitemap creates a different URL compared to slug() this causes problems with google since the URLs may vary.

Current behavior before PR:

The URLs for products are constructed by ID and name

Desired behavior after PR is merged:

Both URLs should be the exact same.

-- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

sebasdrk17 commented 1 year ago

This PR will be used as a patch.