[jira] Resolved: (WW-493) Refactor Configuration classes

["Don Brown (JIRA)" <ji...@apache.org>]

     [ http://issues.apache.org/struts/browse/WW-493?page=all ]

Don Brown resolved WW-493.
--------------------------

    Resolution: Fixed

The configuration system has been redesigned.  The settings are now folded into the regular xwork configuration system, allowing properties to be set in xml or via any other configuration provider.

> Refactor Configuration classes
> ------------------------------
>
>                 Key: WW-493
>                 URL: http://issues.apache.org/struts/browse/WW-493
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Configuration
>            Reporter: Bill Lynch
>         Assigned To: Don Brown
>            Priority: Minor
>             Fix For: 2.0.2
>
>
> The Configuration classes (see com\opensymphony\webwork\config\*.java) are poorly designed and should be refactored.
> Suggestions:
> 1) Are there XWork classes we can use? Seems like the XWork configs should be flexible enough to say something like "here are my config files, use them...".
> 2) Move webwork.config.XMLActionConfiguration to com.opensymphony.webwork.config.XMLActionConfiguration -- or use the XWork XML config stuff (seems like it should be taken care of in there?). I don't know why that class is in the webwork.config (old) package when it's pretty integral to everything.
> 3) Make Configuration abstract and fold in DefaultConfiguration into it. There is no reason to have all the *Impl(..) methods when the class should just be abstract anyway. All the code used in DefaultConfiguration can be easily moved into Configuration.
> 4) Remove get(String) method that returns an object. Is this even really used? We should just stipulate that a property is a String only.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira