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_Newsletter cannot be disabled #26194
Comments
Hi @kassner. 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:
For more details, please, review the Magento Contributor Assistant documentation. @kassner do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
I could write a PR for that, but given this will break backwards compatibility, which Magento release could a PR be targeted to? |
2.4-develop i would imagine is currently accepting breaking changes as 2.4 is not out yet. |
Hi @shikhamis11. Thank you for working on this issue.
|
Hi @augrinovska. Thank you for working on this issue.
|
Hi @kassner, Your Actual Result is related to the permissions of the generated folder.
Please follow the documentation to set up the right permissions to the folder -> https://devdocs.magento.com/guides/v2.3/config-guide/prod/prod_file-sys-perms.html @magento-engcom-team I couldn't reproduce the issue on |
The Actual result has two lines, the first one being:
Because that class doesn't exist, the model generation for the second one fails as well. It has nothing to do with the permissions, the second error happens because it was trying to read the class from the In any case, this can be also very easily checked by looking into the source code. For example, app/code/Magento/Customer/Block/Account/Dashboard.php on So, please open the ticket again, as you will not be able to test this error on |
Hi @engcom-Hotel. Thank you for working on this issue.
|
✅ Confirmed by @engcom-Hotel Issue Available: @engcom-Hotel, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Hi @vimal-ranosys. Thank you for working on this issue.
|
@magento I am working on this. |
#dmcdindia2020 |
@sivaschenko @sidolov This looks like a fun issue to work on, but it's still unclear what would be the best way to approach it, considering backwards compatibility. For example, What kind of changes would be welcome here & are there any plans to have a 2.5-develop branch to point such PRs to ? |
@korostii it looks like we'll need to introduce the alternative to the API classes dependent on Newsletter and mark that API as deprecated first (I think it would be good to do that in 2.4, so that we can remove them in 2.5). @sidolov what do you think about this? Yes, there are plans to create 2.5-develop branch soon |
Any updates on this? |
There are several classes under the
Magento\Customer
(and one underMagento\Review
) namespace that are requiringMagento\Newsletter
classes, making it impossible to successfully disable/remove the newsletter module, if one uses another solution.Magento\Customer\Block\Account\Dashboard
Magento\Customer\Block\Account\Dashboard\Info
Magento\Customer\Block\Adminhml\Edit\Tab\Newsletter
Magento\Customer\Block\Adminhml\Edit\Tab\Newsletter\Grid
Magento\Customer\Block\Adminhml\Edit\Tab\Newsletter\Grid\Renderer\Status
Magento\Customer\Controller\Adminhtml\Index
Magento\Customer\Controller\Adminhtml\Index\Cart
Magento\Customer\Controller\Adminhtml\Index\Newsletter
Magento\Customer\Controller\Adminhtml\Index\Save
Magento\Customer\Controller\Adminhtml\Index\Viewfile
Magento\Customer\Controller\Adminhtml\Customer\InvalidateToken
Magento\Review\Block\Customer\ListCustomer
From those classes, several ones are requiring
\Magento\Newsletter\Model\SubscriberFactory
but they end up not even using it, basically declaring dependencies to classes that are not needed.Preconditions
composer.json
:Steps to reproduce
composer update --lock -vvv
;php bin/magento setup:di:compile
Expected result
Actual result
The text was updated successfully, but these errors were encountered: