Hi Rishi, Benjami,

I agree, it seems like something unfinished (from pre Apache era) and I can't 
see what it adds as is

Handling it at the service level seems sufficient to me. No need for an useless 
and confusing definition in data model.
I'd rather add a comment there (in the data model)  to document the service 
level functionality.
I guess it was somehow the initial purpose of this incomplete relation 

I also agree with Rishi about generalising a such cleaning, but not w/o adding 



Le 06/04/2018 à 15:07, Rishi Solanki a écrit :

The base idea behind such data modeling to force business user to use the
category attribute based on category type. But it does not force at entity
level as such does not mean at all at db level.
IMO we can remove such relations from db layer as anyways we are going to
maintain such constraints on service layer only.

Alternatively, we should rethink on this type of modeling if we want to
maintain relationship somehow. In first look it seems that we should remove
such occurrences.

I would prefer to take others opinion on this, I may be wrong.

Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847

On Thu, Apr 5, 2018 at 8:57 PM, Benjamin Jugl <benjamin.j...@ecomify.de>

Hi everyone,

while I was working onJira Issue OFBIZ-10327 <
https://issues.apache.org/jira/browse/OFBIZ-10327> and 10328 <
https://issues.apache.org/jira/browse/OFBIZ-10328> I stumbled across the
entity definition for ProductCategoryAttribute  (in

The last lines of the definition state:

   <relation type="many" rel-entity-name="ProductCategoryTypeAttr">
         <key-map field-name="attrName"/>

I am quite new to this, but I think this statement does not make sense.
a) The primary key of ProductCategoryAttribute is composite. Just the
field "attrName" does not suffice for a relation to another table, if I am
not mistaken.
b) I am not quite sure about the nature of a relation between
CategoryAttribute and CategoryTypeAttribute.

Does anyone have background knowledge to this and can perhaps explain or
even verify that this is obsolete?

Many thanks, yours Benjamin

Reply via email to