Defect #33110
closed
Sort does not work with group by datetime columns
0%
Description
It is not possible to change sort order with introducted group by date columns (see #13803).
If you want to change sort order by clicking on column header, no changes are visible.
Files
Related issues
Updated by Go MAEDA about 4 years ago
Could you describe more details? I could not find any problem. Rows are sorted if you click the column header.
Updated by Alexander Meindl about 4 years ago
Hi,
after testing it again, I found out, that this problem exists only in time entries list and project list. Issue list works as excepted.
E.g. for the bug:
- go to spent time list
- Group results by: Created (or any other of type TimestampQueryColumn)
- try to sort colmns (this does not work for any column)
ps: no plugins are installed for my tests. Redmine 4.1, Ruby 2.6.5, MySQL (also tested it with Postgres)
Updated by Marius BĂLTEANU about 4 years ago
- Status changed from New to Confirmed
Updated by Marius BĂLTEANU about 4 years ago
- Related to Feature #13803: Implement grouping issues by date (start, due, creation, update, closing dates) added
Updated by Marius BĂLTEANU about 4 years ago
- File 0001-Fix-sort-does-not-work-with-group-by-datetime-column.patch 0001-Fix-sort-does-not-work-with-group-by-datetime-column.patch added
- Assignee set to Jean-Philippe Lang
- Target version set to 4.1.1
I've investigated this issue and the sort works, but not as expected because the sort takes into consideration the time of timestamp columns and not only the date value as the grouping does. Also, the problem can be reproduced on issues list as well if you have the correct test data. More details in the patch.
I was to able to fix by using the attached patch. Jean-Philippe, please take a look on this. All tests pass on my CI: https://gitlab.com/redmine-org/redmine/pipelines/130760933
Alexander Kulemin, can you confirm the fix, please?
Updated by Marius BĂLTEANU about 4 years ago
- Subject changed from Sort does not work with group by date columns to Sort does not work with group by datetime columns
Updated by Alexander Meindl about 4 years ago
Hi Marius,
thanks for the patch! This solved the problem. Create work and not easy to address the problem.
Updated by Jean-Philippe Lang about 4 years ago
- Status changed from Confirmed to Resolved
Committed, thanks.
Updated by Jean-Philippe Lang about 4 years ago
- Status changed from Resolved to Closed
- Resolution set to Fixed
Merged.