[
https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16967511#comment-16967511
]
Samuel Trégouët commented on OFBIZ-11275:
-----------------------------------------
I have attached patch formatted with `git format-patch`. So patches contain
diff *and* commit messages, as we now use git as primary vcs I hope these kind
of patch will ease commiter work. Let me know if I'm wrong.
> 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
> Reporter: Samuel Trégouët
> Priority: Major
> 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
>
>
> 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)
