Edgewall Software
Modify

Opened 18 years ago

Closed 17 years ago

Last modified 15 years ago

#3163 closed enhancement (fixed)

Implement Description field modification permission outside TICKET_ADMIN level

Reported by: anonymous Owned by: Christian Boos
Priority: high Milestone: 0.11
Component: ticket system Version: 0.9.5
Severity: major Keywords: PermissionPolicy
Cc: nulleke76@…, frans.godschalk@…, trac-ja@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I've found a very limiting restriction not being able to give Description field modification permission to users without TICKET_ADMIN privileges.

I see two solutions:

  • moving Description field modification permission to TICKET_MODIFY level
  • creating a new permission level between TICKET_MODIFY e TICKET_ADMIN (TICKET_SUPERVISOR, for example) with Description field modification permission

Attachments (1)

Trac-0.11dev_r6814_ticketdescription.patch (5.7 KB ) - added by trac-ja@… 17 years ago.
Patch againsts source:trunk@6814

Download all attachments as: .zip

Change History (18)

comment:1 by Christian Boos, 18 years ago

Owner: changed from Jonas Borgström to Christian Boos

I would suggest the first case, provided we also give access to the Description change history (see #2945)

comment:2 by Christian Boos, 18 years ago

Milestone: 0.11

Maybe this should become configurable, e.g.

[ticket]
editable_description = false

(= current behavior, could be the default)

in reply to:  2 comment:3 by anonymous, 18 years ago

Replying to cboos:

Maybe this should become configurable, e.g.

[ticket]
editable_description = false

(= current behavior, could be the default)

I think a new permission would be more appropriate. TICKET_ADMIN would of course already have the new permission, but this permission could also be given to other users independent of the rest of the TICKET_ permissions.

comment:4 by sid, 18 years ago

I think a configuration option is not a great solution because this is really a permissions issue.

Adding to an existing permission (TICKET_CHGPROP or TICKET_MODIFY) would be a good solution. Adding another TICKET_ permission would work as well, but there are quite a few TICKET_* permissions already, and adding too many will start to get tedious if you have to assign permissions for editing every field.

in reply to:  4 comment:5 by Christian Boos, 18 years ago

Keywords: PermissionPolicy added

Replying to sid:

Adding another TICKET_ permission would work as well, but there are quite a few TICKET_* permissions already, and adding too many will start to get tedious if you have to assign permissions for editing every field.

Ideally, this would be an EDIT permission on the ticket description field. I wonder if we could add the field granularity to the fine-grained permissions…

It would certainly be more convenient to implement as a new TICKET_EDIT permission, though.

comment:6 by anonymous, 17 years ago

Cc: nulleke76@… frans.godschalk@… added

comment:7 by nulleke76@…, 17 years ago

Another idea is to only allow editing of the description field for TICKET_CHGPROP for people that initiated the ticket (until it is accepted by the person that is assigned to it.) after that the assigned person will be able to edit the ticket description.

in reply to:  7 comment:8 by Christian Boos, 17 years ago

Replying to nulleke76@gmail.com:

This could probably be done by a PermissionPolicy plugin, after the TICKET_EDIT permission has been added.

comment:9 by anonymous, 17 years ago

Summary: Implement Description field modification permission outside TICKET_ADMIN levelImplement Description field modification permission outside11 TICKET_ADMIN level

comment:10 by Christian Boos, 17 years ago

Severity: normalmajor
Summary: Implement Description field modification permission outside11 TICKET_ADMIN levelImplement Description field modification permission outside TICKET_ADMIN level

comment:11 by sid, 17 years ago

#5251 is marked as duplicate of this ticket. It requests finer grained control over ticket permissions. Specifically, it asks for a separate permission for changing the ticket resolution.

comment:12 by Christian Boos, 17 years ago

Priority: normalhigh

by trac-ja@…, 17 years ago

Patch againsts source:trunk@6814

comment:13 by trac-ja@…, 17 years ago

Cc: trac-ja@… added

I wrote a patch as add TICKET_EDIT_DESCRIPTION permission.

comment:14 by Christian Boos, 17 years ago

Milestone: 0.11.10.11

Thanks for the patch (and please once again: could you tell us who's behind the trac-ja@ address, so that we can fix the THANKS file?)

comment:15 by Christian Boos, 17 years ago

Resolution: fixed
Status: newclosed

comment:16 by Christian Boos, 17 years ago

(patch was applied in r6833)

comment:17 by Christian Boos, 15 years ago

See #8548 for a follow-up.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.