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

ContentUtil: orderAfter does not work propertly

XMLWordPrintable

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

      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).

        Acceptance criteria

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

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD