You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Frédéric Donckels (JIRA)" <ji...@apache.org> on 2011/06/30 14:26:28 UTC

[jira] [Updated] (WICKET-3850) Use of Application inside PropertiesFactory is inconsistent

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

Frédéric Donckels updated WICKET-3850:
--------------------------------------

    Description: 
PropertiesFactory is taking an Application argument in its constructor and then in the load(..) method it does: Application.get().xxxx.
This seems inconsistent to me.

In addition, when looking at the uses of the application instance, it could just have an IResourceSettings parameter, rather than a full blown Application instance.


  was:
PropertiesFactory is taking an Application argument in its constructor and then in the load(..) method it does: Application.get().xxxx.
This seems inconsistent to me.

In addition, when looking at the uses of the application instance, it could just have a ResourceSettings parameter, rather than a full blown Application instance.



> Use of Application inside PropertiesFactory is inconsistent
> -----------------------------------------------------------
>
>                 Key: WICKET-3850
>                 URL: https://issues.apache.org/jira/browse/WICKET-3850
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket-core
>    Affects Versions: 1.4.17, 1.5-RC5.1
>            Reporter: Frédéric Donckels
>            Priority: Minor
>
> PropertiesFactory is taking an Application argument in its constructor and then in the load(..) method it does: Application.get().xxxx.
> This seems inconsistent to me.
> In addition, when looking at the uses of the application instance, it could just have an IResourceSettings parameter, rather than a full blown Application instance.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira