Re: [Django] #5335: Add an append method to ErrorDict

2013-08-21 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  closed
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:  duplicate
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by mjtamlyn):

 * status:  new => closed
 * resolution:   => duplicate


Comment:

 I agree that this is not the best way forwards. The ticket #20867 is now
 covering the `Form.add_errors()` approach. I'm going to close this ticket
 as a "duplicate" cos I think that's closest to what's happening.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.1ed667bfa96937a4cfeb4964cefc0d0e%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2013-08-21 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by alasdair):

 * cc: alasdair (added)


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.1f7626dbf3a3daff40502d4b544f8cef%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2013-08-20 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by loic84):

 @alasdair: that looks like a nasty incompatibility with no obvious
 workaround.

 The only reason I fixed this despite #20867 was so internally it wouldn't
 be needed to check if keys existed, it's clearly not worth breaking
 existing templates over this.

 I think we should close this ticket as wontfix, especially now that
 another ticket addresses the main issue.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.a2473d8d5eb45afb2a9e895167b9cd0b%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2013-08-20 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by simon29):

 Check out the [https://groups.google.com/forum/#!topic/django-
 developers/rTbfg3JtLkA related discussion] and also ticket #20867

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.50a4c131358b39ccf8af7a936113d14f%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2013-08-20 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by alasdair):

 Looping through defaultdicts in templates can cause problems. See #16335.
 I believe this change will stop the following template snippet from
 working.

 {{{
 
   {% for field, errors in form.errors.items %}
{{ field }}: {{errors}}
   {% endford %}
 
 }}}

 I realise it's a slightly artificial example, and that usually you would
 use `{{ form.errors }}`.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.f9950760824f449bf4d982da6bc0%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2013-08-16 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by loic84):

 Actually there is merit in making `ErrorDict` inherit from `defaultdict`.

 PR https://github.com/django/django/pull/1481

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.c952f14ab267c34bfbec27b12344b917%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2013-08-16 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by loic84):

 The problem is that `ErrorDict` should be `ErrorList` agnostic. The latest
 patch hardcodes the `ErrorList` type which can be customized at the form
 level with the `error_class` argument of `Form.__init__()` .

 The `Form.add_errors()` method discussed
 [https://code.djangoproject.com/ticket/20867#comment:4 here] is IMO a much
 better approach.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/089.750be6201b4adb2553764d6ac54db7b2%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: [Django] #5335: Add an append method to ErrorDict

2012-11-15 Thread Django
#5335: Add an append method to ErrorDict
-+-
 Reporter:  Thomas Güttler   |Owner:
   |  jkocherhans
 Type:  New feature  |   Status:  new
Component:  Forms|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  append errodict  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by anonymous):

 Check [http://www.google.com]

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: [Django] #5335: Add an append method to ErrorDict

2011-09-11 Thread Django
#5335: Add an append method to ErrorDict
-+-
   Reporter:  Thomas |  Owner:  jkocherhans
  Güttler  | Status:  new
   Type:  New|  Component:  Forms
  feature|   Severity:  Normal
  Milestone:  1.4|   Keywords:  append errodict
Version:  SVN|  Has patch:  1
 Resolution: |Needs tests:  0
   Triage Stage:  Accepted   |  Easy pickings:  0
Needs documentation:  0  |
Patch needs improvement:  1  |
  UI/UX:  0  |
-+-

Comment (by kmtracey):

 Current patch (form_error_append_r16741.diff) will auto-create an
 `ErrorList()` object for a field's error list even if the including form
 has over-ridden the form's error class
 (https://docs.djangoproject.com/en/1.3/ref/forms/api/#customizing-the-
 error-list-format). If we are going to add this (and it would be nice) it
 needs to be done in a way that will use the form's custom error class if
 it has been defined. This case should also be tested, in a test similar
 to:
 
https://code.djangoproject.com/browser/django/tags/releases/1.3.1/tests/regressiontests/forms/tests/error_messages.py#L199

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

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



Re: [Django] #5335: Add an append method to ErrorDict

2011-09-10 Thread Django
#5335: Add an append method to ErrorDict
-+-
   Reporter:  Thomas |  Owner:  jkocherhans
  Güttler  | Status:  new
   Type:  New|  Component:  Forms
  feature|   Severity:  Normal
  Milestone:  1.4|   Keywords:  append errodict
Version:  SVN|  Has patch:  1
 Resolution: |Needs tests:  0
   Triage Stage:  Accepted   |  Easy pickings:  0
Needs documentation:  0  |
Patch needs improvement:  1  |
  UI/UX:  0  |
-+-
Changes (by bronger):

 * cc: bronger (added)


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

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



Re: [Django] #5335: Add an append method to ErrorDict

2011-09-09 Thread Django
#5335: Add an append method to ErrorDict
-+-
   Reporter:  Thomas |  Owner:  jkocherhans
  Güttler  | Status:  new
   Type:  New|  Component:  Forms
  feature|   Severity:  Normal
  Milestone:  1.4|   Keywords:  append errodict
Version:  SVN|  Has patch:  1
 Resolution: |Needs tests:  0
   Triage Stage:  Accepted   |  Easy pickings:  0
Needs documentation:  0  |
Patch needs improvement:  1  |
  UI/UX:  0  |
-+-
Changes (by julien):

 * ui_ux:   => 0
 * easy:   => 0


Comment:

 Thanks a lot for updating the patch. I showed it to Alex and he wondered
 if you could write a test that verifies that doing errors['foo'] without
 actually altering it doesn't cause any problem?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

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



Re: [Django] #5335: Add an append method to ErrorDict

2011-02-21 Thread Django
#5335: Add an append method to ErrorDict
+---
   Reporter:  Thomas Güttler  | Owner:  
jkocherhans
 Status:  new   | Milestone:  
1.4
  Component:  Forms |   Version:  
SVN
 Resolution:|  Keywords:  
append errodict
   Triage Stage:  Accepted  | Has patch:  1 
 
Needs documentation:  0 |   Needs tests:  0 
 
Patch needs improvement:  1 |  
+---
Changes (by lukeplant):

  * needs_better_patch:  0 => 1
  * milestone:  1.3 => 1.4


Comment:

 Looks like Joseph didn't remember :-( This is a feature, so will have to
 wait again. Patch also needs updating (especially to use unittest rather
 than doctests).

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

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



Re: [Django] #5335: Add an append method to ErrorDict

2010-02-21 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
jkocherhans
Status:  new   | Milestone:  1.3

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Changes (by jkocherhans):

  * owner:  nobody => jkocherhans
  * status:  reopened => new
  * milestone:  1.2 => 1.3

Comment:

 This a a feature, so I'm bumping it to 1.3. Sorry. I'm assigning it to
 myself it to increase the chances that I'll remember it for the next
 cycle.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

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



Re: [Django] #5335: Add an append method to ErrorDict

2010-02-08 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone:  1.2

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Changes (by miracle2k):

 * cc: miracle2k (added)

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

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



Re: [Django] #5335: Add an append method to ErrorDict

2009-12-28 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone:  1.2

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Comment (by Honza_Kral):

 This approach is in conflict with #4752 ([6142]).

 The method would either have to be on the Form class (not entirely
 insane). Another way how to propagate your error onto a field is using
 ComplexValidators present in model-validation branch, that way you don't
 have to worry about the internals of ErrorDict...

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

--

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




Re: [Django] #5335: Add an append method to ErrorDict

2009-03-31 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone:  1.2

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Comment (by simon29):

 Definitely. Often views need to perform more involved steps *after* the
 form is validated; which may fail, and ultimately mean there was a problem
 with what was entered in the form.

 There needs to be an easy, documented, encouraged way of appending errors
 to NON_FIELD_ERRORS as well as to specific fields; post-validation (ie
 from the view).

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~--~~~~--~~--~--~---



Re: [Django] #5335: Add an append method to ErrorDict

2009-03-31 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone:  1.2

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Changes (by jkocherhans):

  * milestone:  1.1 => 1.2

Comment:

 Bumping to 1.2. For the record, I have to do this all the time and I'd
 like to see something like this, but I'm not entirely sold on this way
 yet.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~--~~~~--~~--~--~---



Re: [Django] #5335: Add an append method to ErrorDict

2009-01-04 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone: 

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Comment (by Alex):

 There's actually a slight disconnect between the docs and the code, the
 docs say an ErrorList is created, but __getitem__ adds a list([]), I think
 an ErrorList would be the correct behavior.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~--~~~~--~~--~--~---



Re: [Django] #5335: Add an append method to ErrorDict

2009-01-04 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone: 

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Comment (by kratorius):

 Even better, I used {{{setdefault}}} so in the end is a one-line patch.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~--~~~~--~~--~--~---



Re: [Django] #5335: Add an append method to ErrorDict

2009-01-04 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone: 

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Comment (by Alex):

 The docs look ok by me, 2 small things though, instead of {{{
 self.has_key(field) }}} do {{{ field in self }}}.  Also there's no need to
 use super() for setitem since it isn't overode in the subclass, you can
 actually just use setattr(self, field, []).

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~--~~~~--~~--~--~---



Re: [Django] #5335: Add an append method to ErrorDict

2009-01-04 Thread Django
#5335: Add an append method to ErrorDict
---+
  Reporter:  Thomas Güttler   | Owner:  
nobody 
Status:  reopened  | Milestone: 

 Component:  Forms |   Version:  SVN

Resolution:|  Keywords:  append 
errodict
 Stage:  Design decision needed| Has_patch:  1  

Needs_docs:  0 |   Needs_tests:  0  

Needs_better_patch:  0 |  
---+
Changes (by kratorius):

  * keywords:  => append errodict
  * summary:  newforms: Custom validation. => Add an append method to
  ErrorDict

Comment:

 Attached patch implements {{{ErrorDict}}} as a {{{defaultdict}}} (can't
 inherit directly from {{{defaultdict}}} since it's new in 2.5 or 2.4,
 can't remember, but definitely not in 2.3). It would be helpful if someone
 could review the documentation part of the patch since I'm not a native
 english speaker.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~--~~~~--~~--~--~---