Changeset 6450

Timestamp:

10/03/07 20:12:00 (1 year ago)

Author:

mtredinnick

Message:

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.

Files:

• django/trunk/django/newforms/widgets.py (modified) (2 diffs)

django/trunk/django/newforms/widgets.py

@@ -8 +8 @@
-    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
@@ -32 +34 @@
-        else:
-            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):