[ 
http://www.stripesframework.org/jira/browse/STS-483?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tim Fennell updated STS-483:
----------------------------

    Fix Version/s: Release 1.5

Seems reasonable.  Except the log - I tend to think subclasses should make 
their own log so that the class name emitted in the log is the actual class 
producing the message.

> Refactor DefaultActionBeanPropertyBinder for easier extending
> -------------------------------------------------------------
>
>                 Key: STS-483
>                 URL: http://www.stripesframework.org/jira/browse/STS-483
>             Project: Stripes
>          Issue Type: Improvement
>          Components: Validation
>    Affects Versions: Release 1.5
>            Reporter: Karel Kolman
>            Assignee: Tim Fennell
>             Fix For: Release 1.5
>
>
> The DefaultActionBeanPropertyBinder has a lot of useful code inside. I'd like 
> it to be more easily extendible.
> Could getters for some of the private class properties be created ? Like for 
> the configuration property
> protected Configuration getConfiguration() {
>     return this.configuration;
> }
> The same for the static Log instance.
> There is also a Row class inside the source file. Could it be made into a 
> nested class of DefaultActionBeanPropertyBinder (for use in inherited classes)
> public class DefaultActionBeanPropertyBinder {
> ..
>    protected static class Row {
>     ..
>   }
> }

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

        

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to