Skip to content
Browse files

Added a __deepcopy__() method to the Widget class in order to avoid a…

… number of easy-to-trigger problems when copying Widget subclasses. Subclasses which are intended to have extra mutable fields should override this method. Refs #5505.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
1 parent 4d8b51f commit b1cc3318c7ac740784b556e16bae594bc20010f4 @malcolmt malcolmt committed
Showing with 8 additions and 0 deletions.
  1. +8 −0 django/newforms/
8 django/newforms/
@@ -7,7 +7,9 @@
except NameError:
from sets import Set as set # Python 2.3 fallback
+import copy
from itertools import chain
from django.utils.datastructures import MultiValueDict
from django.utils.html import escape
from django.utils.translation import ugettext
@@ -32,6 +34,12 @@ def __init__(self, attrs=None):
self.attrs = {}
+ def __deepcopy__(self, memo):
+ obj = copy.copy(self)
+ obj.attrs = self.attrs.copy()
+ memo[id(self)] = obj
+ return obj
def render(self, name, value, attrs=None):
Returns this Widget rendered as HTML, as a Unicode string.

0 comments on commit b1cc331

Please sign in to comment.
Something went wrong with that request. Please try again.