Changes between Version 1 and Version 2 of Ticket #12915, comment 4
- Timestamp:
- Sep 24, 2017, 2:15:31 PM (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #12915, comment 4
v1 v2 1 1 Recursion is fixed in r16329. 2 2 3 I can see how the logic is similar to [browser:tags/trac-1.2.2/trac/perm.py#L179 DefaultPermissionStore.get_user_permissions]. `subjects`, as calculated in that method, is the set of permission groups possessed by the user. So we canjust copy/paste and modify that method: [4e719b32d/rjollos.git].3 I can see how the logic is similar to [browser:tags/trac-1.2.2/trac/perm.py#L179 DefaultPermissionStore.get_user_permissions]. `subjects`, as calculated in that method, is the set of permission groups possessed by the user. We could just copy/paste and modify that method: [4e719b32d/rjollos.git]. 4 4 5 5 However, another thing to consider is that `DefaultPermissionStore` is really storing permission groups, so maybe it should implement `IPermissionGroupProvider`: [bf3fe0818/rjollos.git].