Defect #23839
closed"Invalid query" (Error 500) message with MS SQL when displaying an issue from a list grouped and sorted by fixed version
0%
Description
Prerequisite: this happens only with MS SQL
- Filter a list of issues by grouping on the fixed version field, and showing the priority field
- Click on the Fixed version field to sort on it in descending order, click on the priority field to also sort on that field
This should result in a query like this one for example - Click on one bug of the list
Result: Error 500, Invalid query. In the log, it is indicated that there are duplicated column names in an ORDER BY clause
Note: the issue also happens on MySQL, but MySQL does not consider duplicated sort columns as an error.
The issue happens in the feature to be able to navigate with arrows to the next or previous bug in the list we came from.
As the Fixed_version field implements a call back with its own sortable columns, this comes into the group_by_sort_order statement, which then brings columns duplicated with the ones coming from the options.
Note that with a MySQL-backed Redmine site, the effect is that the order of the list can be incorrect in the previous/Next arrows on the displayed bug.
See attached patch for a suggested fix.
Files
Updated by Olivier Houdas about 8 years ago
It's nicer when testing for non-nil object before calling a method on it (map)... sorry for the bugged previous patch.
... and forgot to save after adding missing spaces on last line...
Updated by Jean-Philippe Lang about 8 years ago
- Status changed from New to Resolved
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Different fix applied in r15852 with a test. Thanks for pointing this out.
Updated by Jean-Philippe Lang about 8 years ago
- Status changed from Resolved to Closed