On Fri, Apr 3, 2009 at 12:07 PM, Simon Greenwood <sfgreenw...@gmail.com>wrote:
> > On Apr 3, 4:05 pm, Ayaz Ahmed Khan <a...@ayaz.pk> wrote: > > On 03-Apr-09, at 7:43 PM, simong wrote: > > > > > if request.method == 'POST': > > > productform = ProductForm(request.POST, instance=product) > > > productform.user = user > > > if productform.is_valid: > > > > `is_valid` is a method bound to the BaseForm class. The expression in > > the snippet above _will_ evaluate to True because it will return a > > reference to the bound method, as opposed to return nothing, but will > > not, of course, execute the referenced method. > > > > I have now removed the instance, which I suspected wasn't necessary > anyway, and I still get the same attribute error. It also occurs if I > create a new product, and there is still this lack of useful error > information. > > It would appear that is_valid doesn't return False, or at least not > True as I seem to be reaching a contradictory traceback. > Right, that's what Ayaz was trying to say. is_valid is a method. productform.is_valid will always return true -- it's a reference to a method. You need to call the method: productform.is_valid(). The code that you have right now is not even attempting to execute the validation logic. Karen --~--~---------~--~----~------------~-------~--~----~ 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 django-users+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/django-users?hl=en -~----------~----~----~----~------~----~------~--~---
