Shipping data duplicated

[Jacques Le Roux]

Hi,

Is there a reason why Shipping data are now duplicated in trunk at

     applications/commonext/data/ShippingData.xml

and

     /applications/datamodel/data/demo/OrderDemoData.xml

This is not the case with current stable

Jacques

[Rishi Solanki]

This should be part of effort when we were moving all the seed, ext, demo
data to datamodel component. I see there is no entry to load the
ShippingData.xml.
I will check this in next week and fix the duplicate data exists in the
system, or share the the reason of having this.

Probably, data moved but not removed from commonext. But I'll confirm and
get back.

Thanks!



Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com
www.hotwax.co

On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
[hidden email]> wrote:

> Hi,
>
> Is there a reason why Shipping data are now duplicated in trunk at
>
>     applications/commonext/data/ShippingData.xml
>
> and
>
>     /applications/datamodel/data/demo/OrderDemoData.xml
>
> This is not the case with current stable
>
> Jacques
>
>

[Jacques Le Roux]

Thanks Rishi!

Jacques


Le 17/08/2018 à 15:19, Rishi Solanki a écrit :

> This should be part of effort when we were moving all the seed, ext, demo
> data to datamodel component. I see there is no entry to load the
> ShippingData.xml.
> I will check this in next week and fix the duplicate data exists in the
> system, or share the the reason of having this.
>
> Probably, data moved but not removed from commonext. But I'll confirm and
> get back.
>
> Thanks!
>
>
>
> Rishi Solanki
> Sr Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: [phone]
> http://www.hotwaxsystems.com
> www.hotwax.co
>
> On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
> [hidden email]> wrote:
>
>> Hi,
>>
>> Is there a reason why Shipping data are now duplicated in trunk at
>>
>>      applications/commonext/data/ShippingData.xml
>>
>> and
>>
>>      /applications/datamodel/data/demo/OrderDemoData.xml
>>
>> This is not the case with current stable
>>
>> Jacques
>>
>>

[Rishi Solanki]

Hi Jacques,
I dig into it today and found that the data exists in the
"applications/datamodel/data/demo/OrderDemoData.xml" was moved from
"applications/order/data/DemoShipping.xml"
Also the data claimed as duplicate in
"applications/commonext/data/ShippingData.xml" is similar but not exact
matches with OrderDemoData.xml. The ShippingData.xml  file has no entry in
any ofbiz-component.xml.

So here we can decide weather we should keep that data in ShippingData.xml
(if someone introduce the file intentionally) or we can remove it from
trunk. In case no objection I would like to remove it as most data is
duplicate.

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com
www.hotwax.co


On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
[hidden email]> wrote:

> Thanks Rishi!
>
> Jacques
>
>
> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
> > This should be part of effort when we were moving all the seed, ext, demo
> > data to datamodel component. I see there is no entry to load the
> > ShippingData.xml.
> > I will check this in next week and fix the duplicate data exists in the
> > system, or share the the reason of having this.
> >
> > Probably, data moved but not removed from commonext. But I'll confirm and
> > get back.
> >
> > Thanks!
> >
> >
> >
> > Rishi Solanki
> > Sr Manager, Enterprise Software Development
> > HotWax Systems Pvt. Ltd.
> > Direct: [phone]
> > http://www.hotwaxsystems.com
> > www.hotwax.co
> >
> > On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
> > [hidden email]> wrote:
> >
> >> Hi,
> >>
> >> Is there a reason why Shipping data are now duplicated in trunk at
> >>
> >>      applications/commonext/data/ShippingData.xml
> >>
> >> and
> >>
> >>      /applications/datamodel/data/demo/OrderDemoData.xml
> >>
> >> This is not the case with current stable
> >>
> >> Jacques
> >>
> >>
>
>

[Jacques Le Roux]

Hi Rishi,

Thanks for your feedback.

Looking closely at history, ShippingData.xml was put in 9 years ago with http://svn.apache.org/viewvc?view=revision&revision=816083, so for the
ofbizsetup app, which is a bit weirdo in OFBiz.

Then was modified with OFBIZ-5890 and OFBIZ-7673

Rest inline...


Le 22/09/2018 à 09:27, Rishi Solanki a écrit :
> Hi Jacques,
> I dig into it today and found that the data exists in the
> "applications/datamodel/data/demo/OrderDemoData.xml" was moved from
> "applications/order/data/DemoShipping.xml"
> Also the data claimed as duplicate in
> "applications/commonext/data/ShippingData.xml" is similar but not exact
> matches with OrderDemoData.xml.
Yes, I was only speaking about "Shipping data", ie:
ShipmentMethodType
CarrierShipmentMethod
QuantityBreak
ShipmentBoxType

Those are real duplicate
> The ShippingData.xml  file has no entry in
> any ofbiz-component.xml.
Indeed, they are only used by the setup app. In SetupEvents.xml there is

    <!-- run demo Shipping  -->
    <set field="importShippingCtx.filename" value="${sys:getProperty('ofbiz.home')}/applications/commonext/data/ShippingData.xml"/>

Reading that, now I think we should not only keep the "Shipping data" in  ShippingData.xml but also the file. I would rather rename this file and
other ofbizsetup related files (at least data files) with a Setup prefix to clearly signal they are  part of this app.
But I also wonder if the ofbizsetup app is still alive, maintained and used by users. Last time I tried I crossed issues (not biggie IIRW). I found
this https://issues.apache.org/jira/issues/?filter=12344840

What do you think?

Jacques

>
> So here we can decide weather we should keep that data in ShippingData.xml
> (if someone introduce the file intentionally) or we can remove it from
> trunk. In case no objection I would like to remove it as most data is
> duplicate.
>
> --
> Rishi Solanki
> Sr Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: [phone]
> http://www.hotwaxsystems.com
> www.hotwax.co
>
>
> On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
> [hidden email]> wrote:
>
>> Thanks Rishi!
>>
>> Jacques
>>
>>
>> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
>>> This should be part of effort when we were moving all the seed, ext, demo
>>> data to datamodel component. I see there is no entry to load the
>>> ShippingData.xml.
>>> I will check this in next week and fix the duplicate data exists in the
>>> system, or share the the reason of having this.
>>>
>>> Probably, data moved but not removed from commonext. But I'll confirm and
>>> get back.
>>>
>>> Thanks!
>>>
>>>
>>>
>>> Rishi Solanki
>>> Sr Manager, Enterprise Software Development
>>> HotWax Systems Pvt. Ltd.
>>> Direct: [phone]
>>> http://www.hotwaxsystems.com
>>> www.hotwax.co
>>>
>>> On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
>>> [hidden email]> wrote:
>>>
>>>> Hi,
>>>>
>>>> Is there a reason why Shipping data are now duplicated in trunk at
>>>>
>>>>       applications/commonext/data/ShippingData.xml
>>>>
>>>> and
>>>>
>>>>       /applications/datamodel/data/demo/OrderDemoData.xml
>>>>
>>>> This is not the case with current stable
>>>>
>>>> Jacques
>>>>
>>>>
>>

[Rishi Solanki]

Jacques,
Thanks for more insights.
IMO, we should rename the files as you suggested and also add some
description in the file so that we won't confuse by this in future. And
also we should keep the duplicate data as well, because when ofbizsetup app
set the data for store I assume we don't load the demo data.
About the ofbizsetup app uses, right now I could not think of it. But will
get back on it soon with details and get the inputs from community. For now
my understanding is to rename the setup files and let duplicate data exists
in the setup files.
Let me know if we can proceed with the above plan then will rename the
files and do the needful changes in the setup code. On ofbizsetup app we
will discuss once I come with more details.

Thanks!

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com
www.hotwax.co


On Sat, Sep 22, 2018 at 2:10 PM Jacques Le Roux <
[hidden email]> wrote:

> Hi Rishi,
>
> Thanks for your feedback.
>
> Looking closely at history, ShippingData.xml was put in 9 years ago with
> http://svn.apache.org/viewvc?view=revision&revision=816083, so for the
> ofbizsetup app, which is a bit weirdo in OFBiz.
>
> Then was modified with OFBIZ-5890 and OFBIZ-7673
>
> Rest inline...
>
>
> Le 22/09/2018 à 09:27, Rishi Solanki a écrit :
> > Hi Jacques,
> > I dig into it today and found that the data exists in the
> > "applications/datamodel/data/demo/OrderDemoData.xml" was moved from
> > "applications/order/data/DemoShipping.xml"
> > Also the data claimed as duplicate in
> > "applications/commonext/data/ShippingData.xml" is similar but not exact
> > matches with OrderDemoData.xml.
> Yes, I was only speaking about "Shipping data", ie:
> ShipmentMethodType
> CarrierShipmentMethod
> QuantityBreak
> ShipmentBoxType
>
> Those are real duplicate
> > The ShippingData.xml  file has no entry in
> > any ofbiz-component.xml.
> Indeed, they are only used by the setup app. In SetupEvents.xml there is
>
>     <!-- run demo Shipping  -->
>     <set field="importShippingCtx.filename"
> value="${sys:getProperty('ofbiz.home')}/applications/commonext/data/ShippingData.xml"/>
>
> Reading that, now I think we should not only keep the "Shipping data" in
> ShippingData.xml but also the file. I would rather rename this file and
> other ofbizsetup related files (at least data files) with a Setup prefix
> to clearly signal they are  part of this app.
> But I also wonder if the ofbizsetup app is still alive, maintained and
> used by users. Last time I tried I crossed issues (not biggie IIRW). I
> found
> this https://issues.apache.org/jira/issues/?filter=12344840
>
> What do you think?
>
> Jacques
> >
> > So here we can decide weather we should keep that data in
> ShippingData.xml
> > (if someone introduce the file intentionally) or we can remove it from
> > trunk. In case no objection I would like to remove it as most data is
> > duplicate.
> >
> > --
> > Rishi Solanki
> > Sr Manager, Enterprise Software Development
> > HotWax Systems Pvt. Ltd.
> > Direct: [phone]
> > http://www.hotwaxsystems.com
> > www.hotwax.co
> >
> >
> > On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
> > [hidden email]> wrote:
> >
> >> Thanks Rishi!
> >>
> >> Jacques
> >>
> >>
> >> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
> >>> This should be part of effort when we were moving all the seed, ext,
> demo
> >>> data to datamodel component. I see there is no entry to load the
> >>> ShippingData.xml.
> >>> I will check this in next week and fix the duplicate data exists in the
> >>> system, or share the the reason of having this.
> >>>
> >>> Probably, data moved but not removed from commonext. But I'll confirm
> and
> >>> get back.
> >>>
> >>> Thanks!
> >>>
> >>>
> >>>
> >>> Rishi Solanki
> >>> Sr Manager, Enterprise Software Development
> >>> HotWax Systems Pvt. Ltd.
> >>> Direct: [phone]
> >>> http://www.hotwaxsystems.com
> >>> www.hotwax.co
> >>>
> >>> On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
> >>> [hidden email]> wrote:
> >>>
> >>>> Hi,
> >>>>
> >>>> Is there a reason why Shipping data are now duplicated in trunk at
> >>>>
> >>>>       applications/commonext/data/ShippingData.xml
> >>>>
> >>>> and
> >>>>
> >>>>       /applications/datamodel/data/demo/OrderDemoData.xml
> >>>>
> >>>> This is not the case with current stable
> >>>>
> >>>> Jacques
> >>>>
> >>>>
> >>
>
>

Hi Rishi,

Inline...


Le 22/09/2018 à 12:34, Rishi Solanki a écrit :

> Jacques,
> Thanks for more insights.
> IMO, we should rename the files as you suggested and also add some
> description in the file so that we won't confuse by this in future. And
> also we should keep the duplicate data as well, because when ofbizsetup app
> set the data for store I assume we don't load the demo data.
> About the ofbizsetup app uses, right now I could not think of it. But will
> get back on it soon with details and get the inputs from community. For now
> my understanding is to rename the setup files and let duplicate data exists
> in the setup files.
> Let me know if we can proceed with the above plan then will rename the
> files and do the needful changes in the setup code.

Agreed, I created OFBIZ-10598 for that

> On ofbizsetup app we
> will discuss once I come with more details.
Yes no hurry, I think it needs a bit more review, love and documentation/information
Maybe starting by completing https://cwiki.apache.org/confluence/display/OFBENDUSER/How+to+Install+OFBiz+without+the+Demo+Data ?

Jacques

>
> Thanks!
>
> --
> Rishi Solanki
> Sr Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: [phone]
> http://www.hotwaxsystems.com
> www.hotwax.co
>
>
> On Sat, Sep 22, 2018 at 2:10 PM Jacques Le Roux <
> [hidden email]> wrote:
>
>> Hi Rishi,
>>
>> Thanks for your feedback.
>>
>> Looking closely at history, ShippingData.xml was put in 9 years ago with
>> http://svn.apache.org/viewvc?view=revision&revision=816083, so for the
>> ofbizsetup app, which is a bit weirdo in OFBiz.
>>
>> Then was modified with OFBIZ-5890 and OFBIZ-7673
>>
>> Rest inline...
>>
>>
>> Le 22/09/2018 à 09:27, Rishi Solanki a écrit :
>>> Hi Jacques,
>>> I dig into it today and found that the data exists in the
>>> "applications/datamodel/data/demo/OrderDemoData.xml" was moved from
>>> "applications/order/data/DemoShipping.xml"
>>> Also the data claimed as duplicate in
>>> "applications/commonext/data/ShippingData.xml" is similar but not exact
>>> matches with OrderDemoData.xml.
>> Yes, I was only speaking about "Shipping data", ie:
>> ShipmentMethodType
>> CarrierShipmentMethod
>> QuantityBreak
>> ShipmentBoxType
>>
>> Those are real duplicate
>>> The ShippingData.xml  file has no entry in
>>> any ofbiz-component.xml.
>> Indeed, they are only used by the setup app. In SetupEvents.xml there is
>>
>>      <!-- run demo Shipping  -->
>>      <set field="importShippingCtx.filename"
>> value="${sys:getProperty('ofbiz.home')}/applications/commonext/data/ShippingData.xml"/>
>>
>> Reading that, now I think we should not only keep the "Shipping data" in
>> ShippingData.xml but also the file. I would rather rename this file and
>> other ofbizsetup related files (at least data files) with a Setup prefix
>> to clearly signal they are  part of this app.
>> But I also wonder if the ofbizsetup app is still alive, maintained and
>> used by users. Last time I tried I crossed issues (not biggie IIRW). I
>> found
>> this https://issues.apache.org/jira/issues/?filter=12344840
>>
>> What do you think?
>>
>> Jacques
>>> So here we can decide weather we should keep that data in
>> ShippingData.xml
>>> (if someone introduce the file intentionally) or we can remove it from
>>> trunk. In case no objection I would like to remove it as most data is
>>> duplicate.
>>>
>>> --
>>> Rishi Solanki
>>> Sr Manager, Enterprise Software Development
>>> HotWax Systems Pvt. Ltd.
>>> Direct: [phone]
>>> http://www.hotwaxsystems.com
>>> www.hotwax.co
>>>
>>>
>>> On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
>>> [hidden email]> wrote:
>>>
>>>> Thanks Rishi!
>>>>
>>>> Jacques
>>>>
>>>>
>>>> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
>>>>> This should be part of effort when we were moving all the seed, ext,
>> demo
>>>>> data to datamodel component. I see there is no entry to load the
>>>>> ShippingData.xml.
>>>>> I will check this in next week and fix the duplicate data exists in the
>>>>> system, or share the the reason of having this.
>>>>>
>>>>> Probably, data moved but not removed from commonext. But I'll confirm
>> and
>>>>> get back.
>>>>>
>>>>> Thanks!
>>>>>
>>>>>
>>>>>
>>>>> Rishi Solanki
>>>>> Sr Manager, Enterprise Software Development
>>>>> HotWax Systems Pvt. Ltd.
>>>>> Direct: [phone]
>>>>> http://www.hotwaxsystems.com
>>>>> www.hotwax.co
>>>>>
>>>>> On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
>>>>> [hidden email]> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> Is there a reason why Shipping data are now duplicated in trunk at
>>>>>>
>>>>>>        applications/commonext/data/ShippingData.xml
>>>>>>
>>>>>> and
>>>>>>
>>>>>>        /applications/datamodel/data/demo/OrderDemoData.xml
>>>>>>
>>>>>> This is not the case with current stable
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>

[Rishi Solanki]

Thanks Jacques for your reply and Jira ticket. I'll take care of it asap.

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com
www.hotwax.co


On Mon, Oct 8, 2018 at 2:49 PM [hidden email] <[hidden email]>
wrote:

