Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-2168

MoveInsidePossibilityPredicate sometimes allows wrong operations.

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • 5.1
    • None
    • content app
    • None

    Description

      Currently Move Action is too permissive... For example in page app it is possible to move pages under other pages without any check.
      These check are done in TemplateTypeRestrictionDropConstraint for pages app.

      MoveDialogPresenter uses that constraint, but the predicate that validates possibility of moving item inside the selected one sometimes grants permission before the constraint is used. It assumes that the destination node is root because the getParent() call for JcrNodeAdapter returns null (see MGNLUI-2169).

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                apchelintcev Aleksandr Pchelintcev
                ehechinger Eric Hechinger
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD