Edgewall Software

Changes between Version 1 and Version 2 of Ticket #12915, comment 4


Ignore:
Timestamp:
Sep 24, 2017, 2:15:31 PM (4 years ago)
Author:
Ryan J Ollos

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #12915, comment 4

    v1 v2  
    11Recursion is fixed in r16329.
    22
    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 can just copy/paste and modify that method: [4e719b32d/rjollos.git].
     3I 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].
    44
    55However, another thing to consider is that `DefaultPermissionStore` is really storing permission groups, so maybe it should implement `IPermissionGroupProvider`: [bf3fe0818/rjollos.git].