Letractively / mandubian-play-crud-siena

Automatically exported from code.google.com/p/mandubian-play-crud-siena
0 stars 0 forks source link

MaxSize broken again in play 1.2.1 #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Same as issue 5 which was fixed and closed

What is the expected output? What do you see instead?
Same as issue 5

What version of the product are you using? On what operating system?
crudsiena 1.6_play1.2

Please provide any additional information below.
Tried on Chrome and Firefox OSX
Using JQuery 1.4.2 as per issue 7

THANKS FOR A GREAT MODULE!!

Original issue reported on code.google.com by andrew.m...@gmail.com on 16 May 2011 at 10:51

Attachments:

GoogleCodeExporter commented 9 years ago
Digging a little deeper, I see that it does work with @Max(200) where @Max is 
siena's own annotation.
Not sure if this therefore a bug?

Original comment by andrew.m...@gmail.com on 18 May 2011 at 12:06

GoogleCodeExporter commented 9 years ago
I'm going to deliver a new version asap from current trunk.
Roch corrected this issue very recently.
If you can test current trunk, don't hesitate, it should correct your problem 
(and validate it works ;) )

Original comment by pascal.v...@gmail.com on 18 May 2011 at 7:09

GoogleCodeExporter commented 9 years ago
I tried the trunk and it works like it says on the tin!  Thanks!  I would have 
tried that before, but I thought it had been fixed a long time ago (from one of 
the posts I read somewhere).  So apologies for not trying the obvious.
THANKS AGAIN FOR THE MODULE.  IT IS THE REASON I USE PLAY FRAMEWORK.

Original comment by andrew.m...@gmail.com on 19 May 2011 at 8:47

GoogleCodeExporter commented 9 years ago
Great!
Thanks for using this module!
Keep an eye on it because I'm going to release very soon play-siena/crudsiena 
v2.0.0 with new siena v1.0.0. The code has been refactored completely to make 
it compliant to play v1.2.1 features. 
Basically, the not-very-well-designed SienaSupport class will be replaced by a 
Model class (in the same design as Play JPA support) and now Siena models will 
be enhanced and you won't need to override all() functions anymore (if 
everything works as I expect). 
It brings support for H2/Mysql/Postgres + GAE which is quite good for people 
developing apps on different platforms ;)
And it will bring all Siena v1.0.0 new features...

Original comment by pascal.v...@gmail.com on 19 May 2011 at 8:56