[MGNLUI-6883] Grid is sorted by cells HTML content instead of the visible column value Created: 23/Sep/21  Updated: 26/Oct/21  Resolved: 14/Oct/21

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.2
Fix Version/s: 6.2.13

Type: Bug Priority: Neutral
Reporter: Nguyen Phung Chi Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Relates
relates to MGNLUI-6923 Sorting does not work for items at bo... Open
causality
is causing EXTDAM-150 [S3, Bynder] Sorting by name does not... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[X]* Doc/release notes changes? Comment present?
[X]* Downstream builds green?
[X]* Solution information and context easily available?
[X]* Tests
[X]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: Migrate apps to new UI framework
Sprint: UI FW 38, UI FW 39
Story Points: 3

 Description   

Steps to reproduce

  1. Go to https://demo.magnolia-cms.com/.magnolia/admincentral#app:dam
  2. Switch to list view (easier to reproduce but affects the tree view as well)
  3. Sort by the name column

Expected results

Items are sorted according to the visible asset name.

Actual results

  1. Items are sorted by the HTML content (including thumbnail links).

Workaround

None.

Development notes



 Comments   
Comment by Roman Kovařík [ 07/Oct/21 ]

Since we can't sort by JCR path in the backend, we need to sorting logic in FilterableListDataProvider.
We set custom comparator for column with HtmlRenderer which sorts according to the value retrieved from property set factory (which is a plain text value such as an asset name).

 

Comment by Roman Kovařík [ 11/Oct/21 ]

Reopened: can't sort by activation status.

Generated at Mon Feb 12 09:40:58 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.