Opened 14 years ago
Closed 14 years ago
#8339 closed defect (fixed)
[patch] Unable to see link "Ticket Template"
Reported by: | anonymous | Owned by: | Richard Liao |
---|---|---|---|
Priority: | high | Component: | TracTicketTemplatePlugin |
Severity: | normal | Keywords: | trivial permission |
Cc: | Trac Release: | 0.12 |
Description
I have installed it as I did for other plugins(that work fine). building from 0.11/ source. adding egg file to plugins/ . in trac.ini: enabled component tickettemplate.*. added [tickettemplate].... restarting apache. Unable to see the link Ticket Template. Please help Thanks
Attachments (0)
Change History (7)
comment:1 Changed 14 years ago by
Status: | new → assigned |
---|
comment:3 Changed 14 years ago by
Keywords: | permission added |
---|---|
Summary: | Unable to see link "Ticket Template" → [patch] Unable to see link "Ticket Template" |
This is related to assigning a wrong permission.
I've found this issue with 0.6 and it continued when upgrading to 0.7 once more today. So I just went on to get it fixed. Following small patch provides the intended behavior - show the admin page with 'Ticket Template' view for any user with TT_ADMIN
permission, (or another admin page and a 'Ticket Template' link, if user has more admin permissions):
@@ -133,7 +133,7 @@ # IAdminPanelProvider methods def get_admin_panels(self, req): - if 'TT_ADMIN' in req.perm: + if 'TRAC_ADMIN' in req.perm: yield ('ticket', _('Ticket System'), 'tickettemplate', _('Ticket Template')) def render_admin_panel(self, req, cat, page, path_info):
Again, this should really be fixed at least for TracTicketTemplatePlugin 0.6 and 0.7, since these versions are know to be affected in a way, that the TT_ADMIN
permission is rendered useless and only users with TRAC_ADMIN
(that implicitly collects all permission anyway) are able to access the admin page.
comment:4 Changed 14 years ago by
Keywords: | trivial added |
---|---|
Priority: | normal → high |
pushing this a bit more
comment:5 Changed 14 years ago by
Uh, very sorry, the previous patch is actually just the other way around:
@@ -133,7 +133,7 @@ # IAdminPanelProvider methods def get_admin_panels(self, req): - if 'TRAC_ADMIN' in req.perm: + if 'TT_ADMIN' in req.perm: yield ('ticket', _('Ticket System'), 'tickettemplate', _('Ticket Template')) def render_admin_panel(self, req, cat, page, path_info):
comment:6 Changed 14 years ago by
See #8609 for a follow-up on permissions, that proposes an second permission to better control the "my templates" selection instead of just 'all-or-nothing'.
comment:7 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
You can check if simplejson is installed first. If yes, look up at trac's log file can get more information.