On 3 oct, 01:34, Michael <[EMAIL PROTECTED]> wrote:
> On Thu, Oct 2, 2008 at 6:23 PM, bruno desthuilliers <
>
>
>
> [EMAIL PROTECTED]> wrote:
>
> > On 2 oct, 16:51, Michael Newman <[EMAIL PROTECTED]> wrote:
> > > Kurczak;
>
> > > Short of this is, catching specific errors in the save method and
> > > displaying them to the user are hard and you might be regurgitating
> > > items back to the user that they really don't have any business seeing
> > > (SQL errors, programming errors, etc.).
>
> > So far so good.
>
> > > The best way to check for the
> > > proper values is to do it at the form level.
>
> > (snip)
>
> > > In this way you are avoiding the model validation on the save method
>
> > Err...
>
> > The best way is to do it at *both* levels (yes, not totally DRY,
> > but...). The model is responsible for data correctness and integrity.
> > You definitively don't want to "avoid model validation" and allow a
> > model object with incorrect data to be saved.
>
> > My 2 cents.
>
> Sigh. Model level validation is going to get into Django at some point.
There even was a short-lived first attempt at it.
> However, doing this type of checking is not model level validation,
Which "type of checking" ? You wrote "The best way to check for the
proper values is to do it at the form level.". Are you pretending that
"proper values" are not the business of the models ???
> and no
> matter where it is put, will not become model level validation. Here you are
> validating input, checking and cleaning it
> and reporting back to the user.
> The best way to do that is in the form's clean methods.
Not all validation can be done by the controller, and models are not
necessarily accessed thru one single controller. Heck, there's just no
point having models at all if the controllers have to micro-manage
them. So while the controller indeed has to hand the model sensible
data (which usually requires some conversion, which themselves are a
first validation stage) and
to provide usefull feedback to the user (which usually imply handling
a first validation stage), this doesn't mean the models should just
take whatever the controller fed him and be happy with it. I can
assure you - and believe me, I learned that lesson the hard way - that
bypassing model-level validation "because the controller already did
it" is *not* an option. period. At least in my team (I have better
things to do than trying to recover from corrupted data in a
customer's mission-critical app, thanks). Now if you're willing to
take the risk, well, it's your life, but at least pity other
developpers and do not advise doing so.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django users" group.
To post to this group, send email to django-users@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-users?hl=en
-~----------~----~----~----~------~----~------~--~---
