Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-3479

ContentUtil: orderAfter does not work propertly

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • 4.3.9, 4.4.2, 4.5
    • 4.3.8, 4.4.1
    • core
    • None
    • Yes
    • Yes

    Description

      The algorithm can't work, it just orders all nodes in front of the 'nodeToMove' until the 'targetNodeName' is in front. But that can't work in most cases (at least when there is more than three nodes as in the existing textcase).

      I created a patch with test cases, all these should work properly (ContentUtilTest.java.patch).
      While testing the orderAfter i tested the orderBefore too.
      The orderBefore works fine, still adding a patch here, so the test cases (no cases there yet) can be added&tested at the same time (ContentUtilTest.java.patch2).

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              ochytil Ondrej Chytil
              cringele Christian Ringele
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD