Rewriting the OrderItemAssociation entity

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

Rewriting the OrderItemAssociation entity

Jacopo Cappellato
Hi,

I really think we should refactor the OrderItemAssociation entity, in
order to make it more generic.

Right now the entity has the following fields:

<field name="salesOrderId" type="id-ne"/>
<field name="soItemSeqId" type="id-ne"/>
<field name="purchaseOrderId" type="id-ne"/>
<field name="poItemSeqId" type="id-ne"/>

I think that we should at least modify them in this way:

<field name="orderId" type="id-ne"/>
<field name="orderItemSeqId" type="id-ne"/>
<field name="orderIdTo" type="id-ne"/>
<field name="orderItemSeqIdTo" type="id-ne"/>
<field name="orderItemAssocTypeId" type="id-ne"/>

We should also rename it to "OrderItemAssoc" and create the new entity
"OrderItemAssocType".
In this way we could link not just a sales order to a purchase order,
but also two sales orders together (for example a replacement order).

I think we should also add the ability to associate sub-item quantities,
maybe adding an OrderItemAssoc.quantity field and adding two
OrderItemAssoc.shipGroupSeqId and OrderItemAssoc.shipGroupSeqIdTo.

So the final entity should look like:

OrderItemAssoc
<field name="orderId" type="id-ne"/> PK
<field name="orderItemSeqId" type="id-ne"/> PK
<field name="shipGroupSeqId" type="id-ne"/> PK
<field name="orderIdTo" type="id-ne"/> PK
<field name="orderItemSeqIdTo" type="id-ne"/> PK
<field name="shipGroupSeqIdTo" type="id-ne"/> PK
<field name="orderItemAssocTypeId" type="id-ne"/> PK
<field name="quantity" type="floating-point"/>

Does it make sense?
If it's ok for all of you, I could take care of this migration since I'm
working on the return of type "replacement" and on drop shipments and
I'll need some of this features soon.

Jacopo

Reply | Threaded
Open this post in threaded view
|

Re: Rewriting the OrderItemAssociation entity

David E Jones-2

The restructuring of the entity to make it more flexible sounds fine.

The first question though would whether or not anyone is using this.  
If so it's a bit of a pain because data migration is needed for the  
update.

The pattern for this sort of update would be:

1. change the entity name from OrderItemAssociation to  
OldOrderItemAssociation
2. set the table name on the entity to ORDER_ITEM_ASSOCIATION
3. create a service to move data from the OldOrderItemAssociation  
entity to the new OrderItemAssoc entity

-David


On Dec 5, 2006, at 12:25 AM, Jacopo Cappellato wrote:

> Hi,
>
> I really think we should refactor the OrderItemAssociation entity,  
> in order to make it more generic.
>
> Right now the entity has the following fields:
>
> <field name="salesOrderId" type="id-ne"/>
> <field name="soItemSeqId" type="id-ne"/>
> <field name="purchaseOrderId" type="id-ne"/>
> <field name="poItemSeqId" type="id-ne"/>
>
> I think that we should at least modify them in this way:
>
> <field name="orderId" type="id-ne"/>
> <field name="orderItemSeqId" type="id-ne"/>
> <field name="orderIdTo" type="id-ne"/>
> <field name="orderItemSeqIdTo" type="id-ne"/>
> <field name="orderItemAssocTypeId" type="id-ne"/>
>
> We should also rename it to "OrderItemAssoc" and create the new  
> entity "OrderItemAssocType".
> In this way we could link not just a sales order to a purchase  
> order, but also two sales orders together (for example a  
> replacement order).
>
> I think we should also add the ability to associate sub-item  
> quantities, maybe adding an OrderItemAssoc.quantity field and  
> adding two OrderItemAssoc.shipGroupSeqId and  
> OrderItemAssoc.shipGroupSeqIdTo.
>
> So the final entity should look like:
>
> OrderItemAssoc
> <field name="orderId" type="id-ne"/> PK
> <field name="orderItemSeqId" type="id-ne"/> PK
> <field name="shipGroupSeqId" type="id-ne"/> PK
> <field name="orderIdTo" type="id-ne"/> PK
> <field name="orderItemSeqIdTo" type="id-ne"/> PK
> <field name="shipGroupSeqIdTo" type="id-ne"/> PK
> <field name="orderItemAssocTypeId" type="id-ne"/> PK
> <field name="quantity" type="floating-point"/>
>
> Does it make sense?
> If it's ok for all of you, I could take care of this migration  
> since I'm working on the return of type "replacement" and on drop  
> shipments and I'll need some of this features soon.
>
> Jacopo
>