Hi Dhanji,
The validation work is done by bval-guice. Exception are catched and
errors put as request attributes so the template can show them. It can
validate the forms as class attribute and and method parameter. Another
option would have been to rely on AOP. In that case, working with an
external module would have been possible, but that's not the road I have
taken (I am not sure all the functions I have implemented would have
been possible with AOP).
There is also an extra jar (commons-fileupload) for the multipart form.
For this, I can move that in an extra tiny module without problem.
Before doing anything, I will wait on you further reviews.
Thx again, Eric
On 17/05/2013 22:30, Dhanji R. Prasanna wrote:
> Hey, sorry about that. I have been away for a month or more on other
> work related stuff. Let's try to merge it this week. Ideally if there
> are any new jars added, I would like them in a separate module rather
> that sitebricks-core.
>
> Just keep bugging me I will make it happen =) Thanks for your patience.
>
>
> On Wed, May 15, 2013 at 3:20 PM, Eric Charles (GMail)
> <
eric.umg...@gmail.com <mailto:
eric.umg...@gmail.com>> wrote:
>
> Hi,
>
> I continued to work on that branch which has now the following features:
>
> - validation (in various flavors)
> - errors widget
> - form and multipartform binding as method param
> - support for javax.inject.Named
> - @At on pages method (same logic for subpath as for the @Service)
>
> (The page method has broken the
> RestfuWebServiceWithSubpaths2A__cceptanceTest, will fix it asap).
>
> I am just now a bit worried if all those changes get a chance to go
> in master. This is open since so long...
>
> Thx, Eric
>
>
>
> On 19/04/2013 07:19, Eric Charles (GMail) wrote:
>
> Thx for your early comment on pull-request 54.
> I have answered to these and pushed back the changed.
>
> Let's keep iterating,
>
> Eric
>
>
> On 18/04/2013 16:23, Eric Charles (GMail) wrote:
>
> On 18/04/2013 16:07, Dhanji R. Prasanna wrote:
>
> Hey so sorry, I have had a hellish few weeks at work.
>
>
> Welcome to the club :)
>
> Will have a look
> this weekend!
>
>
>
> Cool!
>
> Eric
>
> On Thu, Apr 18, 2013 at 8:21 PM, Eric Charles (GMail)
> <
eric.umg...@gmail.com
> <mailto:
eric.umg...@gmail.com>
> <mailto:
eric.umg.charles@__
gmail.com
> <mailto:
eric.umg...@gmail.com>>> wrote:
>
> Hi,
>
> I have done some work on automatic validation (+
> form injection via
> method parameters). The pull request is on [1].
>
> Before going further with changes (for better
> support multipart
> form), I would love to hear your feedbacks,
> especially Dhanji one :)
>
> Thx, Eric
>
> [1]
>
https://github.com/dhanji/____sitebricks/pull/54/files
> <
https://github.com/dhanji/__sitebricks/pull/54/files>
> google-sitebricks+unsubscribe@____
googlegroups.com
> <
http://googlegroups.com>
>
> <mailto:
google-sitebricks%__2Buns...@googlegroups.com
> <mailto:
google-sitebricks%252Buns...@googlegroups.com>__>.
> For more options, visit
>
https://groups.google.com/____groups/opt_out
> <
https://groups.google.com/__groups/opt_out>
> an email to
google-sitebri...@googlegroups.com.