Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

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

Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

Pierre Smits
In what sense is 'Disables the ebaystore component in ofbiz-component.xml'
related  OFBIZ-9182 and thus leading to have referenced in this commit
description ? Nothing in the description of the JIRA issue details such
requirement. Is this commit wrong, or is the description of the JIRA issue
wrong?

Best regards,




Pierre Smits

ORRTIZ.COM <http://www.orrtiz.com>
OFBiz based solutions & services

OFBiz Extensions Marketplace
http://oem.ofbizci.net/oci-2/

On Sun, Jan 29, 2017 at 2:07 PM, <[hidden email]> wrote:

> Author: taher
> Date: Sun Jan 29 13:07:32 2017
> New Revision: 1780790
>
> URL: http://svn.apache.org/viewvc?rev=1780790&view=rev
> Log:
> Improved: fully refactored the component loading logic in Gradle
> (OFBIZ-9182)
>
> This commit provides the following:
> - Fully refactor common.gradle so that it mirrors the component loading
>   logic found in ComponentContainer. This makes the loading very flexible
>   and changing the directory structure should have no impact on the build
>   scripts.
> - Gradle will now ignore a component if it exists but disabled in
>   ofbiz-component.xml (i.e. enabled="false")
> - Removes /plugins/component-load.xml
> - Disables the ebaystore component in ofbiz-component.xml
>
> The above is needed as a first step toward preparing the project for svn
> restructure
>
> Removed:
>     ofbiz/trunk/plugins/component-load.xml
> Modified:
>     ofbiz/trunk/common.gradle
>     ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
>
> Modified: ofbiz/trunk/common.gradle
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/common.gradle?rev=
> 1780790&r1=1780789&r2=1780790&view=diff
> ============================================================
> ==================
> --- ofbiz/trunk/common.gradle (original)
> +++ ofbiz/trunk/common.gradle Sun Jan 29 13:07:32 2017
> @@ -17,27 +17,37 @@
>   * under the License.
>   */
>  def iterateOverActiveComponents(applyFunction) {
> -    def frameworkComponents = new XmlParser().parse("${rootDir}/
> framework/component-load.xml")
> -    def applicationsComponents = new XmlParser().parse("${rootDir}/
> applications/component-load.xml")
> -    def pluginsComponents = new XmlParser().parse("${rootDir}/
> plugins/component-load.xml")
>
> +    // Start is not a real component, therefore loading it manually
>      applyFunction file("${rootDir}/framework/start")
>
> -    frameworkComponents.children().each { component ->
> -        applyFunction file("${rootDir}/framework/"+component.@"component-
> location")
> -    }
> -    applicationsComponents.children().each { component ->
> -        applyFunction file("${rootDir}/applications/"+component.@
> "component-location")
> -    }
> -    pluginsComponents.children().each { component ->
> -        applyFunction file("${rootDir}/plugins/"+component.@"component-
> location")
> -    }
> +    def rootComponents = new XmlParser().parse("${rootDir}/
> framework/base/config/component-load.xml")
> +    rootComponents.children().each { rootComponent ->
> +        File componentLoadFile = file "${rootDir}/"+ rootComponent.@
> "parent-directory" + "/component-load.xml"
>
> -    file("${rootDir}/themes").eachDir { component ->
> -        applyFunction(component)
> +        if(componentLoadFile.exists()) {
> +            // iterate through the components defined in
> component-load.xml
> +            def parsedComponents = new XmlParser().parse(
> componentLoadFile.toString())
> +            parsedComponents.children().each { component ->
> +                def componentLocation = file "${rootDir}/"+
> rootComponent.@"parent-directory" + '/' + component.@"component-location"
> +                applyIfEnabled(componentLocation, applyFunction)
> +            }
> +        } else {
> +            // iterate through all components (subdirectories of the root
> component)
> +            file(rootComponent.@"parent-directory").eachDir {
> componentLocation ->
> +                applyIfEnabled(componentLocation, applyFunction)
> +            }
> +        }
>      }
> -    file("${rootDir}/hot-deploy").eachDir { component ->
> -        applyFunction(component)
> +}
> +
> +def applyIfEnabled(componentDir, applyFunction) {
> +    File componentFile = file componentDir.toString() +
> '/ofbiz-component.xml'
> +    if(componentFile.exists()) {
> +        def parsedComponent = new XmlParser().parse(
> componentFile.toString())
> +        if(parsedComponent.@enabled == null || parsedComponent.@enabled
> == "true") {
> +            applyFunction componentDir
> +        }
>      }
>  }
>
>
> Modified: ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/plugins/ebaystore/
> ofbiz-component.xml?rev=1780790&r1=1780789&r2=1780790&view=diff
> ============================================================
> ==================
> --- ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml (original)
> +++ ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml Sun Jan 29 13:07:32
> 2017
> @@ -1,48 +1,48 @@
> -<?xml version="1.0" encoding="UTF-8"?>
> -<!--
> -Licensed to the Apache Software Foundation (ASF) under one
> -or more contributor license agreements.  See the NOTICE file
> -distributed with this work for additional information
> -regarding copyright ownership.  The ASF licenses this file
> -to you under the Apache License, Version 2.0 (the
> -"License"); you may not use this file except in compliance
> -with the License.  You may obtain a copy of the License at
> -
> -http://www.apache.org/licenses/LICENSE-2.0
> -
> -Unless required by applicable law or agreed to in writing,
> -software distributed under the License is distributed on an
> -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> -KIND, either express or implied.  See the License for the
> -specific language governing permissions and limitations
> -under the License.
> --->
> -
> -<ofbiz-component name="ebaystore" enabled="true"
> -        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> -        xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/
> ofbiz-component.xsd">
> -    <resource-loader name="main" type="component"/>
> -    <classpath type="dir" location="config"/>
> -
> -    <entity-resource type="model" reader-name="main" loader="main"
> location="entitydef/entitymodel.xml"/>
> -
> -    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> -    <entity-resource type="data" reader-name="demo" loader="main"
> location="data/EbayStoreSecurityGroupDemoData.xml"/>
> -    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStoreTypeData.xml"/>
> -    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStoreHelpData.xml"/>
> -    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStorePortletData.xml"/>
> -    <entity-resource type="data" reader-name="demo" loader="main"
> location="data/DemoEbayStoreData.xml"/>
> -
> -    <service-resource type="eca" loader="main" location="servicedef/secas.
> xml"/>
> -    <service-resource type="model" loader="main"
> location="servicedef/services.xml"/>
> -    <service-resource type="model" loader="main"
> location="servicedef/services_store.xml"/>
> -
> -    <webapp name="ebaystore"
> -        title="eBay Store"
> -        menu-name="secondary"
> -        server="default-server"
> -        location="webapp/ebaystore"
> -        base-permission="EBAYSTORE"
> -        mount-point="/ebaystore"
> -        app-bar-display="true"/>
> -</ofbiz-component>
> +<?xml version="1.0" encoding="UTF-8"?>
> +<!--
> +Licensed to the Apache Software Foundation (ASF) under one
> +or more contributor license agreements.  See the NOTICE file
> +distributed with this work for additional information
> +regarding copyright ownership.  The ASF licenses this file
> +to you under the Apache License, Version 2.0 (the
> +"License"); you may not use this file except in compliance
> +with the License.  You may obtain a copy of the License at
> +
> +http://www.apache.org/licenses/LICENSE-2.0
> +
> +Unless required by applicable law or agreed to in writing,
> +software distributed under the License is distributed on an
> +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> +KIND, either express or implied.  See the License for the
> +specific language governing permissions and limitations
> +under the License.
> +-->
> +
> +<ofbiz-component name="ebaystore" enabled="false"
> +        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> +        xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/
> ofbiz-component.xsd">
> +    <resource-loader name="main" type="component"/>
> +    <classpath type="dir" location="config"/>
> +
> +    <entity-resource type="model" reader-name="main" loader="main"
> location="entitydef/entitymodel.xml"/>
> +
> +    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> +    <entity-resource type="data" reader-name="demo" loader="main"
> location="data/EbayStoreSecurityGroupDemoData.xml"/>
> +    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStoreTypeData.xml"/>
> +    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStoreHelpData.xml"/>
> +    <entity-resource type="data" reader-name="seed" loader="main"
> location="data/EbayStorePortletData.xml"/>
> +    <entity-resource type="data" reader-name="demo" loader="main"
> location="data/DemoEbayStoreData.xml"/>
> +
> +    <service-resource type="eca" loader="main" location="servicedef/secas.
> xml"/>
> +    <service-resource type="model" loader="main"
> location="servicedef/services.xml"/>
> +    <service-resource type="model" loader="main"
> location="servicedef/services_store.xml"/>
> +
> +    <webapp name="ebaystore"
> +        title="eBay Store"
> +        menu-name="secondary"
> +        server="default-server"
> +        location="webapp/ebaystore"
> +        base-permission="EBAYSTORE"
> +        mount-point="/ebaystore"
> +        app-bar-display="true"/>
> +</ofbiz-component>
>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

taher
The ebaystore component was commented out in component-load.xml. Since this
file is removed we need another mechanism to disable the component,
therefore the build scripts now ignore components that are disabled in
ofbiz-component.xml. This is not only a workaround but a cleaner solution
that is consistent with the way components are loaded from the framework.

On Jan 29, 2017 4:43 PM, "Pierre Smits" <[hidden email]> wrote:

> In what sense is 'Disables the ebaystore component in ofbiz-component.xml'
> related  OFBIZ-9182 and thus leading to have referenced in this commit
> description ? Nothing in the description of the JIRA issue details such
> requirement. Is this commit wrong, or is the description of the JIRA issue
> wrong?
>
> Best regards,
>
>
>
>
> Pierre Smits
>
> ORRTIZ.COM <http://www.orrtiz.com>
> OFBiz based solutions & services
>
> OFBiz Extensions Marketplace
> http://oem.ofbizci.net/oci-2/
>
> On Sun, Jan 29, 2017 at 2:07 PM, <[hidden email]> wrote:
>
> > Author: taher
> > Date: Sun Jan 29 13:07:32 2017
> > New Revision: 1780790
> >
> > URL: http://svn.apache.org/viewvc?rev=1780790&view=rev
> > Log:
> > Improved: fully refactored the component loading logic in Gradle
> > (OFBIZ-9182)
> >
> > This commit provides the following:
> > - Fully refactor common.gradle so that it mirrors the component loading
> >   logic found in ComponentContainer. This makes the loading very flexible
> >   and changing the directory structure should have no impact on the build
> >   scripts.
> > - Gradle will now ignore a component if it exists but disabled in
> >   ofbiz-component.xml (i.e. enabled="false")
> > - Removes /plugins/component-load.xml
> > - Disables the ebaystore component in ofbiz-component.xml
> >
> > The above is needed as a first step toward preparing the project for svn
> > restructure
> >
> > Removed:
> >     ofbiz/trunk/plugins/component-load.xml
> > Modified:
> >     ofbiz/trunk/common.gradle
> >     ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> >
> > Modified: ofbiz/trunk/common.gradle
> > URL: http://svn.apache.org/viewvc/ofbiz/trunk/common.gradle?rev=
> > 1780790&r1=1780789&r2=1780790&view=diff
> > ============================================================
> > ==================
> > --- ofbiz/trunk/common.gradle (original)
> > +++ ofbiz/trunk/common.gradle Sun Jan 29 13:07:32 2017
> > @@ -17,27 +17,37 @@
> >   * under the License.
> >   */
> >  def iterateOverActiveComponents(applyFunction) {
> > -    def frameworkComponents = new XmlParser().parse("${rootDir}/
> > framework/component-load.xml")
> > -    def applicationsComponents = new XmlParser().parse("${rootDir}/
> > applications/component-load.xml")
> > -    def pluginsComponents = new XmlParser().parse("${rootDir}/
> > plugins/component-load.xml")
> >
> > +    // Start is not a real component, therefore loading it manually
> >      applyFunction file("${rootDir}/framework/start")
> >
> > -    frameworkComponents.children().each { component ->
> > -        applyFunction file("${rootDir}/framework/"+component.@
> "component-
> > location")
> > -    }
> > -    applicationsComponents.children().each { component ->
> > -        applyFunction file("${rootDir}/applications/"+component.@
> > "component-location")
> > -    }
> > -    pluginsComponents.children().each { component ->
> > -        applyFunction file("${rootDir}/plugins/"+component.@"component-
> > location")
> > -    }
> > +    def rootComponents = new XmlParser().parse("${rootDir}/
> > framework/base/config/component-load.xml")
> > +    rootComponents.children().each { rootComponent ->
> > +        File componentLoadFile = file "${rootDir}/"+ rootComponent.@
> > "parent-directory" + "/component-load.xml"
> >
> > -    file("${rootDir}/themes").eachDir { component ->
> > -        applyFunction(component)
> > +        if(componentLoadFile.exists()) {
> > +            // iterate through the components defined in
> > component-load.xml
> > +            def parsedComponents = new XmlParser().parse(
> > componentLoadFile.toString())
> > +            parsedComponents.children().each { component ->
> > +                def componentLocation = file "${rootDir}/"+
> > rootComponent.@"parent-directory" + '/' + component.@"component-
> location"
> > +                applyIfEnabled(componentLocation, applyFunction)
> > +            }
> > +        } else {
> > +            // iterate through all components (subdirectories of the
> root
> > component)
> > +            file(rootComponent.@"parent-directory").eachDir {
> > componentLocation ->
> > +                applyIfEnabled(componentLocation, applyFunction)
> > +            }
> > +        }
> >      }
> > -    file("${rootDir}/hot-deploy").eachDir { component ->
> > -        applyFunction(component)
> > +}
> > +
> > +def applyIfEnabled(componentDir, applyFunction) {
> > +    File componentFile = file componentDir.toString() +
> > '/ofbiz-component.xml'
> > +    if(componentFile.exists()) {
> > +        def parsedComponent = new XmlParser().parse(
> > componentFile.toString())
> > +        if(parsedComponent.@enabled == null || parsedComponent.@enabled
> > == "true") {
> > +            applyFunction componentDir
> > +        }
> >      }
> >  }
> >
> >
> > Modified: ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > URL: http://svn.apache.org/viewvc/ofbiz/trunk/plugins/ebaystore/
> > ofbiz-component.xml?rev=1780790&r1=1780789&r2=1780790&view=diff
> > ============================================================
> > ==================
> > --- ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml (original)
> > +++ ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml Sun Jan 29
> 13:07:32
> > 2017
> > @@ -1,48 +1,48 @@
> > -<?xml version="1.0" encoding="UTF-8"?>
> > -<!--
> > -Licensed to the Apache Software Foundation (ASF) under one
> > -or more contributor license agreements.  See the NOTICE file
> > -distributed with this work for additional information
> > -regarding copyright ownership.  The ASF licenses this file
> > -to you under the Apache License, Version 2.0 (the
> > -"License"); you may not use this file except in compliance
> > -with the License.  You may obtain a copy of the License at
> > -
> > -http://www.apache.org/licenses/LICENSE-2.0
> > -
> > -Unless required by applicable law or agreed to in writing,
> > -software distributed under the License is distributed on an
> > -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > -KIND, either express or implied.  See the License for the
> > -specific language governing permissions and limitations
> > -under the License.
> > --->
> > -
> > -<ofbiz-component name="ebaystore" enabled="true"
> > -        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > -        xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/
> > ofbiz-component.xsd">
> > -    <resource-loader name="main" type="component"/>
> > -    <classpath type="dir" location="config"/>
> > -
> > -    <entity-resource type="model" reader-name="main" loader="main"
> > location="entitydef/entitymodel.xml"/>
> > -
> > -    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > -    <entity-resource type="data" reader-name="demo" loader="main"
> > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > -    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStoreTypeData.xml"/>
> > -    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStoreHelpData.xml"/>
> > -    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStorePortletData.xml"/>
> > -    <entity-resource type="data" reader-name="demo" loader="main"
> > location="data/DemoEbayStoreData.xml"/>
> > -
> > -    <service-resource type="eca" loader="main"
> location="servicedef/secas.
> > xml"/>
> > -    <service-resource type="model" loader="main"
> > location="servicedef/services.xml"/>
> > -    <service-resource type="model" loader="main"
> > location="servicedef/services_store.xml"/>
> > -
> > -    <webapp name="ebaystore"
> > -        title="eBay Store"
> > -        menu-name="secondary"
> > -        server="default-server"
> > -        location="webapp/ebaystore"
> > -        base-permission="EBAYSTORE"
> > -        mount-point="/ebaystore"
> > -        app-bar-display="true"/>
> > -</ofbiz-component>
> > +<?xml version="1.0" encoding="UTF-8"?>
> > +<!--
> > +Licensed to the Apache Software Foundation (ASF) under one
> > +or more contributor license agreements.  See the NOTICE file
> > +distributed with this work for additional information
> > +regarding copyright ownership.  The ASF licenses this file
> > +to you under the Apache License, Version 2.0 (the
> > +"License"); you may not use this file except in compliance
> > +with the License.  You may obtain a copy of the License at
> > +
> > +http://www.apache.org/licenses/LICENSE-2.0
> > +
> > +Unless required by applicable law or agreed to in writing,
> > +software distributed under the License is distributed on an
> > +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > +KIND, either express or implied.  See the License for the
> > +specific language governing permissions and limitations
> > +under the License.
> > +-->
> > +
> > +<ofbiz-component name="ebaystore" enabled="false"
> > +        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > +        xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/
> > ofbiz-component.xsd">
> > +    <resource-loader name="main" type="component"/>
> > +    <classpath type="dir" location="config"/>
> > +
> > +    <entity-resource type="model" reader-name="main" loader="main"
> > location="entitydef/entitymodel.xml"/>
> > +
> > +    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > +    <entity-resource type="data" reader-name="demo" loader="main"
> > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > +    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStoreTypeData.xml"/>
> > +    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStoreHelpData.xml"/>
> > +    <entity-resource type="data" reader-name="seed" loader="main"
> > location="data/EbayStorePortletData.xml"/>
> > +    <entity-resource type="data" reader-name="demo" loader="main"
> > location="data/DemoEbayStoreData.xml"/>
> > +
> > +    <service-resource type="eca" loader="main"
> location="servicedef/secas.
> > xml"/>
> > +    <service-resource type="model" loader="main"
> > location="servicedef/services.xml"/>
> > +    <service-resource type="model" loader="main"
> > location="servicedef/services_store.xml"/>
> > +
> > +    <webapp name="ebaystore"
> > +        title="eBay Store"
> > +        menu-name="secondary"
> > +        server="default-server"
> > +        location="webapp/ebaystore"
> > +        base-permission="EBAYSTORE"
> > +        mount-point="/ebaystore"
> > +        app-bar-display="true"/>
> > +</ofbiz-component>
> >
> >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

Pierre Smits
I know that the component was commented out in component-load.xml. It has
been done so with rev
http://svn.apache.org/viewvc?view=revision&sortby=rev&revision=1759583

Based on the commit comment, due to some unforeseen difficulty with
required 3rd party code libraries...

Apparently we are not addressing the underlying issue, but keep pilling
workaround on workaround.
Would it not be better to rethink this?



Pierre Smits

ORRTIZ.COM <http://www.orrtiz.com>
OFBiz based solutions & services

OFBiz Extensions Marketplace
http://oem.ofbizci.net/oci-2/

On Sun, Jan 29, 2017 at 2:49 PM, Taher Alkhateeb <[hidden email]
> wrote:

> The ebaystore component was commented out in component-load.xml. Since this
> file is removed we need another mechanism to disable the component,
> therefore the build scripts now ignore components that are disabled in
> ofbiz-component.xml. This is not only a workaround but a cleaner solution
> that is consistent with the way components are loaded from the framework.
>
> On Jan 29, 2017 4:43 PM, "Pierre Smits" <[hidden email]> wrote:
>
> > In what sense is 'Disables the ebaystore component in
> ofbiz-component.xml'
> > related  OFBIZ-9182 and thus leading to have referenced in this commit
> > description ? Nothing in the description of the JIRA issue details such
> > requirement. Is this commit wrong, or is the description of the JIRA
> issue
> > wrong?
> >
> > Best regards,
> >
> >
> >
> >
> > Pierre Smits
> >
> > ORRTIZ.COM <http://www.orrtiz.com>
> > OFBiz based solutions & services
> >
> > OFBiz Extensions Marketplace
> > http://oem.ofbizci.net/oci-2/
> >
> > On Sun, Jan 29, 2017 at 2:07 PM, <[hidden email]> wrote:
> >
> > > Author: taher
> > > Date: Sun Jan 29 13:07:32 2017
> > > New Revision: 1780790
> > >
> > > URL: http://svn.apache.org/viewvc?rev=1780790&view=rev
> > > Log:
> > > Improved: fully refactored the component loading logic in Gradle
> > > (OFBIZ-9182)
> > >
> > > This commit provides the following:
> > > - Fully refactor common.gradle so that it mirrors the component loading
> > >   logic found in ComponentContainer. This makes the loading very
> flexible
> > >   and changing the directory structure should have no impact on the
> build
> > >   scripts.
> > > - Gradle will now ignore a component if it exists but disabled in
> > >   ofbiz-component.xml (i.e. enabled="false")
> > > - Removes /plugins/component-load.xml
> > > - Disables the ebaystore component in ofbiz-component.xml
> > >
> > > The above is needed as a first step toward preparing the project for
> svn
> > > restructure
> > >
> > > Removed:
> > >     ofbiz/trunk/plugins/component-load.xml
> > > Modified:
> > >     ofbiz/trunk/common.gradle
> > >     ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > >
> > > Modified: ofbiz/trunk/common.gradle
> > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/common.gradle?rev=
> > > 1780790&r1=1780789&r2=1780790&view=diff
> > > ============================================================
> > > ==================
> > > --- ofbiz/trunk/common.gradle (original)
> > > +++ ofbiz/trunk/common.gradle Sun Jan 29 13:07:32 2017
> > > @@ -17,27 +17,37 @@
> > >   * under the License.
> > >   */
> > >  def iterateOverActiveComponents(applyFunction) {
> > > -    def frameworkComponents = new XmlParser().parse("${rootDir}/
> > > framework/component-load.xml")
> > > -    def applicationsComponents = new XmlParser().parse("${rootDir}/
> > > applications/component-load.xml")
> > > -    def pluginsComponents = new XmlParser().parse("${rootDir}/
> > > plugins/component-load.xml")
> > >
> > > +    // Start is not a real component, therefore loading it manually
> > >      applyFunction file("${rootDir}/framework/start")
> > >
> > > -    frameworkComponents.children().each { component ->
> > > -        applyFunction file("${rootDir}/framework/"+component.@
> > "component-
> > > location")
> > > -    }
> > > -    applicationsComponents.children().each { component ->
> > > -        applyFunction file("${rootDir}/applications/"+component.@
> > > "component-location")
> > > -    }
> > > -    pluginsComponents.children().each { component ->
> > > -        applyFunction file("${rootDir}/plugins/"+component.@
> "component-
> > > location")
> > > -    }
> > > +    def rootComponents = new XmlParser().parse("${rootDir}/
> > > framework/base/config/component-load.xml")
> > > +    rootComponents.children().each { rootComponent ->
> > > +        File componentLoadFile = file "${rootDir}/"+ rootComponent.@
> > > "parent-directory" + "/component-load.xml"
> > >
> > > -    file("${rootDir}/themes").eachDir { component ->
> > > -        applyFunction(component)
> > > +        if(componentLoadFile.exists()) {
> > > +            // iterate through the components defined in
> > > component-load.xml
> > > +            def parsedComponents = new XmlParser().parse(
> > > componentLoadFile.toString())
> > > +            parsedComponents.children().each { component ->
> > > +                def componentLocation = file "${rootDir}/"+
> > > rootComponent.@"parent-directory" + '/' + component.@"component-
> > location"
> > > +                applyIfEnabled(componentLocation, applyFunction)
> > > +            }
> > > +        } else {
> > > +            // iterate through all components (subdirectories of the
> > root
> > > component)
> > > +            file(rootComponent.@"parent-directory").eachDir {
> > > componentLocation ->
> > > +                applyIfEnabled(componentLocation, applyFunction)
> > > +            }
> > > +        }
> > >      }
> > > -    file("${rootDir}/hot-deploy").eachDir { component ->
> > > -        applyFunction(component)
> > > +}
> > > +
> > > +def applyIfEnabled(componentDir, applyFunction) {
> > > +    File componentFile = file componentDir.toString() +
> > > '/ofbiz-component.xml'
> > > +    if(componentFile.exists()) {
> > > +        def parsedComponent = new XmlParser().parse(
> > > componentFile.toString())
> > > +        if(parsedComponent.@enabled == null ||
> parsedComponent.@enabled
> > > == "true") {
> > > +            applyFunction componentDir
> > > +        }
> > >      }
> > >  }
> > >
> > >
> > > Modified: ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/plugins/ebaystore/
> > > ofbiz-component.xml?rev=1780790&r1=1780789&r2=1780790&view=diff
> > > ============================================================
> > > ==================
> > > --- ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml (original)
> > > +++ ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml Sun Jan 29
> > 13:07:32
> > > 2017
> > > @@ -1,48 +1,48 @@
> > > -<?xml version="1.0" encoding="UTF-8"?>
> > > -<!--
> > > -Licensed to the Apache Software Foundation (ASF) under one
> > > -or more contributor license agreements.  See the NOTICE file
> > > -distributed with this work for additional information
> > > -regarding copyright ownership.  The ASF licenses this file
> > > -to you under the Apache License, Version 2.0 (the
> > > -"License"); you may not use this file except in compliance
> > > -with the License.  You may obtain a copy of the License at
> > > -
> > > -http://www.apache.org/licenses/LICENSE-2.0
> > > -
> > > -Unless required by applicable law or agreed to in writing,
> > > -software distributed under the License is distributed on an
> > > -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > -KIND, either express or implied.  See the License for the
> > > -specific language governing permissions and limitations
> > > -under the License.
> > > --->
> > > -
> > > -<ofbiz-component name="ebaystore" enabled="true"
> > > -        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > -        xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/
> > > ofbiz-component.xsd">
> > > -    <resource-loader name="main" type="component"/>
> > > -    <classpath type="dir" location="config"/>
> > > -
> > > -    <entity-resource type="model" reader-name="main" loader="main"
> > > location="entitydef/entitymodel.xml"/>
> > > -
> > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > -    <entity-resource type="data" reader-name="demo" loader="main"
> > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStoreTypeData.xml"/>
> > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStoreHelpData.xml"/>
> > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStorePortletData.xml"/>
> > > -    <entity-resource type="data" reader-name="demo" loader="main"
> > > location="data/DemoEbayStoreData.xml"/>
> > > -
> > > -    <service-resource type="eca" loader="main"
> > location="servicedef/secas.
> > > xml"/>
> > > -    <service-resource type="model" loader="main"
> > > location="servicedef/services.xml"/>
> > > -    <service-resource type="model" loader="main"
> > > location="servicedef/services_store.xml"/>
> > > -
> > > -    <webapp name="ebaystore"
> > > -        title="eBay Store"
> > > -        menu-name="secondary"
> > > -        server="default-server"
> > > -        location="webapp/ebaystore"
> > > -        base-permission="EBAYSTORE"
> > > -        mount-point="/ebaystore"
> > > -        app-bar-display="true"/>
> > > -</ofbiz-component>
> > > +<?xml version="1.0" encoding="UTF-8"?>
> > > +<!--
> > > +Licensed to the Apache Software Foundation (ASF) under one
> > > +or more contributor license agreements.  See the NOTICE file
> > > +distributed with this work for additional information
> > > +regarding copyright ownership.  The ASF licenses this file
> > > +to you under the Apache License, Version 2.0 (the
> > > +"License"); you may not use this file except in compliance
> > > +with the License.  You may obtain a copy of the License at
> > > +
> > > +http://www.apache.org/licenses/LICENSE-2.0
> > > +
> > > +Unless required by applicable law or agreed to in writing,
> > > +software distributed under the License is distributed on an
> > > +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > +KIND, either express or implied.  See the License for the
> > > +specific language governing permissions and limitations
> > > +under the License.
> > > +-->
> > > +
> > > +<ofbiz-component name="ebaystore" enabled="false"
> > > +        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > +        xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/
> > > ofbiz-component.xsd">
> > > +    <resource-loader name="main" type="component"/>
> > > +    <classpath type="dir" location="config"/>
> > > +
> > > +    <entity-resource type="model" reader-name="main" loader="main"
> > > location="entitydef/entitymodel.xml"/>
> > > +
> > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > +    <entity-resource type="data" reader-name="demo" loader="main"
> > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStoreTypeData.xml"/>
> > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStoreHelpData.xml"/>
> > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > location="data/EbayStorePortletData.xml"/>
> > > +    <entity-resource type="data" reader-name="demo" loader="main"
> > > location="data/DemoEbayStoreData.xml"/>
> > > +
> > > +    <service-resource type="eca" loader="main"
> > location="servicedef/secas.
> > > xml"/>
> > > +    <service-resource type="model" loader="main"
> > > location="servicedef/services.xml"/>
> > > +    <service-resource type="model" loader="main"
> > > location="servicedef/services_store.xml"/>
> > > +
> > > +    <webapp name="ebaystore"
> > > +        title="eBay Store"
> > > +        menu-name="secondary"
> > > +        server="default-server"
> > > +        location="webapp/ebaystore"
> > > +        base-permission="EBAYSTORE"
> > > +        mount-point="/ebaystore"
> > > +        app-bar-display="true"/>
> > > +</ofbiz-component>
> > >
> > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

Scott Gray-3
Hi Pierre,

It was disabled and it is still disabled, the only change relevant to
ebaystore is the means of disablement.  I'm not sure what needs to be
re-thought?  Eventually someone will decide that addressing the 3rd party
library issues in ebaystore is an itch they want to scratch, but until then
the status quo has been maintained so I don't see an issue.

Regards
Scott



On 30 January 2017 at 03:46, Pierre Smits <[hidden email]> wrote:

> I know that the component was commented out in component-load.xml. It has
> been done so with rev
> http://svn.apache.org/viewvc?view=revision&sortby=rev&revision=1759583
>
> Based on the commit comment, due to some unforeseen difficulty with
> required 3rd party code libraries...
>
> Apparently we are not addressing the underlying issue, but keep pilling
> workaround on workaround.
> Would it not be better to rethink this?
>
>
>
> Pierre Smits
>
> ORRTIZ.COM <http://www.orrtiz.com>
> OFBiz based solutions & services
>
> OFBiz Extensions Marketplace
> http://oem.ofbizci.net/oci-2/
>
> On Sun, Jan 29, 2017 at 2:49 PM, Taher Alkhateeb <
> [hidden email]
> > wrote:
>
> > The ebaystore component was commented out in component-load.xml. Since
> this
> > file is removed we need another mechanism to disable the component,
> > therefore the build scripts now ignore components that are disabled in
> > ofbiz-component.xml. This is not only a workaround but a cleaner solution
> > that is consistent with the way components are loaded from the framework.
> >
> > On Jan 29, 2017 4:43 PM, "Pierre Smits" <[hidden email]> wrote:
> >
> > > In what sense is 'Disables the ebaystore component in
> > ofbiz-component.xml'
> > > related  OFBIZ-9182 and thus leading to have referenced in this commit
> > > description ? Nothing in the description of the JIRA issue details such
> > > requirement. Is this commit wrong, or is the description of the JIRA
> > issue
> > > wrong?
> > >
> > > Best regards,
> > >
> > >
> > >
> > >
> > > Pierre Smits
> > >
> > > ORRTIZ.COM <http://www.orrtiz.com>
> > > OFBiz based solutions & services
> > >
> > > OFBiz Extensions Marketplace
> > > http://oem.ofbizci.net/oci-2/
> > >
> > > On Sun, Jan 29, 2017 at 2:07 PM, <[hidden email]> wrote:
> > >
> > > > Author: taher
> > > > Date: Sun Jan 29 13:07:32 2017
> > > > New Revision: 1780790
> > > >
> > > > URL: http://svn.apache.org/viewvc?rev=1780790&view=rev
> > > > Log:
> > > > Improved: fully refactored the component loading logic in Gradle
> > > > (OFBIZ-9182)
> > > >
> > > > This commit provides the following:
> > > > - Fully refactor common.gradle so that it mirrors the component
> loading
> > > >   logic found in ComponentContainer. This makes the loading very
> > flexible
> > > >   and changing the directory structure should have no impact on the
> > build
> > > >   scripts.
> > > > - Gradle will now ignore a component if it exists but disabled in
> > > >   ofbiz-component.xml (i.e. enabled="false")
> > > > - Removes /plugins/component-load.xml
> > > > - Disables the ebaystore component in ofbiz-component.xml
> > > >
> > > > The above is needed as a first step toward preparing the project for
> > svn
> > > > restructure
> > > >
> > > > Removed:
> > > >     ofbiz/trunk/plugins/component-load.xml
> > > > Modified:
> > > >     ofbiz/trunk/common.gradle
> > > >     ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > >
> > > > Modified: ofbiz/trunk/common.gradle
> > > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/common.gradle?rev=
> > > > 1780790&r1=1780789&r2=1780790&view=diff
> > > > ============================================================
> > > > ==================
> > > > --- ofbiz/trunk/common.gradle (original)
> > > > +++ ofbiz/trunk/common.gradle Sun Jan 29 13:07:32 2017
> > > > @@ -17,27 +17,37 @@
> > > >   * under the License.
> > > >   */
> > > >  def iterateOverActiveComponents(applyFunction) {
> > > > -    def frameworkComponents = new XmlParser().parse("${rootDir}/
> > > > framework/component-load.xml")
> > > > -    def applicationsComponents = new XmlParser().parse("${rootDir}/
> > > > applications/component-load.xml")
> > > > -    def pluginsComponents = new XmlParser().parse("${rootDir}/
> > > > plugins/component-load.xml")
> > > >
> > > > +    // Start is not a real component, therefore loading it manually
> > > >      applyFunction file("${rootDir}/framework/start")
> > > >
> > > > -    frameworkComponents.children().each { component ->
> > > > -        applyFunction file("${rootDir}/framework/"+component.@
> > > "component-
> > > > location")
> > > > -    }
> > > > -    applicationsComponents.children().each { component ->
> > > > -        applyFunction file("${rootDir}/applications/"+component.@
> > > > "component-location")
> > > > -    }
> > > > -    pluginsComponents.children().each { component ->
> > > > -        applyFunction file("${rootDir}/plugins/"+component.@
> > "component-
> > > > location")
> > > > -    }
> > > > +    def rootComponents = new XmlParser().parse("${rootDir}/
> > > > framework/base/config/component-load.xml")
> > > > +    rootComponents.children().each { rootComponent ->
> > > > +        File componentLoadFile = file "${rootDir}/"+ rootComponent.@
> > > > "parent-directory" + "/component-load.xml"
> > > >
> > > > -    file("${rootDir}/themes").eachDir { component ->
> > > > -        applyFunction(component)
> > > > +        if(componentLoadFile.exists()) {
> > > > +            // iterate through the components defined in
> > > > component-load.xml
> > > > +            def parsedComponents = new XmlParser().parse(
> > > > componentLoadFile.toString())
> > > > +            parsedComponents.children().each { component ->
> > > > +                def componentLocation = file "${rootDir}/"+
> > > > rootComponent.@"parent-directory" + '/' + component.@"component-
> > > location"
> > > > +                applyIfEnabled(componentLocation, applyFunction)
> > > > +            }
> > > > +        } else {
> > > > +            // iterate through all components (subdirectories of the
> > > root
> > > > component)
> > > > +            file(rootComponent.@"parent-directory").eachDir {
> > > > componentLocation ->
> > > > +                applyIfEnabled(componentLocation, applyFunction)
> > > > +            }
> > > > +        }
> > > >      }
> > > > -    file("${rootDir}/hot-deploy").eachDir { component ->
> > > > -        applyFunction(component)
> > > > +}
> > > > +
> > > > +def applyIfEnabled(componentDir, applyFunction) {
> > > > +    File componentFile = file componentDir.toString() +
> > > > '/ofbiz-component.xml'
> > > > +    if(componentFile.exists()) {
> > > > +        def parsedComponent = new XmlParser().parse(
> > > > componentFile.toString())
> > > > +        if(parsedComponent.@enabled == null ||
> > parsedComponent.@enabled
> > > > == "true") {
> > > > +            applyFunction componentDir
> > > > +        }
> > > >      }
> > > >  }
> > > >
> > > >
> > > > Modified: ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/plugins/ebaystore/
> > > > ofbiz-component.xml?rev=1780790&r1=1780789&r2=1780790&view=diff
> > > > ============================================================
> > > > ==================
> > > > --- ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml (original)
> > > > +++ ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml Sun Jan 29
> > > 13:07:32
> > > > 2017
> > > > @@ -1,48 +1,48 @@
> > > > -<?xml version="1.0" encoding="UTF-8"?>
> > > > -<!--
> > > > -Licensed to the Apache Software Foundation (ASF) under one
> > > > -or more contributor license agreements.  See the NOTICE file
> > > > -distributed with this work for additional information
> > > > -regarding copyright ownership.  The ASF licenses this file
> > > > -to you under the Apache License, Version 2.0 (the
> > > > -"License"); you may not use this file except in compliance
> > > > -with the License.  You may obtain a copy of the License at
> > > > -
> > > > -http://www.apache.org/licenses/LICENSE-2.0
> > > > -
> > > > -Unless required by applicable law or agreed to in writing,
> > > > -software distributed under the License is distributed on an
> > > > -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > -KIND, either express or implied.  See the License for the
> > > > -specific language governing permissions and limitations
> > > > -under the License.
> > > > --->
> > > > -
> > > > -<ofbiz-component name="ebaystore" enabled="true"
> > > > -        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > > -        xsi:noNamespaceSchemaLocation="
> http://ofbiz.apache.org/dtds/
> > > > ofbiz-component.xsd">
> > > > -    <resource-loader name="main" type="component"/>
> > > > -    <classpath type="dir" location="config"/>
> > > > -
> > > > -    <entity-resource type="model" reader-name="main" loader="main"
> > > > location="entitydef/entitymodel.xml"/>
> > > > -
> > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > > -    <entity-resource type="data" reader-name="demo" loader="main"
> > > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStoreTypeData.xml"/>
> > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStoreHelpData.xml"/>
> > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStorePortletData.xml"/>
> > > > -    <entity-resource type="data" reader-name="demo" loader="main"
> > > > location="data/DemoEbayStoreData.xml"/>
> > > > -
> > > > -    <service-resource type="eca" loader="main"
> > > location="servicedef/secas.
> > > > xml"/>
> > > > -    <service-resource type="model" loader="main"
> > > > location="servicedef/services.xml"/>
> > > > -    <service-resource type="model" loader="main"
> > > > location="servicedef/services_store.xml"/>
> > > > -
> > > > -    <webapp name="ebaystore"
> > > > -        title="eBay Store"
> > > > -        menu-name="secondary"
> > > > -        server="default-server"
> > > > -        location="webapp/ebaystore"
> > > > -        base-permission="EBAYSTORE"
> > > > -        mount-point="/ebaystore"
> > > > -        app-bar-display="true"/>
> > > > -</ofbiz-component>
> > > > +<?xml version="1.0" encoding="UTF-8"?>
> > > > +<!--
> > > > +Licensed to the Apache Software Foundation (ASF) under one
> > > > +or more contributor license agreements.  See the NOTICE file
> > > > +distributed with this work for additional information
> > > > +regarding copyright ownership.  The ASF licenses this file
> > > > +to you under the Apache License, Version 2.0 (the
> > > > +"License"); you may not use this file except in compliance
> > > > +with the License.  You may obtain a copy of the License at
> > > > +
> > > > +http://www.apache.org/licenses/LICENSE-2.0
> > > > +
> > > > +Unless required by applicable law or agreed to in writing,
> > > > +software distributed under the License is distributed on an
> > > > +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > +KIND, either express or implied.  See the License for the
> > > > +specific language governing permissions and limitations
> > > > +under the License.
> > > > +-->
> > > > +
> > > > +<ofbiz-component name="ebaystore" enabled="false"
> > > > +        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > > +        xsi:noNamespaceSchemaLocation="
> http://ofbiz.apache.org/dtds/
> > > > ofbiz-component.xsd">
> > > > +    <resource-loader name="main" type="component"/>
> > > > +    <classpath type="dir" location="config"/>
> > > > +
> > > > +    <entity-resource type="model" reader-name="main" loader="main"
> > > > location="entitydef/entitymodel.xml"/>
> > > > +
> > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > > +    <entity-resource type="data" reader-name="demo" loader="main"
> > > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStoreTypeData.xml"/>
> > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStoreHelpData.xml"/>
> > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > location="data/EbayStorePortletData.xml"/>
> > > > +    <entity-resource type="data" reader-name="demo" loader="main"
> > > > location="data/DemoEbayStoreData.xml"/>
> > > > +
> > > > +    <service-resource type="eca" loader="main"
> > > location="servicedef/secas.
> > > > xml"/>
> > > > +    <service-resource type="model" loader="main"
> > > > location="servicedef/services.xml"/>
> > > > +    <service-resource type="model" loader="main"
> > > > location="servicedef/services_store.xml"/>
> > > > +
> > > > +    <webapp name="ebaystore"
> > > > +        title="eBay Store"
> > > > +        menu-name="secondary"
> > > > +        server="default-server"
> > > > +        location="webapp/ebaystore"
> > > > +        base-permission="EBAYSTORE"
> > > > +        mount-point="/ebaystore"
> > > > +        app-bar-display="true"/>
> > > > +</ofbiz-component>
> > > >
> > > >
> > > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

Pierre Smits
Maybe removing code that doesn't work from the code base?

Best regard,

Pierre Smits

ORRTIZ.COM <http://www.orrtiz.com>
OFBiz based solutions & services

OFBiz Extensions Marketplace
http://oem.ofbizci.net/oci-2/

On Sun, Jan 29, 2017 at 9:49 PM, Scott Gray <[hidden email]>
wrote:

> Hi Pierre,
>
> It was disabled and it is still disabled, the only change relevant to
> ebaystore is the means of disablement.  I'm not sure what needs to be
> re-thought?  Eventually someone will decide that addressing the 3rd party
> library issues in ebaystore is an itch they want to scratch, but until then
> the status quo has been maintained so I don't see an issue.
>
> Regards
> Scott
>
>
>
> On 30 January 2017 at 03:46, Pierre Smits <[hidden email]> wrote:
>
> > I know that the component was commented out in component-load.xml. It has
> > been done so with rev
> > http://svn.apache.org/viewvc?view=revision&sortby=rev&revision=1759583
> >
> > Based on the commit comment, due to some unforeseen difficulty with
> > required 3rd party code libraries...
> >
> > Apparently we are not addressing the underlying issue, but keep pilling
> > workaround on workaround.
> > Would it not be better to rethink this?
> >
> >
> >
> > Pierre Smits
> >
> > ORRTIZ.COM <http://www.orrtiz.com>
> > OFBiz based solutions & services
> >
> > OFBiz Extensions Marketplace
> > http://oem.ofbizci.net/oci-2/
> >
> > On Sun, Jan 29, 2017 at 2:49 PM, Taher Alkhateeb <
> > [hidden email]
> > > wrote:
> >
> > > The ebaystore component was commented out in component-load.xml. Since
> > this
> > > file is removed we need another mechanism to disable the component,
> > > therefore the build scripts now ignore components that are disabled in
> > > ofbiz-component.xml. This is not only a workaround but a cleaner
> solution
> > > that is consistent with the way components are loaded from the
> framework.
> > >
> > > On Jan 29, 2017 4:43 PM, "Pierre Smits" <[hidden email]>
> wrote:
> > >
> > > > In what sense is 'Disables the ebaystore component in
> > > ofbiz-component.xml'
> > > > related  OFBIZ-9182 and thus leading to have referenced in this
> commit
> > > > description ? Nothing in the description of the JIRA issue details
> such
> > > > requirement. Is this commit wrong, or is the description of the JIRA
> > > issue
> > > > wrong?
> > > >
> > > > Best regards,
> > > >
> > > >
> > > >
> > > >
> > > > Pierre Smits
> > > >
> > > > ORRTIZ.COM <http://www.orrtiz.com>
> > > > OFBiz based solutions & services
> > > >
> > > > OFBiz Extensions Marketplace
> > > > http://oem.ofbizci.net/oci-2/
> > > >
> > > > On Sun, Jan 29, 2017 at 2:07 PM, <[hidden email]> wrote:
> > > >
> > > > > Author: taher
> > > > > Date: Sun Jan 29 13:07:32 2017
> > > > > New Revision: 1780790
> > > > >
> > > > > URL: http://svn.apache.org/viewvc?rev=1780790&view=rev
> > > > > Log:
> > > > > Improved: fully refactored the component loading logic in Gradle
> > > > > (OFBIZ-9182)
> > > > >
> > > > > This commit provides the following:
> > > > > - Fully refactor common.gradle so that it mirrors the component
> > loading
> > > > >   logic found in ComponentContainer. This makes the loading very
> > > flexible
> > > > >   and changing the directory structure should have no impact on the
> > > build
> > > > >   scripts.
> > > > > - Gradle will now ignore a component if it exists but disabled in
> > > > >   ofbiz-component.xml (i.e. enabled="false")
> > > > > - Removes /plugins/component-load.xml
> > > > > - Disables the ebaystore component in ofbiz-component.xml
> > > > >
> > > > > The above is needed as a first step toward preparing the project
> for
> > > svn
> > > > > restructure
> > > > >
> > > > > Removed:
> > > > >     ofbiz/trunk/plugins/component-load.xml
> > > > > Modified:
> > > > >     ofbiz/trunk/common.gradle
> > > > >     ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > > >
> > > > > Modified: ofbiz/trunk/common.gradle
> > > > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/common.gradle?rev=
> > > > > 1780790&r1=1780789&r2=1780790&view=diff
> > > > > ============================================================
> > > > > ==================
> > > > > --- ofbiz/trunk/common.gradle (original)
> > > > > +++ ofbiz/trunk/common.gradle Sun Jan 29 13:07:32 2017
> > > > > @@ -17,27 +17,37 @@
> > > > >   * under the License.
> > > > >   */
> > > > >  def iterateOverActiveComponents(applyFunction) {
> > > > > -    def frameworkComponents = new XmlParser().parse("${rootDir}/
> > > > > framework/component-load.xml")
> > > > > -    def applicationsComponents = new
> XmlParser().parse("${rootDir}/
> > > > > applications/component-load.xml")
> > > > > -    def pluginsComponents = new XmlParser().parse("${rootDir}/
> > > > > plugins/component-load.xml")
> > > > >
> > > > > +    // Start is not a real component, therefore loading it
> manually
> > > > >      applyFunction file("${rootDir}/framework/start")
> > > > >
> > > > > -    frameworkComponents.children().each { component ->
> > > > > -        applyFunction file("${rootDir}/framework/"+component.@
> > > > "component-
> > > > > location")
> > > > > -    }
> > > > > -    applicationsComponents.children().each { component ->
> > > > > -        applyFunction file("${rootDir}/applications/"+component.@
> > > > > "component-location")
> > > > > -    }
> > > > > -    pluginsComponents.children().each { component ->
> > > > > -        applyFunction file("${rootDir}/plugins/"+component.@
> > > "component-
> > > > > location")
> > > > > -    }
> > > > > +    def rootComponents = new XmlParser().parse("${rootDir}/
> > > > > framework/base/config/component-load.xml")
> > > > > +    rootComponents.children().each { rootComponent ->
> > > > > +        File componentLoadFile = file "${rootDir}/"+
> rootComponent.@
> > > > > "parent-directory" + "/component-load.xml"
> > > > >
> > > > > -    file("${rootDir}/themes").eachDir { component ->
> > > > > -        applyFunction(component)
> > > > > +        if(componentLoadFile.exists()) {
> > > > > +            // iterate through the components defined in
> > > > > component-load.xml
> > > > > +            def parsedComponents = new XmlParser().parse(
> > > > > componentLoadFile.toString())
> > > > > +            parsedComponents.children().each { component ->
> > > > > +                def componentLocation = file "${rootDir}/"+
> > > > > rootComponent.@"parent-directory" + '/' + component.@"component-
> > > > location"
> > > > > +                applyIfEnabled(componentLocation, applyFunction)
> > > > > +            }
> > > > > +        } else {
> > > > > +            // iterate through all components (subdirectories of
> the
> > > > root
> > > > > component)
> > > > > +            file(rootComponent.@"parent-directory").eachDir {
> > > > > componentLocation ->
> > > > > +                applyIfEnabled(componentLocation, applyFunction)
> > > > > +            }
> > > > > +        }
> > > > >      }
> > > > > -    file("${rootDir}/hot-deploy").eachDir { component ->
> > > > > -        applyFunction(component)
> > > > > +}
> > > > > +
> > > > > +def applyIfEnabled(componentDir, applyFunction) {
> > > > > +    File componentFile = file componentDir.toString() +
> > > > > '/ofbiz-component.xml'
> > > > > +    if(componentFile.exists()) {
> > > > > +        def parsedComponent = new XmlParser().parse(
> > > > > componentFile.toString())
> > > > > +        if(parsedComponent.@enabled == null ||
> > > parsedComponent.@enabled
> > > > > == "true") {
> > > > > +            applyFunction componentDir
> > > > > +        }
> > > > >      }
> > > > >  }
> > > > >
> > > > >
> > > > > Modified: ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/plugins/ebaystore/
> > > > > ofbiz-component.xml?rev=1780790&r1=1780789&r2=1780790&view=diff
> > > > > ============================================================
> > > > > ==================
> > > > > --- ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml (original)
> > > > > +++ ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml Sun Jan 29
> > > > 13:07:32
> > > > > 2017
> > > > > @@ -1,48 +1,48 @@
> > > > > -<?xml version="1.0" encoding="UTF-8"?>
> > > > > -<!--
> > > > > -Licensed to the Apache Software Foundation (ASF) under one
> > > > > -or more contributor license agreements.  See the NOTICE file
> > > > > -distributed with this work for additional information
> > > > > -regarding copyright ownership.  The ASF licenses this file
> > > > > -to you under the Apache License, Version 2.0 (the
> > > > > -"License"); you may not use this file except in compliance
> > > > > -with the License.  You may obtain a copy of the License at
> > > > > -
> > > > > -http://www.apache.org/licenses/LICENSE-2.0
> > > > > -
> > > > > -Unless required by applicable law or agreed to in writing,
> > > > > -software distributed under the License is distributed on an
> > > > > -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > > -KIND, either express or implied.  See the License for the
> > > > > -specific language governing permissions and limitations
> > > > > -under the License.
> > > > > --->
> > > > > -
> > > > > -<ofbiz-component name="ebaystore" enabled="true"
> > > > > -        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > > > -        xsi:noNamespaceSchemaLocation="
> > http://ofbiz.apache.org/dtds/
> > > > > ofbiz-component.xsd">
> > > > > -    <resource-loader name="main" type="component"/>
> > > > > -    <classpath type="dir" location="config"/>
> > > > > -
> > > > > -    <entity-resource type="model" reader-name="main" loader="main"
> > > > > location="entitydef/entitymodel.xml"/>
> > > > > -
> > > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > > > -    <entity-resource type="data" reader-name="demo" loader="main"
> > > > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStoreTypeData.xml"/>
> > > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStoreHelpData.xml"/>
> > > > > -    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStorePortletData.xml"/>
> > > > > -    <entity-resource type="data" reader-name="demo" loader="main"
> > > > > location="data/DemoEbayStoreData.xml"/>
> > > > > -
> > > > > -    <service-resource type="eca" loader="main"
> > > > location="servicedef/secas.
> > > > > xml"/>
> > > > > -    <service-resource type="model" loader="main"
> > > > > location="servicedef/services.xml"/>
> > > > > -    <service-resource type="model" loader="main"
> > > > > location="servicedef/services_store.xml"/>
> > > > > -
> > > > > -    <webapp name="ebaystore"
> > > > > -        title="eBay Store"
> > > > > -        menu-name="secondary"
> > > > > -        server="default-server"
> > > > > -        location="webapp/ebaystore"
> > > > > -        base-permission="EBAYSTORE"
> > > > > -        mount-point="/ebaystore"
> > > > > -        app-bar-display="true"/>
> > > > > -</ofbiz-component>
> > > > > +<?xml version="1.0" encoding="UTF-8"?>
> > > > > +<!--
> > > > > +Licensed to the Apache Software Foundation (ASF) under one
> > > > > +or more contributor license agreements.  See the NOTICE file
> > > > > +distributed with this work for additional information
> > > > > +regarding copyright ownership.  The ASF licenses this file
> > > > > +to you under the Apache License, Version 2.0 (the
> > > > > +"License"); you may not use this file except in compliance
> > > > > +with the License.  You may obtain a copy of the License at
> > > > > +
> > > > > +http://www.apache.org/licenses/LICENSE-2.0
> > > > > +
> > > > > +Unless required by applicable law or agreed to in writing,
> > > > > +software distributed under the License is distributed on an
> > > > > +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > > +KIND, either express or implied.  See the License for the
> > > > > +specific language governing permissions and limitations
> > > > > +under the License.
> > > > > +-->
> > > > > +
> > > > > +<ofbiz-component name="ebaystore" enabled="false"
> > > > > +        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > > > +        xsi:noNamespaceSchemaLocation="
> > http://ofbiz.apache.org/dtds/
> > > > > ofbiz-component.xsd">
> > > > > +    <resource-loader name="main" type="component"/>
> > > > > +    <classpath type="dir" location="config"/>
> > > > > +
> > > > > +    <entity-resource type="model" reader-name="main" loader="main"
> > > > > location="entitydef/entitymodel.xml"/>
> > > > > +
> > > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > > > +    <entity-resource type="data" reader-name="demo" loader="main"
> > > > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStoreTypeData.xml"/>
> > > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStoreHelpData.xml"/>
> > > > > +    <entity-resource type="data" reader-name="seed" loader="main"
> > > > > location="data/EbayStorePortletData.xml"/>
> > > > > +    <entity-resource type="data" reader-name="demo" loader="main"
> > > > > location="data/DemoEbayStoreData.xml"/>
> > > > > +
> > > > > +    <service-resource type="eca" loader="main"
> > > > location="servicedef/secas.
> > > > > xml"/>
> > > > > +    <service-resource type="model" loader="main"
> > > > > location="servicedef/services.xml"/>
> > > > > +    <service-resource type="model" loader="main"
> > > > > location="servicedef/services_store.xml"/>
> > > > > +
> > > > > +    <webapp name="ebaystore"
> > > > > +        title="eBay Store"
> > > > > +        menu-name="secondary"
> > > > > +        server="default-server"
> > > > > +        location="webapp/ebaystore"
> > > > > +        base-permission="EBAYSTORE"
> > > > > +        mount-point="/ebaystore"
> > > > > +        app-bar-display="true"/>
> > > > > +</ofbiz-component>
> > > > >
> > > > >
> > > > >
> > > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1780790 - in /ofbiz/trunk: common.gradle plugins/component-load.xml plugins/ebaystore/ofbiz-component.xml

Scott Gray-3
Perhaps, but I can't recall what the issue was with the 3rd party
libraries.  If it isn't something we can get past from a licensing point of
view then the options are:
1. Provide instructions on how to download the incompatible libs and keep
the component disabled
2. Remove it altogether

Perhaps it wasn't even related to licensing but instead was because we
don't have a jcenter source for the libs.

I can't really provide an opinion on it without doing some research first.
Regardless though, the topic isn't really relevant to Taher's commit.

Regards
Scott

On 30 January 2017 at 09:54, Pierre Smits <[hidden email]> wrote:

> Maybe removing code that doesn't work from the code base?
>
> Best regard,
>
> Pierre Smits
>
> ORRTIZ.COM <http://www.orrtiz.com>
> OFBiz based solutions & services
>
> OFBiz Extensions Marketplace
> http://oem.ofbizci.net/oci-2/
>
> On Sun, Jan 29, 2017 at 9:49 PM, Scott Gray <[hidden email]>
> wrote:
>
> > Hi Pierre,
> >
> > It was disabled and it is still disabled, the only change relevant to
> > ebaystore is the means of disablement.  I'm not sure what needs to be
> > re-thought?  Eventually someone will decide that addressing the 3rd party
> > library issues in ebaystore is an itch they want to scratch, but until
> then
> > the status quo has been maintained so I don't see an issue.
> >
> > Regards
> > Scott
> >
> >
> >
> > On 30 January 2017 at 03:46, Pierre Smits <[hidden email]>
> wrote:
> >
> > > I know that the component was commented out in component-load.xml. It
> has
> > > been done so with rev
> > > http://svn.apache.org/viewvc?view=revision&sortby=rev&revision=1759583
> > >
> > > Based on the commit comment, due to some unforeseen difficulty with
> > > required 3rd party code libraries...
> > >
> > > Apparently we are not addressing the underlying issue, but keep pilling
> > > workaround on workaround.
> > > Would it not be better to rethink this?
> > >
> > >
> > >
> > > Pierre Smits
> > >
> > > ORRTIZ.COM <http://www.orrtiz.com>
> > > OFBiz based solutions & services
> > >
> > > OFBiz Extensions Marketplace
> > > http://oem.ofbizci.net/oci-2/
> > >
> > > On Sun, Jan 29, 2017 at 2:49 PM, Taher Alkhateeb <
> > > [hidden email]
> > > > wrote:
> > >
> > > > The ebaystore component was commented out in component-load.xml.
> Since
> > > this
> > > > file is removed we need another mechanism to disable the component,
> > > > therefore the build scripts now ignore components that are disabled
> in
> > > > ofbiz-component.xml. This is not only a workaround but a cleaner
> > solution
> > > > that is consistent with the way components are loaded from the
> > framework.
> > > >
> > > > On Jan 29, 2017 4:43 PM, "Pierre Smits" <[hidden email]>
> > wrote:
> > > >
> > > > > In what sense is 'Disables the ebaystore component in
> > > > ofbiz-component.xml'
> > > > > related  OFBIZ-9182 and thus leading to have referenced in this
> > commit
> > > > > description ? Nothing in the description of the JIRA issue details
> > such
> > > > > requirement. Is this commit wrong, or is the description of the
> JIRA
> > > > issue
> > > > > wrong?
> > > > >
> > > > > Best regards,
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Pierre Smits
> > > > >
> > > > > ORRTIZ.COM <http://www.orrtiz.com>
> > > > > OFBiz based solutions & services
> > > > >
> > > > > OFBiz Extensions Marketplace
> > > > > http://oem.ofbizci.net/oci-2/
> > > > >
> > > > > On Sun, Jan 29, 2017 at 2:07 PM, <[hidden email]> wrote:
> > > > >
> > > > > > Author: taher
> > > > > > Date: Sun Jan 29 13:07:32 2017
> > > > > > New Revision: 1780790
> > > > > >
> > > > > > URL: http://svn.apache.org/viewvc?rev=1780790&view=rev
> > > > > > Log:
> > > > > > Improved: fully refactored the component loading logic in Gradle
> > > > > > (OFBIZ-9182)
> > > > > >
> > > > > > This commit provides the following:
> > > > > > - Fully refactor common.gradle so that it mirrors the component
> > > loading
> > > > > >   logic found in ComponentContainer. This makes the loading very
> > > > flexible
> > > > > >   and changing the directory structure should have no impact on
> the
> > > > build
> > > > > >   scripts.
> > > > > > - Gradle will now ignore a component if it exists but disabled in
> > > > > >   ofbiz-component.xml (i.e. enabled="false")
> > > > > > - Removes /plugins/component-load.xml
> > > > > > - Disables the ebaystore component in ofbiz-component.xml
> > > > > >
> > > > > > The above is needed as a first step toward preparing the project
> > for
> > > > svn
> > > > > > restructure
> > > > > >
> > > > > > Removed:
> > > > > >     ofbiz/trunk/plugins/component-load.xml
> > > > > > Modified:
> > > > > >     ofbiz/trunk/common.gradle
> > > > > >     ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > > > >
> > > > > > Modified: ofbiz/trunk/common.gradle
> > > > > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/common.gradle?rev=
> > > > > > 1780790&r1=1780789&r2=1780790&view=diff
> > > > > > ============================================================
> > > > > > ==================
> > > > > > --- ofbiz/trunk/common.gradle (original)
> > > > > > +++ ofbiz/trunk/common.gradle Sun Jan 29 13:07:32 2017
> > > > > > @@ -17,27 +17,37 @@
> > > > > >   * under the License.
> > > > > >   */
> > > > > >  def iterateOverActiveComponents(applyFunction) {
> > > > > > -    def frameworkComponents = new XmlParser().parse("${rootDir}/
> > > > > > framework/component-load.xml")
> > > > > > -    def applicationsComponents = new
> > XmlParser().parse("${rootDir}/
> > > > > > applications/component-load.xml")
> > > > > > -    def pluginsComponents = new XmlParser().parse("${rootDir}/
> > > > > > plugins/component-load.xml")
> > > > > >
> > > > > > +    // Start is not a real component, therefore loading it
> > manually
> > > > > >      applyFunction file("${rootDir}/framework/start")
> > > > > >
> > > > > > -    frameworkComponents.children().each { component ->
> > > > > > -        applyFunction file("${rootDir}/framework/"+component.@
> > > > > "component-
> > > > > > location")
> > > > > > -    }
> > > > > > -    applicationsComponents.children().each { component ->
> > > > > > -        applyFunction file("${rootDir}/applications/
> "+component.@
> > > > > > "component-location")
> > > > > > -    }
> > > > > > -    pluginsComponents.children().each { component ->
> > > > > > -        applyFunction file("${rootDir}/plugins/"+component.@
> > > > "component-
> > > > > > location")
> > > > > > -    }
> > > > > > +    def rootComponents = new XmlParser().parse("${rootDir}/
> > > > > > framework/base/config/component-load.xml")
> > > > > > +    rootComponents.children().each { rootComponent ->
> > > > > > +        File componentLoadFile = file "${rootDir}/"+
> > rootComponent.@
> > > > > > "parent-directory" + "/component-load.xml"
> > > > > >
> > > > > > -    file("${rootDir}/themes").eachDir { component ->
> > > > > > -        applyFunction(component)
> > > > > > +        if(componentLoadFile.exists()) {
> > > > > > +            // iterate through the components defined in
> > > > > > component-load.xml
> > > > > > +            def parsedComponents = new XmlParser().parse(
> > > > > > componentLoadFile.toString())
> > > > > > +            parsedComponents.children().each { component ->
> > > > > > +                def componentLocation = file "${rootDir}/"+
> > > > > > rootComponent.@"parent-directory" + '/' + component.@"component-
> > > > > location"
> > > > > > +                applyIfEnabled(componentLocation,
> applyFunction)
> > > > > > +            }
> > > > > > +        } else {
> > > > > > +            // iterate through all components (subdirectories of
> > the
> > > > > root
> > > > > > component)
> > > > > > +            file(rootComponent.@"parent-directory").eachDir {
> > > > > > componentLocation ->
> > > > > > +                applyIfEnabled(componentLocation,
> applyFunction)
> > > > > > +            }
> > > > > > +        }
> > > > > >      }
> > > > > > -    file("${rootDir}/hot-deploy").eachDir { component ->
> > > > > > -        applyFunction(component)
> > > > > > +}
> > > > > > +
> > > > > > +def applyIfEnabled(componentDir, applyFunction) {
> > > > > > +    File componentFile = file componentDir.toString() +
> > > > > > '/ofbiz-component.xml'
> > > > > > +    if(componentFile.exists()) {
> > > > > > +        def parsedComponent = new XmlParser().parse(
> > > > > > componentFile.toString())
> > > > > > +        if(parsedComponent.@enabled == null ||
> > > > parsedComponent.@enabled
> > > > > > == "true") {
> > > > > > +            applyFunction componentDir
> > > > > > +        }
> > > > > >      }
> > > > > >  }
> > > > > >
> > > > > >
> > > > > > Modified: ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml
> > > > > > URL: http://svn.apache.org/viewvc/ofbiz/trunk/plugins/ebaystore/
> > > > > > ofbiz-component.xml?rev=1780790&r1=1780789&r2=1780790&view=diff
> > > > > > ============================================================
> > > > > > ==================
> > > > > > --- ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml (original)
> > > > > > +++ ofbiz/trunk/plugins/ebaystore/ofbiz-component.xml Sun Jan 29
> > > > > 13:07:32
> > > > > > 2017
> > > > > > @@ -1,48 +1,48 @@
> > > > > > -<?xml version="1.0" encoding="UTF-8"?>
> > > > > > -<!--
> > > > > > -Licensed to the Apache Software Foundation (ASF) under one
> > > > > > -or more contributor license agreements.  See the NOTICE file
> > > > > > -distributed with this work for additional information
> > > > > > -regarding copyright ownership.  The ASF licenses this file
> > > > > > -to you under the Apache License, Version 2.0 (the
> > > > > > -"License"); you may not use this file except in compliance
> > > > > > -with the License.  You may obtain a copy of the License at
> > > > > > -
> > > > > > -http://www.apache.org/licenses/LICENSE-2.0
> > > > > > -
> > > > > > -Unless required by applicable law or agreed to in writing,
> > > > > > -software distributed under the License is distributed on an
> > > > > > -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > > > -KIND, either express or implied.  See the License for the
> > > > > > -specific language governing permissions and limitations
> > > > > > -under the License.
> > > > > > --->
> > > > > > -
> > > > > > -<ofbiz-component name="ebaystore" enabled="true"
> > > > > > -        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > > > > -        xsi:noNamespaceSchemaLocation="
> > > http://ofbiz.apache.org/dtds/
> > > > > > ofbiz-component.xsd">
> > > > > > -    <resource-loader name="main" type="component"/>
> > > > > > -    <classpath type="dir" location="config"/>
> > > > > > -
> > > > > > -    <entity-resource type="model" reader-name="main"
> loader="main"
> > > > > > location="entitydef/entitymodel.xml"/>
> > > > > > -
> > > > > > -    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > > > > -    <entity-resource type="data" reader-name="demo"
> loader="main"
> > > > > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > > > > -    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStoreTypeData.xml"/>
> > > > > > -    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStoreHelpData.xml"/>
> > > > > > -    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStorePortletData.xml"/>
> > > > > > -    <entity-resource type="data" reader-name="demo"
> loader="main"
> > > > > > location="data/DemoEbayStoreData.xml"/>
> > > > > > -
> > > > > > -    <service-resource type="eca" loader="main"
> > > > > location="servicedef/secas.
> > > > > > xml"/>
> > > > > > -    <service-resource type="model" loader="main"
> > > > > > location="servicedef/services.xml"/>
> > > > > > -    <service-resource type="model" loader="main"
> > > > > > location="servicedef/services_store.xml"/>
> > > > > > -
> > > > > > -    <webapp name="ebaystore"
> > > > > > -        title="eBay Store"
> > > > > > -        menu-name="secondary"
> > > > > > -        server="default-server"
> > > > > > -        location="webapp/ebaystore"
> > > > > > -        base-permission="EBAYSTORE"
> > > > > > -        mount-point="/ebaystore"
> > > > > > -        app-bar-display="true"/>
> > > > > > -</ofbiz-component>
> > > > > > +<?xml version="1.0" encoding="UTF-8"?>
> > > > > > +<!--
> > > > > > +Licensed to the Apache Software Foundation (ASF) under one
> > > > > > +or more contributor license agreements.  See the NOTICE file
> > > > > > +distributed with this work for additional information
> > > > > > +regarding copyright ownership.  The ASF licenses this file
> > > > > > +to you under the Apache License, Version 2.0 (the
> > > > > > +"License"); you may not use this file except in compliance
> > > > > > +with the License.  You may obtain a copy of the License at
> > > > > > +
> > > > > > +http://www.apache.org/licenses/LICENSE-2.0
> > > > > > +
> > > > > > +Unless required by applicable law or agreed to in writing,
> > > > > > +software distributed under the License is distributed on an
> > > > > > +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > > > +KIND, either express or implied.  See the License for the
> > > > > > +specific language governing permissions and limitations
> > > > > > +under the License.
> > > > > > +-->
> > > > > > +
> > > > > > +<ofbiz-component name="ebaystore" enabled="false"
> > > > > > +        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > > > > > +        xsi:noNamespaceSchemaLocation="
> > > http://ofbiz.apache.org/dtds/
> > > > > > ofbiz-component.xsd">
> > > > > > +    <resource-loader name="main" type="component"/>
> > > > > > +    <classpath type="dir" location="config"/>
> > > > > > +
> > > > > > +    <entity-resource type="model" reader-name="main"
> loader="main"
> > > > > > location="entitydef/entitymodel.xml"/>
> > > > > > +
> > > > > > +    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStoreSecurityPermissionSeedData.xml"/>
> > > > > > +    <entity-resource type="data" reader-name="demo"
> loader="main"
> > > > > > location="data/EbayStoreSecurityGroupDemoData.xml"/>
> > > > > > +    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStoreTypeData.xml"/>
> > > > > > +    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStoreHelpData.xml"/>
> > > > > > +    <entity-resource type="data" reader-name="seed"
> loader="main"
> > > > > > location="data/EbayStorePortletData.xml"/>
> > > > > > +    <entity-resource type="data" reader-name="demo"
> loader="main"
> > > > > > location="data/DemoEbayStoreData.xml"/>
> > > > > > +
> > > > > > +    <service-resource type="eca" loader="main"
> > > > > location="servicedef/secas.
> > > > > > xml"/>
> > > > > > +    <service-resource type="model" loader="main"
> > > > > > location="servicedef/services.xml"/>
> > > > > > +    <service-resource type="model" loader="main"
> > > > > > location="servicedef/services_store.xml"/>
> > > > > > +
> > > > > > +    <webapp name="ebaystore"
> > > > > > +        title="eBay Store"
> > > > > > +        menu-name="secondary"
> > > > > > +        server="default-server"
> > > > > > +        location="webapp/ebaystore"
> > > > > > +        base-permission="EBAYSTORE"
> > > > > > +        mount-point="/ebaystore"
> > > > > > +        app-bar-display="true"/>
> > > > > > +</ofbiz-component>
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>