[jira] [Commented] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16981520#comment-16981520 ]

Samuel Trégouët commented on OFBIZ-11275:
-----------------------------------------

just added a new patch (0004-Fixed-graphSpec-should-be-a-LinkedHashMap-to-preserv.patch) to fix previous error

> bug in depends-on in ofbiz-component.xml
> ----------------------------------------
>
>                 Key: OFBIZ-11275
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11275
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: Trunk, 18.12.01
>            Reporter: Samuel Trégouët
>            Assignee: Mathieu Lirzin
>            Priority: Major
>             Fix For: Trunk
>
>         Attachments: 0001-Implemented-Show-dependency-resolution-algorithm-pro.patch, 0002-Implemented-Add-a-generic-directed-graph-utilitary-c.patch, 0003-Fixed-Remove-dependency-management-from-ComponentCon.patch, 0004-Fixed-graphSpec-should-be-a-LinkedHashMap-to-preserv.patch
>
>
> when using `depends-on` tag in obfiz-component.xml one should expect ofbiz to load component (and in particular container listed in ofbiz-component.xml) in particular order.
> I mean if component `accounting` has this line in its ofbiz-component.xml
> {noformat}
> <depends-on component="order" />{noformat}
> ofbiz should load order component first and then accounting. This is not the case. Only classpath is modified according to depends-on declaration (and this is not really a usefull/used feature but we'll see this in another issue ;) )
> So here are patches to fix this issue. First one is only a test (which is skipped to allow ./gradlew test to be ok, so if one need to be convinced one should comment Ignore annotation) to illustrate issue, the two others are actual fixes
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)