Edgewall Software
Modify ↓

Opened 19 years ago

Closed 19 years ago

#1533 closed defect (fixed)

Allow redefinition of dynamic report variables in URL

Reported by: pkou at ua.fm Owned by: Christopher Lenz
Priority: normal Milestone: 0.9
Component: report system Version: 0.8
Severity: trivial Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Automatic variables that are created for reports (e.g. USER) shall not be overridden if they are specified in URL.

For now, Trac does not allow to see My Tickets report for other users. Example: http://url/project/report/7?USER=another-user

Patch for trunk: 

Index: Report.py
===================================================================
--- Report.py   (revision 1648)
+++ Report.py   (working copy)
@@ -457,7 +457,8 @@
             report_args[arg] = val

         # Set some default dynamic variables
-        report_args['USER'] = req.authname
+        if not report_args.has_key('USER'):
+            report_args['USER'] = req.authname

         return report_args

Attachments (0)

Change History (2)

comment:1 by Christopher Lenz, 19 years ago

Owner: changed from daniel to Christopher Lenz
Status: new → assigned

comment:2 by Christopher Lenz, 19 years ago

Resolution: → fixed
Status: assigned → closed

Patch applied in [1940]. Thanks.

Modify Ticket

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

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