> Hi Rishi,
>
> Inline...
>
>
> Le 22/09/2018 à 12:34, Rishi Solanki a écrit :
> > Jacques,
> > Thanks for more insights.
> > IMO, we should rename the files as you suggested and also add some
> > description in the file so that we won't confuse by this in future. And
> > also we should keep the duplicate data as well, because when ofbizsetup
> app
> > set the data for store I assume we don't load the demo data.
> > About the ofbizsetup app uses, right now I could not think of it. But
> will
> > get back on it soon with details and get the inputs from community. For
> now
> > my understanding is to rename the setup files and let duplicate data
> exists
> > in the setup files.
> > Let me know if we can proceed with the above plan then will rename the
> > files and do the needful changes in the setup code.
> Agreed, I created OFBIZ-10598 for that
>
> > On ofbizsetup app we
> > will discuss once I come with more details.
> Yes no hurry, I think it needs a bit more review, love and
> documentation/information
> Maybe starting by completing
> https://cwiki.apache.org/confluence/display/OFBENDUSER/How+to+Install+OFBiz+without+the+Demo+Data
> ?
>
> Jacques
>
> >
> > Thanks!
> >
> > --
> > Rishi Solanki
> > Sr Manager, Enterprise Software Development
> > HotWax Systems Pvt. Ltd.
> > Direct: [phone]
> > http://www.hotwaxsystems.com
> > www.hotwax.co
> >
> >
> > On Sat, Sep 22, 2018 at 2:10 PM Jacques Le Roux <
> > [hidden email]> wrote:
> >
> >> Hi Rishi,
> >>
> >> Thanks for your feedback.
> >>
> >> Looking closely at history, ShippingData.xml was put in 9 years ago with
> >> http://svn.apache.org/viewvc?view=revision&revision=816083, so for the
> >> ofbizsetup app, which is a bit weirdo in OFBiz.
> >>
> >> Then was modified with OFBIZ-5890 and OFBIZ-7673
> >>
> >> Rest inline...
> >>
> >>
> >> Le 22/09/2018 à 09:27, Rishi Solanki a écrit :
> >>> Hi Jacques,
> >>> I dig into it today and found that the data exists in the
> >>> "applications/datamodel/data/demo/OrderDemoData.xml" was moved from
> >>> "applications/order/data/DemoShipping.xml"
> >>> Also the data claimed as duplicate in
> >>> "applications/commonext/data/ShippingData.xml" is similar but not exact
> >>> matches with OrderDemoData.xml.
> >> Yes, I was only speaking about "Shipping data", ie:
> >> ShipmentMethodType
> >> CarrierShipmentMethod
> >> QuantityBreak
> >> ShipmentBoxType
> >>
> >> Those are real duplicate
> >>> The ShippingData.xml  file has no entry in
> >>> any ofbiz-component.xml.
> >> Indeed, they are only used by the setup app. In SetupEvents.xml there is
> >>
> >>      <!-- run demo Shipping  -->
> >>      <set field="importShippingCtx.filename"
> >>
> value="${sys:getProperty('ofbiz.home')}/applications/commonext/data/ShippingData.xml"/>
> >>
> >> Reading that, now I think we should not only keep the "Shipping data" in
> >> ShippingData.xml but also the file. I would rather rename this file and
> >> other ofbizsetup related files (at least data files) with a Setup prefix
> >> to clearly signal they are  part of this app.
> >> But I also wonder if the ofbizsetup app is still alive, maintained and
> >> used by users. Last time I tried I crossed issues (not biggie IIRW). I
> >> found
> >> this https://issues.apache.org/jira/issues/?filter=12344840
> >>
> >> What do you think?
> >>
> >> Jacques
> >>> So here we can decide weather we should keep that data in
> >> ShippingData.xml
> >>> (if someone introduce the file intentionally) or we can remove it from
> >>> trunk. In case no objection I would like to remove it as most data is
> >>> duplicate.
> >>>
> >>> --
> >>> Rishi Solanki
> >>> Sr Manager, Enterprise Software Development
> >>> HotWax Systems Pvt. Ltd.
> >>> Direct: [phone]
> >>> http://www.hotwaxsystems.com
> >>> www.hotwax.co
> >>>
> >>>
> >>> On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
> >>> [hidden email]> wrote:
> >>>
> >>>> Thanks Rishi!
> >>>>
> >>>> Jacques
> >>>>
> >>>>
> >>>> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
> >>>>> This should be part of effort when we were moving all the seed, ext,
> >> demo
> >>>>> data to datamodel component. I see there is no entry to load the
> >>>>> ShippingData.xml.
> >>>>> I will check this in next week and fix the duplicate data exists in
> the
> >>>>> system, or share the the reason of having this.
> >>>>>
> >>>>> Probably, data moved but not removed from commonext. But I'll confirm
> >> and
> >>>>> get back.
> >>>>>
> >>>>> Thanks!
> >>>>>
> >>>>>
> >>>>>
> >>>>> Rishi Solanki
> >>>>> Sr Manager, Enterprise Software Development
> >>>>> HotWax Systems Pvt. Ltd.
> >>>>> Direct: [phone]
> >>>>> http://www.hotwaxsystems.com
> >>>>> www.hotwax.co
> >>>>>
> >>>>> On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
> >>>>> [hidden email]> wrote:
> >>>>>
> >>>>>> Hi,
> >>>>>>
> >>>>>> Is there a reason why Shipping data are now duplicated in trunk at
> >>>>>>
> >>>>>>        applications/commonext/data/ShippingData.xml
> >>>>>>
> >>>>>> and
> >>>>>>
> >>>>>>        /applications/datamodel/data/demo/OrderDemoData.xml
> >>>>>>
> >>>>>> This is not the case with current stable
> >>>>>>
> >>>>>> Jacques
> >>>>>>
> >>>>>>
> >>
>