Bug #3845
closed
Hide empty categories only counting a depth of 1
Description
Hide empty parameter only counts the groups directly in that category. It should use the new groupCountRecursiveTotal function to ensure the user has no access to at least 1 group somewhere within that category (function has unlimited depth with access checks).
Updated by krileon over 11 years ago
This causes an infinite loop as these functions also call the access checks function.
Updated by krileon over 11 years ago
Believe the loop is due to checking parent access while grabbing children and thus the loop. Need to be able to disable various other checks when checking for count as we really just need to know if the category/group can be accessed or not.
Updated by krileon over 11 years ago
- Status changed from Assigned to Rejected
- % Done changed from 0 to 100
Closing as not going to fix. Fixing will result in massive changes to authorization usage as well as result in very large performance hit checking recursively if a group exists. This should be used keeping in mind it only checks 1 depth and description has been updated to reflect this.