Modify ↓
Opened 13 years ago
Closed 13 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 13 years ago by
comment:2 Changed 13 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 13 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 13 years ago by
| Status: | new → assigned |
|---|
comment:6 Changed 13 years ago by
comment:7 Changed 13 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 13 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()