Modify ↓
Opened 12 years ago
Closed 12 years ago
#10096 closed enhancement (fixed)
support permission handling
Reported by: | falkb | Owned by: | falkb |
---|---|---|---|
Priority: | normal | Component: | SimpleMultiProjectPlugin |
Severity: | normal | Keywords: | planned |
Cc: | Christopher Paredes, thomasd, Ryan J Ollos | Trac Release: | 0.11 |
Description
at least we need PROJECT_ADMIN to hide the project editor for normal users
Attachments (0)
Change History (8)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Cc: | Ryan J Ollos added |
---|
Ryan, I hope this was the right way, though somehow I didn't find an example for has_permission()
comment:3 follow-up: 4 Changed 12 years ago by
It looks correct to me. Two minor comments:
- I'm not sure that specifying the realm in
req.perm('projects')
has any effect, but I'm still trying to understand the consequences of specifying a realm / resource. If it does have an effect, you might similarly want to specifyreq.perm('projects').require('PROJECT_ADMIN')
in place ofreq.perm.require('PROJECT_ADMIN')
. - You might want to allow the page label to be translated, as you've already done for the category label:
yield ('projects', _('Manage Projects'), 'simplemultiproject', 'Projects')
->
yield ('projects', _('Manage Projects'), 'simplemultiproject', _('Projects'))
comment:4 Changed 12 years ago by
Status: | new → assigned |
---|
comment:6 Changed 12 years ago by
comment:7 Changed 12 years ago by
Keywords: | testing added |
---|
Replying to falkb:
TODO maybe also PROJECT_LIST is needed
PROJECT_SETTINGS_VIEW seemed to be more appropriate to what it does. All trials to change the project settings end up with PROJECT_ADMIN privileges are required to perform this operation
with just PROJECT_SETTINGS_VIEW.
Before closing I set this ticket to testing state now.
comment:8 Changed 12 years ago by
Keywords: | testing removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
(In [11641]) refs #10096: support PROJECT_ADMIN, check it in get_admin_panels() and render_admin_panel()