elgentos / magento2-regenerate-catalog-urls

This extension adds console commands to be able to regenerate; a product rewrite URL based on its url path; a category rewrite URL based on its url path; a category URL path based on its URL key and its parent categories.
MIT License
264 stars 73 forks source link

Missing rewrite for overwritten url_key #50

Closed mortenbirkelund closed 2 years ago

mortenbirkelund commented 2 years ago

I think i have found a scenario where the module is not generating all urls rewrites correctly.

mortenbirkelund commented 2 years ago

Hmm. I seem a bit silly documenting all of these steps only to see afterwards that this is exactly what #49 is addressing. However it seems strange that you would have to know yourself which categories belongs to a storeview . If you run the command for a storeview, where the category is not included in, then the rewrite is still created. So you would have to know exactly which categories to create for each storeview.

peterjaap commented 2 years ago

@mortenbirkelund so basically you want to add the all store option to the category command as well? And/or set it to the default?

mortenbirkelund commented 2 years ago

Yes basically. Set it as the default or as an option depends, at least in my mind, on how it will work.

If it only creates the same rewrites that magento would also have done itself, then I think it should be default. But if it forcefully generates for all storeview regardless, then It would be better as an option.

What do you think?

peterjaap commented 2 years ago

I agree, it doesn't really matter which we pick, as long as it is consistent. I'm closing this issue in favor of #58 , which should fix this.