Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#7720 closed defect (fixed)

Batchmodify must either expand $USER or filter it out of the owners list.

Reported by: batchmodifyplugin@… Owned by: CuriousCurmudgeon
Priority: normal Component: BatchModifyPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

The owners list (with restricted_owners on) currently shows the 'magic' $USER owner.

In the trac admin this auto expands to your current user. However with batch modifications it sets the owner of tickets to "$USER".

Either it should also be expanded to the current user or skip it all together.
For the latter option I made a small patch for batchmod.js

Attachments (1)

7720-batchmod.htdocs.js.batchmod.patch (455 bytes) - added by anonymous 4 years ago.

Download all attachments as: .zip

Change History (5)

Changed 4 years ago by anonymous

comment:1 Changed 4 years ago by CuriousCurmudgeon

  • Status changed from new to assigned

(In [9370]) refs #7220

  • For consistency I handled this in the same way queries do. The dropdown will still show $USER, but it will be replaced with the current user when actually modifying the ticket.

comment:2 Changed 4 years ago by CuriousCurmudgeon

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:3 Changed 4 years ago by CuriousCurmudgeon

(In [9376]) refs #7720

  • Fixing unit tests Reorganized them into a tests directory like Trac does.
  • Added a few new unit tests for the $USER functionality.

comment:4 Changed 4 years ago by CuriousCurmudgeon

(In [9377]) refs #7720

  • Merged changes from r9376 into 0.11 version.

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from CuriousCurmudgeon. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.