Skip to content

Mage_Core_Model_Store::setConfig() not present in m2 and required #1867

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

Closed
birchestx opened this issue Sep 14, 2015 · 3 comments
Closed

Mage_Core_Model_Store::setConfig() not present in m2 and required #1867

birchestx opened this issue Sep 14, 2015 · 3 comments

Comments

@birchestx
Copy link

This method seems to have been removed from M2, wanting to understand the reasoning for this one. We use it to temporarily set carriers to enabled so that we can call in a backup scenario.

I've checked the M2 code and all the way up the chain there is no access to change the config values dynamically from what I can see.

@kandy
Copy link
Contributor

kandy commented Sep 14, 2015

Please use \Magento\Framework\App\MutableScopeConfig::setValue method instead

@kandy
Copy link
Contributor

kandy commented Sep 14, 2015

The reasoning for this was to move all knowledge about configuration to one module/class and split on read only part and mutable part to follow "interface segregation principle"

@birchestx
Copy link
Author

Thanks for info.

magento-team pushed a commit that referenced this issue Dec 21, 2017
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

2 participants