[MGNLUI-5623] CanMove rule allows an item to move onto itself Created: 22/Jan/20 Updated: 19/Feb/20 Resolved: 10/Feb/20 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | framework |
| Affects Version/s: | 6.2 |
| Fix Version/s: | 6.2 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Michael Duerig | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 5.15h | ||
| Original Estimate: | Not Specified | ||
| 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
|
| Date of First Response: | |
| Sprint: | Declarative REST 15, 6.2 Ramp-up 16 |
| Story Points: | 3 |
| Description |
|
The CanMoveRule under some circumstances allows moving an item onto itself. Also it might not allow a move operation that should be allowed. Steps to reproduce
Additional observationAfter opening any persona for editing and switching back to the browser tab the CanMove rule behaves correctly. Analysis so farThe LocationController injected into the CanMove rule initially tracks a DefaultLocation instance. This instance does not reflect the selected item in the browser. Switching to a different tab injects the ContentBrowserSubApp.BrowserLocation from the SubAppContext, which from there on correctly reflects the selected item. Things triedI tried using LocationContext to track the current location including the selected item in the CanMove rule, however the injected instance differs from that in Browser. At this point I'm not sure whether this is the intended behaviour. WorkaroundOpen a detail tab for any item and switch back to the browser tab. At this point the CanMove rule behaves correctly.
|
| Comments |
| Comment by Dai Ha [ 11/Feb/20 ] |
|
Issue:
Fix:
Verified with latest 6.2 dx-core build. |