Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken sort by date after last upgrade #1004

Closed
ostasevych opened this issue Apr 25, 2024 · 4 comments · Fixed by #1110
Closed

Broken sort by date after last upgrade #1004

ostasevych opened this issue Apr 25, 2024 · 4 comments · Fixed by #1110
Assignees
Labels
3. to review Waiting for reviews bug Something isn't working

Comments

@ostasevych
Copy link

Steps to reproduce

  1. Have some view where the date field is present, eg Date of modification
  2. Set the rule in view to sort by date and check that it is working
  3. Upgrade to 0.7.0
  4. Open any view where sorting by date rule is present
  5. Observe that the sorting is broken
  6. Check sorting by ABC - observe that it is partially working apart of cyrillic letters Є, І, which goes at the end mistakenly

Expected behavior

The sorting by date should work as expected

Actual behavior

The sorting by date is not working at all, independently of ASC or DESC

Tables app version

0.7.0

Browser

Brave

Client operating system

Win10 x64

Operating system

Ubuntu

Web server

Apache

PHP engine version

PHP 8.2

Database

MariaDB

Additional info

I may assume that this may be related to the changes in the date format, however the Date of Modification is shown correctly.

@ostasevych ostasevych added 0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working labels Apr 25, 2024
@enjeck
Copy link
Contributor

enjeck commented Apr 26, 2024

Hi @ostasevych. I'm unable to reproduce this. Can you please share a screencast of what's happening?

@ostasevych
Copy link
Author

ostasevych commented Apr 26, 2024

Hi @ostasevych. I'm unable to reproduce this. Can you please share a screencast of what's happening?

Sadly, I cannot share screencasts, please, find the screenshots where I show how ordering (sorting) is not working by the key Last changed:
image
image

image
image

Would that be possible to make the dates shown in native normal format, eg dd.mm.yyyy?

@ostasevych
Copy link
Author

ostasevych commented Apr 30, 2024

The update to v.0.7.1 hasn't solved the issue :(
I am still having trouble with sorting by date in views.

@ostasevych
Copy link
Author

ostasevych commented May 7, 2024

UPD: I suspect that the command occ tables:legacy:transfer:rows --all broken the data format thus the date data are no more sortable. Can anybody check that?

@luka-nextcloud luka-nextcloud added 3. to review Waiting for reviews and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. labels May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working
Projects
Status: ☑️ Done
Development

Successfully merging a pull request may close this issue.

3 participants