[jira] [Commented] (WICKET-5713) Move /wicket.properties to /META-INF/wicket/xyz.properties

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

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

ASF subversion and git services commented on WICKET-5713:
---------------------------------------------------------

Commit 6b2a65b0f62ab8cb11aeabdb2bd4c551f7433268 in wicket's branch refs/heads/master from [~mgrigorov]
[ https://git-wip-us.apache.org/repos/asf?p=wicket.git;h=6b2a65b ]

WICKET-5713 Move /wicket.properties to /META-INF/wicket/xyz.properties

Add a check for scheme "wsjar" used by WebSphere.
borrowed-from: https://github.com/ronmamo/reflections/pull/46/files


> Move /wicket.properties to /META-INF/wicket/xyz.properties
> ----------------------------------------------------------
>
>                 Key: WICKET-5713
>                 URL: https://issues.apache.org/jira/browse/WICKET-5713
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 6.17.0
>            Reporter: Martin Grigorov
>            Assignee: Martin Grigorov
>            Priority: Minor
>             Fix For: 7.0.0-M4
>
>
> As found at http://markmail.org/message/gzychpjelr4eshly the current location of wicket.properties (at the root of the classpath entry) is problematic when an uberjar is used because usually the developer doesn't know how org.apache.wicket.IInitializer works and doesn't add code to merge the contents of all wicket.propeties files in the classpath into one.
> But even if the developer tried to merge them then it will fail again because org.apache.wicket.Application#load(Properties) doesn't expect more than one value per key, e.g. separated by comma.
> Garret Wilson suggested to move /wicket.properties files in /META-INF/wicket/ and use more qualified names, e.g. org.apache.wicket.core.properties.
> The first problem is that there are no JRE and Servlet APIs to find such files easily. We need to scan the classpath to be able to find them and load them.
> Is this costly ?
> https://github.com/ronmamo/reflections is wildly used library that provides this functionality (and much more) and it is using the same logic and seems to be quite fast!
> So let's give it a try!
> We should still support /wicket.properties for backward compatibility. We can log a WARN message when such is found and suggest to be moved in /META-INF/wicket/ 



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