Re: svn:ignore for new management package

2007-08-23 Thread John Shaffer

On 8/23/07, Russell Keith-Magee <[EMAIL PROTECTED]> wrote:
> Thanks. I've added a few missing svn:ignore properties, including the
> ones you mentioned, in [5993].

Thank you. There's one other directory that should be changed -
tests/regressiontests/test_client_regress has an svn:ignore of
"__init__.pyc", but actually includes several other files.

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@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-developers?hl=en
-~--~~~~--~~--~--~---



Re: forms and designers

2007-07-08 Thread John Shaffer

We use this in Satchmo:



Discounts
Discount code {{
form.discount }}
{% if form.discount.errors %}*** {{
form.discount.errors|join:", " }}{% endif %}




Does that give you enough control?

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@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-developers?hl=en
-~--~~~~--~~--~--~---



Re: newforms issues

2007-05-21 Thread John Shaffer

On 5/21/07, Malcolm Tredinnick <[EMAIL PROTECTED]> wrote:
> > #3896 - pass value to field specific clean function [2]
>
> Grr...don't put two issues in one ticket!
>
> I'm -1 on the first part because it's an unnecessary backwards
> incompatibility change for the most part. It's not like it's a massive
> performance improvement in an area that is currently slow.
>
> Definitely +1 on the second part, though. Removing the first assignment
> to self.cleaned_data would make a lot of sense.

This change would make things cleaner and obvious (as well as being
the best way to fix the bug of cleaned_data having uncleaned values).
I personally don't mind changing code and retesting for that purpose,
but since many others do I'm +0.

But I wanted to point out that the two changes are closely related,
and both are needed to fix the bug. Applying the second change without
the first would break any code that uses a clean_* method right now,
and applying the first without the second would also break code as
well as not actually fixing the bug.

I am not sure that I didn't miss something obvious, though. If I did,
please correct me, I would like to properly understand the proposed
change.

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@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-developers?hl=en
-~--~~~~--~~--~--~---



Re: Changeset [5231] -- do_clean_*() is awkward naming

2007-05-14 Thread John Shaffer

On 5/14/07, Malcolm Tredinnick <[EMAIL PROTECTED]> wrote:
> That was exactly why I didn't change clean_data (my initial "fix" was
> just to name it cleaned_data). It's already used in lots of code, so the
> impact of change it is larger. Changing the undocumented feature is less
> disruptive. Unnecessarily intrusive backwards-incompat changes seem
> mean.

For me, an AttributeError from clean_data is a much more friendly
change than silently ignoring code like:

def clean_email(self):
"""Prevent account hijacking by disallowing duplicate emails."""

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@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-developers?hl=en
-~--~~~~--~~--~--~---