Opened 7 years ago

Last modified 7 years ago

#13234 closed defect

[PATCH] Default to leaving ticket owner unchanged rather than taking ownership — at Version 1

Reported by: Jon Ashley Owned by:
Priority: normal Component: FlexibleAssignToPlugin
Severity: minor Keywords:
Cc: Trac Release: 1.0

Description (last modified by Jon Ashley)

The FlexibleAssignToPlugin pre-selects the name of the authenticated user who is viewing the ticket into the select controls that are managed by the plugin. This means that the default behaviour is for the user to take ownership of a ticket on a ticket update.

The comment at line 198 of the main source file states "Check to see if the current owner is in the list of valid owners; if so, pre-select them in the select control". This suggest that the intended default behaviour was to leave the current owner unchanged (which is why I've reported this as a defect rather than an enhancement).

Whether it was intended or not, I much prefer to have the select control pre-selected with the existing ticket owner, so that the user has to actively choose to take ownership. The attached patch is used in my local installation to achieve this. Without this patch, I find that users do quite a lot of taking ownership by accident.

Change History (2)

Changed 7 years ago by Jon Ashley

Attachment: flexibleassignto.diff added

Patch to flexibleassignto.py to keep ticket ownership unchanged by default

comment:1 Changed 7 years ago by Jon Ashley

Description: modified (diff)
Note: See TracTickets for help on using tickets.