[MGNLUI-6377] Move action in some apps doesn't open the current path in the chooser Created: 30/Oct/20  Updated: 13/Aug/21  Resolved: 11/Aug/21

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.2.5
Fix Version/s: 6.2.12

Type: Bug Priority: Neutral
Reporter: Monica Damborenea Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: VN-Testing
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File move in 5.7.png     PNG File move in 6.2.png    
Issue Links:
Problem/Incident
relation
is related to MGNLUI-6749 RTE links should select last chosen item 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:
[X]* Steps to reproduce, expected, and actual results filled
[X]* Affected version filled
Sprint: UI FW 34

 Description   

When you click on the "Move" action, the chooser dialog does not open the current page path. In 5.7 this was working as expected and the correct path was opened.

This is happening at least in the Pages and JCR apps.

Not happening in the Story app or in the Configuration app

Steps to reproduce

  1. Open the pages app
  2. Select a page and click on "Move Page"
  3. Notice that the dialog does not take into account the current page path that we selected

 

Expected results

The chooser dialog should open the current path where the selected page is. As in Magnolia 5.7:

 

Actual results

The chooser is opened on the root level, the page's path is not taken into account.

Development notes
This is working as expected in other apps like in the Story app or in the Configuration app.
Check out here the pages-app definition, and in git

      move:
        class: info.magnolia.ui.contentapp.action.MoveActionDefinition
        icon: icon-move
        availability: &notDeletedMultipleAvailability
          writePermissionRequired: true
          multiple: true
          rules:
            notDeleted: *notDeleted

The MoveAction class looks to be the common factor.

The stories app uses another class:

      moveStory:
        class: info.magnolia.ui.contentapp.movedialog.action.OpenMoveDialogActionDefinition
        icon: icon-move
        availability:
          writePermissionRequired: true
          multiple: true
          rules:
            IsNotDeletedRule:
              implementationClass: info.magnolia.ui.framework.availability.IsNotDeletedRule


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