[jira] [Commented] (OFBIZ-9394) FileUtil.java: New method to get absolute path from given relative path on base of ofbiz.home dir

["Jacques Le Roux (JIRA)" <ji...@apache.org>]

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

Jacques Le Roux commented on OFBIZ-9394:
----------------------------------------

That's right Deepak, but I find Martin's proposition logical because we somehow miss a such function in it's most logical place: FileUtil class. Now I don't have a strong opinion about that. Maybe we could discuss on dev ML...

> FileUtil.java: New method to get absolute path from given relative path on base of ofbiz.home dir
> -------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-9394
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9394
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Martin Becker
>            Priority: Trivial
>         Attachments: OFBIZ-9394_absolutePathOfbizHome.patch
>
>
> This patch adds a method getAbsolutePathWithOfbizHome(String filename) to FileUtil.java. It returns by the given filename the absolute path starting with "ofbiz.home". Note that a quite similar method already exists in UtilURL.fromOfbizHomePath(String filename).
> While both methods share similar code, the idea of extracting the duplicated code into another method or even class results in unnecessary imports (one or the other has to hold such a method and the other has to import the holding class) and thus has not been and shouldn't be done in the future.



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