I'll think about the decorator.

The IIdList is Johan's, the adaptor is mine just to provide easier usage. So, Johan, let us know what you think of this.

Eelco

Jonathan Locke wrote:


eelco,

it's getting on towards 2AM, so i may not have thought this ALL the way through, but i think i have a solution to the ValidationErrorModelDecorator problem.

it's simple and elegant. override getModel() in FormComponent so that a new Model(getRequestString()) is returned if there is a validation error for the component. if you want to take a crack at this while i'm gone... be my guest. also, i'd like to get rid of IIdList and the IdListAdaptor and just have a simple concrete IdList. i don't see the value in that interface. finally, i'd like to get rid of IFormComponentPersistenceManager (which, sadly, i think i may have suggested some weeks ago in an email i didn't think through very well... ). in any case, i can do these things on wednesday if you don't get to them. btw, i really like the validation delegate, although i rearranged it a bit to make it a singleton with more restricted scope.

      jon



-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop




-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to