[PATCH] Improve "id" field handling in reports
|Reported by:||Matthijs Kooijman||Owned by:||Ryan J Ollos|
|Cc:||Ivanelson Nunes||Trac Release:||0.11|
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)
comment:2 Changed 4 years ago by
|Cc:||Ivanelson Nunes added; anonymous removed|
|Owner:||changed from obs to Ryan J Ollos|
|Status:||new → assigned|