Edgewall Software

Changes between Version 1 and Version 2 of Ticket #11244, comment 3


Ignore:
Timestamp:
Oct 9, 2013, 11:55:28 PM (11 years ago)
Author:
Ryan J Ollos

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #11244, comment 3

    v1 v2  
    33I like that it localizes all of the elevated permission checks for read-only pages to a single location. Additional work is needed - adding functional tests and an environment upgrade step.
    44
    5 With the changes made so far, there is really no use case I can see for removing `WikiPermissionPolicy` from `permission_policies` in //trac.ini//. Removing it would result in a confusing situation in which there is a //read-only// checkbox on the form that has no effect. Therefore we should probably not add the //read-only// checkbox to the edit form unless `WikiPermissionPolicy` is active.
     5Removing `WikiPermissionPolicy` from `permission_policies` in //trac.ini// would result in a confusing situation in which there is a //read-only// checkbox on the form that has no effect. Therefore we should probably not add the //read-only// checkbox to the edit form unless `WikiPermissionPolicy` is active.