[PATCH] Improve "id" field handling in reports
|Reported by:||matthijs||Owned by:||rjollos|
The plugin incorrectly assumes that there will be a field "id" in each row of the report view. Also, it assumes that the "id" field references a ticket, while it may reference other things (like reports. commits, wikipages) through the setting of the "realm" field. Finally, there is the "ticket" field, which can also contain the ticket id. See http://trac.edgewall.org/wiki/TracReports#Automaticallyformattedcolumns for details.
The attached patch fixes all three of these issues. Note that it resorts to not applying access control when no id field is present or it does not reference a ticket, since then it cannot know which ticket is being displayed.
Change History (9)
Changed 6 years ago by matthijs
comment:2 Changed 3 years ago by rjollos
- Cc ivanelson added; anonymous removed
- Owner changed from obs to rjollos
- Status changed from new to assigned