[
http://www.stripesframework.org/jira/browse/STS-483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=11157#action_11157
]
Karel Kolman commented on STS-483:
----------------------------------
You are absolutely right about the log.
I needed to make some adjustments to nested validation logic and ended up
copying the DefaultActionBeanPropertyBinder source in order to make use of its
inner workings, so the refactoring would be a great thing to do (maybe to be
considered for other Default... classes).
> 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
