[jira] [Commented] (OFBIZ-7112) EntityUtilProperties

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

[jira] [Commented] (OFBIZ-7112) EntityUtilProperties

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-7112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15299886#comment-15299886 ]

Deepak Dixit commented on OFBIZ-7112:
-------------------------------------

Thanks Jacques, I miss understood this. I agree with Wai on this we need to return empty/null value if property overridded in database.

I am just curious to know that why we are returning "isExistInDb". I think if we are using pattern to override property file value by database with empty value then we don't need to return isExistInDb flag.

{code}
<String, String> results = new HashMap<>();
        results.put("isExistInDb", "N");
        results.put("value", "");
{code}

> EntityUtilProperties
> --------------------
>
>                 Key: OFBIZ-7112
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7112
>             Project: OFBiz
>          Issue Type: Bug
>          Components: ALL COMPONENTS
>    Affects Versions: Trunk
>            Reporter: Wai
>         Attachments: OFBIZ-7112.patch, OFBIZ-7112.patch
>
>
> Ofbiz reads properties from either a properties file or the entity:SystemProperty. The way it works previously is that ofbiz reads from the entity:SystemProperty first and if there is no value associated with the target propertyname, it would then locate the value from the relevant properties file.
> In other words, if there is a database entry for a property, the database entry should override the associated properties file.
> The issue is that if a database entry exist but the value is empty, it would look for a value from the properties file.  It should not do so.  If a database entry exists for the propertyname of interest, the value should be taken from the database even if it holds an empty value.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)