[MGNLUI-4306] Grid selection in pulse gets inconsistent when switching between categories Created: 17/Oct/17  Updated: 20/Oct/17  Resolved: 18/Oct/17

Status: Closed
Project: Magnolia UI
Component/s: admincentral
Affects Version/s: 5.6
Fix Version/s: 5.6

Type: Bug Priority: Major
Reporter: Federico Grilli Assignee: Federico Grilli
Resolution: Fixed Votes: 0
Labels: quickwin
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: File pulse selection.m4v    
Issue Links:
causality
caused by TASKMGMT-29 Provide equals and hashCode methods f... Closed
relation
is related to MGNLUI-4292 Update Pulse to Vaadin 8 Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Epic Link: Vaadin 8
Sprint: Basel 118
Story Points: 1

 Description   

See also attached screencast.

To reproduce:

  • go to Pulse and select one message
    • UI will correctly display one item selected
  • switch to Tasks and back to Messages
    • UI will display no selection but pulse footer states that one item is selected
      Try to select more items and the count will be offset

This can obviously be conducive to confusion and unwanted behaviour, e.g. deleting items you don't actually want to delete.

After a quick investigation apchelintcev found out the cause

it happens cause Message class has no equals/hashcode impl
you go back and forth, you reload the Grid, the new beans are queried from the backend => your selection isn't there any longer (why footer isn't updated -> that's the internal logic of the presenter)
just checked the Task class case - same story


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