Hi Bernhard,

Yea that sounds good, I'll do it tonight and send across the PR. The
single properties option is a much better idea.

I didn't realise you could put that assert:True on a method as Tim
pointed out - very handy. I think there'll probably be a lot of new
users who would be looking for/expecting a set of LessThan,
GreatherThan, etc, constraints though, so definitely useful to
include.

Daniel

On May 5, 5:55 am, Bernhard Schussek <bschus...@gmail.com> wrote:
> Hi Daniel,
>
> If you could prepare a PR with such constraints, I'd be happy to merge it.
>
> I would prefer if the constraint had only a single option "properties"
> which is also the default option. In case of LessThan, each property
> in that array should be less than all properties after that. So in
> simple cases, you'd write
>
>     /**
>      * @assert:LessThan({"shouldBeLess", "shouldBeGreater"})
>      */
>     class ...
>
> What do you think?
>
> Bernhard

-- 
If you want to report a vulnerability issue on symfony, please send it to 
security at symfony-project.com

You received this message because you are subscribed to the Google
Groups "symfony users" group.
To post to this group, send email to symfony-users@googlegroups.com
To unsubscribe from this group, send email to
symfony-users+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/symfony-users?hl=en

Reply via email to