[jira] [Updated] (OFBIZ-10604) Isolate EntityOperator and EntityConditionValue from EntityConditionBase

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

[jira] [Updated] (OFBIZ-10604) Isolate EntityOperator and EntityConditionValue from EntityConditionBase

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-10604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gil Portenseigne updated OFBIZ-10604:
-------------------------------------
    Description:
While working on OFBIZ-10593 with [~mthl] we discovered a design issue around {{EntityConditionBase}} inheritence.
 Indeed {{EntityOperator}} and {{EntityConditionValue}} inherits from {{EntityConditionBase}}, even if they can't be considered as proper "entity conditions".

It should be determined what was the need behind this inheritance and refactor it to isolate that classes that do not appear to be "entity conditions".

  was:
While working on OFBIZ-10593 with [~mthl] we discovered a design issue around {{EntityConditionBase}} inheritence.
 Indeed {{EntityOperator}} and {{EntityConditionValue}} inherits from {{EntityConditionBase}}, even if they can't be considered as proper "entity conditions".
 It should be determined what was the need behind this inheritance and refactor it to isolate that classes that do not appear to be "entity conditions".


> Isolate EntityOperator and EntityConditionValue from EntityConditionBase
> -------------------------------------------------------------------------
>
>                 Key: OFBIZ-10604
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10604
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Gil Portenseigne
>            Assignee: Gil Portenseigne
>            Priority: Major
>
> While working on OFBIZ-10593 with [~mthl] we discovered a design issue around {{EntityConditionBase}} inheritence.
>  Indeed {{EntityOperator}} and {{EntityConditionValue}} inherits from {{EntityConditionBase}}, even if they can't be considered as proper "entity conditions".
> It should be determined what was the need behind this inheritance and refactor it to isolate that classes that do not appear to be "entity conditions".



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)