-
Notifications
You must be signed in to change notification settings - Fork 18
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
Not needed hard dependency between bundle and catalog search modules #104
Comments
Hi @tzyganu. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
where @tzyganu do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @TomashKhamlai. Thank you for working on this issue.
|
@tzyganu such bug report is not valid as |
Looks like this is not the only modularity violation, |
@tzyganu, thank you for reporting this issue. It is expected that if you broke any dependency Magento will not work. This report seems to me like "add us more granular modularity" feature request. |
It occours if you disable the module Magento_ConfigurableProduct too. When I run bin/magento setup:upgrade I have the follow error: Module 'Magento_Indexer': |
Preconditions (*)
Steps to reproduce (*)
Magento_Bundle
by changing 1 to 0 inconfig.php
prior to installing magento since it cannot be disabled via cli because of circular dependencies.bin/magento indexer:set-mode manual catalogsearch_fulltext
Expected result (*)
manual
.Actual result (*)
1.console command reports an error
Additional info.
The problem is caused by this line
<table name="catalog_product_bundle_selection" entity_column="parent_product_id" />
from themview.xml
file from the Catalog Search module.Moving this line to the bundle module looks like it solves the problem.
The text was updated successfully, but these errors were encountered: