[ 
http://jira.magnolia-cms.com/browse/MAGNOLIA-3479?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Philipp Bärfuss updated MAGNOLIA-3479:
--------------------------------------

    Patch included: [Yes]
     Fix Version/s: 4.4.2
                        (was: 4.2.x)
                        (was: 4.1.x)
                        (was: 4.4.x)

> 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.0, 4.0.1, 4.0.2, 4.0.3, 4.1.1, 4.1.2, 4.1.3, 4.2, 
> 4.2.1, 4.2.2, 4.1.4, 4.2.3, 4.3, 4.3.1, 4.2.4, 4.1.6, 4.3.2, 4.3.3, 4.3.4, 
> 4.3.5, 4.3.6, 4.3.7, 4.3.8, 4.4, 4.4.1
>            Reporter: Christian Ringele
>            Assignee: Ondřej Chytil
>            Priority: Major
>             Fix For: 4.3.x, 4.4.2
>
>         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: <[email protected]>
----------------------------------------------------------------

Reply via email to