-
-
Notifications
You must be signed in to change notification settings - Fork 115
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
Fix loading CategoryFilter items #567
Conversation
Codecov Report
@@ Coverage Diff @@
## 3.x #567 +/- ##
=========================================
Coverage 60.26% 60.26%
Complexity 442 442
=========================================
Files 43 43
Lines 1427 1427
=========================================
Hits 860 860
Misses 567 567
Continue to review full report at Codecov.
|
@@ -20,7 +20,7 @@ | |||
/** | |||
* @method PagerInterface getRootCategoriesPager(int $page = 1, int $limit = 25, array $criteria = []) | |||
* @method PagerInterface getSubCategoriesPager(int $categoryId, int $page = 1, int $limit = 25, array $criteria = []) | |||
* @method CategoryInterface[] getRootCategoriesForContext(ContextInterface|string|null $context) | |||
* @method CategoryInterface[] getRootCategoriesForContext(ContextInterface|null $context) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add a test about this ?
2c1283b
d35f7e5
to
96a318f
Compare
Done |
b02f7f8
to
320c47c
Compare
In sonata-project#567 the tests introduced throw some warnings because the interface does not have the mocked method.
In #567 the tests introduced throw some warnings because the interface does not have the mocked method.
Subject
The filter was calling the wrong method. The called method does allow passing a string value as a parameter.
I am targeting this branch, because this is a bugfix.
Changelog