Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Magento produces product category rewrites even if "Use Categories Path for Product URLs" is disabled #932

Open
tmotyl opened this issue Apr 22, 2020 · 6 comments
Labels
performance Performance related

Comments

@tmotyl
Copy link
Contributor

tmotyl commented Apr 22, 2020

Despite configuration "Use Categories Path for Product URLs" (catalog/seo/product_use_categories) is disabled Magento adds all urls of product in categories to the url rewirte table, making it unnecessary big and making indexing slower.

https://magento.stackexchange.com/questions/62999/prevent-automatic-url-rewrites-created-that-include-category-url-key-in-product

@tmotyl tmotyl added performance Performance related url rewrites labels Apr 22, 2020
@rvelhote
Copy link
Contributor

We have 98 store views and our core_url_rewrite table has 7.8 million records. As you can imagine reindexing is one of the major pain points we have with Magento.

I've always wanted to try the module referenced in that StackExchange post however there are incompatibilities with a MageWorx SEO module we are currently using that also overrides some of the indexing classes.

@tmotyl
Copy link
Contributor Author

tmotyl commented Apr 27, 2020

@rvelhote want to join forces? I'm working on the solution and have a prof of concept already. Are you on OpenMage discord?
if not, please join https://discord.gg/vvZW5Q and we can talk
I have checked the dnd module, but it did not solve the issue in all cases and have some compatibility and quality issues.

@damien-biasotto
Copy link
Contributor

@tmotyl @rvelhote When I was using Magento (4-5 years ago) I was using the following plugin to address this issue: https://github.com/mklooss/Loewenstark_UrlIndexer.

@tmotyl
Copy link
Contributor Author

tmotyl commented May 12, 2020

thanks @damien-biasotto I will take a look

@leissbua
Copy link
Contributor

Please skip Loewenstark_UrlIndexer in favor of DnD_PatchIndexUrl, it has core quality and could directly be integrated. We use that since years for nearly every Magento1 project that we have.

@kiatng
Copy link
Contributor

kiatng commented Jan 19, 2024

@leissbua Have you checkout this draft PR #3267?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance related
Projects
None yet
6 participants