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

Add missing methods to category collection class. #884

Conversation

colinmollenhour
Copy link
Member

@colinmollenhour colinmollenhour commented Jan 16, 2020

These two methods exist on Mage_Catalog_Model_Resource_Category_Flat_Collection but not on Mage_Catalog_Model_Resource_Category_Collection so if flat categories is enabled but they are being temporarily rebuilt and any frontend code relies on these methods, calls to these methods will generate PHP errors since the flat collection and eav collection classes expose a different set of methods on the public API.

@colinmollenhour colinmollenhour changed the title Add missing method to category collection class. Add missing methods to category collection class. Jan 16, 2020
Copy link
Contributor

@Flyingmana Flyingmana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks fine, and called methods are there.
Did not test it myself

@colinmollenhour colinmollenhour merged commit 2e3acd9 into OpenMage:1.9.4.x Jan 20, 2020
@sreichel sreichel added this to the Release 19.4.1 milestone Jun 27, 2020
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Aug 20, 2020
These two methods exist on Mage_Catalog_Model_Resource_Category_Flat_Collection
but not on Mage_Catalog_Model_Resource_Category_Collection so if flat categories
is enabled but they are being temporarily rebuilt and any frontend code relies
on these methods, calls to these methods will generate PHP errors since the
flat collection and eav collection classes expose a different set of methods
on the public API.
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Aug 24, 2020
These two methods exist on Mage_Catalog_Model_Resource_Category_Flat_Collection
but not on Mage_Catalog_Model_Resource_Category_Collection so if flat categories
is enabled but they are being temporarily rebuilt and any frontend code relies
on these methods, calls to these methods will generate PHP errors since the
flat collection and eav collection classes expose a different set of methods
on the public API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants