Edgewall Software

Ticket #3163 (closed enhancement: fixed)

Opened 3 years ago

Last modified 7 weeks ago

Implement Description field modification permission outside TICKET_ADMIN level

Reported by: anonymous Owned by: cboos
Priority: high Milestone: 0.11
Component: ticket system Version: 0.9.5
Severity: major Keywords: PermissionPolicy
Cc: nulleke76@…, frans.godschalk@…, trac-ja@…

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

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

Change History

  Changed 3 years ago by cboos

  • owner changed from jonas to cboos

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

follow-up: ↓ 3   Changed 2 years ago by cboos

  • milestone set to 0.11

Maybe this should become configurable, e.g.

[ticket]
editable_description = false

(= current behavior, could be the default)

in reply to: ↑ 2   Changed 2 years ago by anonymous

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.

follow-up: ↓ 5   Changed 2 years ago by sid

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   Changed 23 months ago by cboos

  • 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.

  Changed 20 months ago by anonymous

  • cc nulleke76@…, frans.godschalk@… added

follow-up: ↓ 8   Changed 20 months ago by nulleke76@…

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   Changed 20 months ago by cboos

Replying to nulleke76@gmail.com:

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

  Changed 19 months ago by anonymous

  • summary changed from Implement Description field modification permission outside TICKET_ADMIN level to Implement Description field modification permission outside11 TICKET_ADMIN level

  Changed 19 months ago by cboos

  • severity changed from normal to major
  • summary changed from Implement Description field modification permission outside11 TICKET_ADMIN level to Implement Description field modification permission outside TICKET_ADMIN level

  Changed 13 months ago by sid

#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.

  Changed 11 months ago by cboos

  • priority changed from normal to high

Changed 9 months ago by trac-ja@…

Patch againsts source:trunk@6814

  Changed 9 months ago by trac-ja@…

  • cc trac-ja@… added

I wrote a patch as add TICKET_EDIT_DESCRIPTION permission.

  Changed 9 months ago by cboos

  • milestone changed from 0.11.1 to 0.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?)

  Changed 9 months ago by cboos

  • status changed from new to closed
  • resolution set to fixed

  Changed 9 months ago by cboos

(patch was applied in r6833)

Add/Change #3163 (Implement Description field modification permission outside TICKET_ADMIN level)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from cboos. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.