[jira] [Created] (OFBIZ-9479) Use FlexibleLocation.resolveLocation() instead of hard-coded path

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

[jira] [Created] (OFBIZ-9479) Use FlexibleLocation.resolveLocation() instead of hard-coded path

Nicolas Malin (Jira)
Devanshu Vyas created OFBIZ-9479:
------------------------------------

             Summary: Use FlexibleLocation.resolveLocation() instead of hard-coded path
                 Key: OFBIZ-9479
                 URL: https://issues.apache.org/jira/browse/OFBIZ-9479
             Project: OFBiz
          Issue Type: Improvement
          Components: ALL COMPONENTS
    Affects Versions: Trunk
            Reporter: Devanshu Vyas
            Priority: Minor


I found some occurrences where file path URL is prepared using the actual path of the file rather than the relative path. For e.g.
{code}
String imgPropertyFullPath = System.getProperty("ofbiz.home") + "/applications/product/config/ImageProperties.xml";
{code}

IMO the location URL of files should be prepared dynamically using relative paths and this can be achieved by using
FlexibleLocation.resolveLocation() method. This method covers all the scenarios and in future, if we want to change anything, then all we need is to update the method and all instances will be fixed.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)