[jira] [Updated] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml
[ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mathieu Lirzin updated OFBIZ-11275: --- Fix Version/s: (was: Trunk) Upcoming Branch > 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: Upcoming Branch > > 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} > {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)
[jira] [Updated] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml
[ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Samuel Trégouët updated OFBIZ-11275: Attachment: 0004-Fixed-graphSpec-should-be-a-LinkedHashMap-to-preserv.patch > 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} > {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)
[jira] [Updated] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml
[ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mathieu Lirzin updated OFBIZ-11275: --- Component/s: framework > 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 > 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} > {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)
[jira] [Updated] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml
[ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mathieu Lirzin updated OFBIZ-11275: --- Affects Version/s: 18.12.01 Trunk > 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 >Affects Versions: Trunk, 18.12.01 >Reporter: Samuel Trégouët >Assignee: Mathieu Lirzin >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} > {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)
[jira] [Updated] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml
[ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Samuel Trégouët updated OFBIZ-11275: Attachment: 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 > 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} > {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)
[jira] [Updated] (OFBIZ-11275) bug in depends-on in ofbiz-component.xml
[ https://issues.apache.org/jira/browse/OFBIZ-11275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Samuel Trégouët updated OFBIZ-11275: Description: 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} {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 was: 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 ``` ``` > 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 > > 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} > {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)