• 
      

    Limit the valid list of LocalSite admin permissions.

    Review Request #6270 — Created Aug. 26, 2014 and submitted

    Information

    Review Board
    release-2.0.x
    aef0d8f...

    Reviewers

    LocalSite admins were being shown as having all available permissions,
    meaning that some code would appear to grant them permission (typically
    for display) while others would disallow (when actually factoring in the
    permission for anything real).

    The main case where this was apparent was that LocalSite admins were
    seeing "Delete Permanently" for review requests, though the option would
    not work.

    We now have a pre-defined list of valid permissions that LocalSite
    admins can have. This was collected from all the call sites in the
    codebase where permissions are checked. No other permissions are
    considered available to the admin.

    Unit tests pass.

    Description From Last Updated

    'templatetags' imported but unused

    reviewbot reviewbot
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/accounts/backends.py
          reviewboard/site/tests.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/accounts/backends.py
          reviewboard/site/tests.py
      
      
    2. reviewboard/site/tests.py (Diff revision 1)
       
       
      Show all issues
       'templatetags' imported but unused
      
    3. 
        
    david
    1. Ship It!

    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Fixed on release-2.0.x (5b208c9)