-
Notifications
You must be signed in to change notification settings - Fork 31
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
Fixed in Magento 1.9.3.1 #9
Comments
Amazing news, thank you @sprankhub! I'll make sure to update the main Now, If only they provided accreditation like they do for security patches. I mean, a website being down is no big deal right ¯\(ツ)/¯ |
What is that accreditation!? ¯\(ツ)/¯ |
You get $$$ for security patches and listed as "Reporter" on the patch breakdown. Kudos basically. I know Magento read this article because I could see them in the referrers, but it would be nice for them to acknowledge that they took this work from here and included it in the core. Oh well, I must try not to be bitter! |
Thanks for the info, I've updated the main https://github.com/AmpersandHQ/magento-ce-ee-config-corruption-bug#update-3-baked-into-the-core |
@convenient Luke Thanks for such awesome fix, we are having a similar issue, sometimes our cart page body comes as white/ black but with correct footer and header. I applied your patches but I think you may share some insight or give more idea to debug this further ? Can you share your last Config.php with the changes you mentioned in Improvement section. |
Thanks for all your work!
You may want to add that this issue is fixed in Magento 1.9.3.1. See https://github.com/OpenMage/magento-mirror/blob/magento-1.9/app/code/core/Mage/Core/Model/Config.php#L260
The text was updated successfully, but these errors were encountered: