ContentUtil: orderAfter does not work propertly
-----------------------------------------------
Key: MAGNOLIA-3479
URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-3479
Project: Magnolia
Issue Type: Bug
Components: core
Affects Versions: 4.4.1, 4.4, 4.3.8, 4.3.7, 4.3.6, 4.3.5, 4.3.4, 4.3.3,
4.3.2, 4.1.6, 4.2.4, 4.3.1, 4.3, 4.2.3, 4.1.4, 4.2.2, 4.2.1, 4.2, 4.1.3, 4.1.2,
4.1.1, 4.0.3, 4.0.2, 4.0.1, 4.0
Reporter: Christian Ringele
Assignee: Ondřej Chytil
Priority: Major
Fix For: 4.2.x, 4.1.x, 4.3.x, 4.4.x
Attachments: ContentUtilTest.java.patch, ContentUtilTest.java.patch2
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).
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com>
----------------------------------------------------------------
