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

Skip disabled and not visible products #8

Open
tmotyl opened this issue Oct 24, 2012 · 5 comments
Open

Skip disabled and not visible products #8

tmotyl opened this issue Oct 24, 2012 · 5 comments

Comments

@tmotyl
Copy link

tmotyl commented Oct 24, 2012

Would be nice if indexer can skip products which are disabled or individually not visible (associated products of a configurable product).

@paales
Copy link

paales commented Jun 17, 2013

@IvanChepurnyi Are you planning on adding this functionality? Maybe have the option to completely disable the URL generation of products and only enable category rewrites.

@IvanChepurnyi
Copy link
Member

Hi @paales, I don't have any time to make this happen. Although there will be a very major refactor of the functionality with moving logic out of core_url_rewrite.

@paales
Copy link

paales commented Jun 17, 2013

Hmm, I see what you mean. For our current project we are going to disable the product url rewrites and maybe create a custom router in the future. Disabling only the product rewrites is relatively easy as far as I understand.

Disable the _generateProductRequestPathIndex method and the _importFromProductRequestPath method we're good to go. (Or am I missing something).

@IvanChepurnyi
Copy link
Member

Yes, both of those methods can be used.

@mklooss
Copy link

mklooss commented Jul 4, 2013

DnD Build up an Extension that can ignore disabled products and also can ignore the categorie path.
So i saw u ( @IvanChepurnyi ) did not replace any model but i did not have time actually to merge these extensions

EDIT: oh i forgot the link
http://www.dnd.fr/2012/09/magento-patch-how-to-optimize-re-index-processing-time-for-url-rewrite/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants