Project

General

Profile

Actions

Defect #33110

closed

Sort does not work with group by datetime columns

Added by Alexander Meindl over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
Issues list
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

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

Related to Redmine - Feature #13803: Implement grouping issues by date (start, due, creation, update, closing dates)ClosedJean-Philippe Lang

Actions
Actions #1

Updated by Go MAEDA over 4 years ago

Could you describe more details? I could not find any problem. Rows are sorted if you click the column header.

Actions #2

Updated by Alexander Meindl over 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:

  1. go to spent time list
  2. Group results by: Created (or any other of type TimestampQueryColumn)
  3. 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)

Actions #3

Updated by Marius BĂLTEANU over 4 years ago

  • Status changed from New to Confirmed
Actions #4

Updated by Marius BĂLTEANU over 4 years ago

  • Related to Feature #13803: Implement grouping issues by date (start, due, creation, update, closing dates) added
Actions #5

Updated by Marius BĂLTEANU over 4 years ago

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?

Actions #6

Updated by Marius BĂLTEANU over 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
Actions #7

Updated by Alexander Meindl over 4 years ago

Hi Marius,

thanks for the patch! This solved the problem. Create work and not easy to address the problem.

Actions #8

Updated by Jean-Philippe Lang over 4 years ago

  • Status changed from Confirmed to Resolved

Committed, thanks.

Actions #9

Updated by Jean-Philippe Lang over 4 years ago

  • Status changed from Resolved to Closed
  • Resolution set to Fixed

Merged.

Actions

Also available in: Atom PDF