-
-
Notifications
You must be signed in to change notification settings - Fork 436
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 bug related to filter in Admin UI for Role Users #1061
Conversation
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.
Thanks!
Looks good! I don't see any problem with the solution. I tested it and it works. |
I can confirm this bug. The type casting was implemented with PATCH SUPEE-11155 CE 1.9.4.0 v3-2019-06-18-08-34-01. Maybe you make a type cast to array? |
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.
This line should always return an array.
Unable to filter Role Users in Permission->Roles->Role Users or go to next Page or change the number of rows displayed per page.
This bug is introduced in pull request d644768
Mage_Adminhtml_Block_Permissions_Role_Grid_User->_getUsers() is returning an int instead of an array on line 160.