Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1249 closed defect (fixed)

[magic-removal][patch] manipulator_validator_unique

Reported by: the.twentysixth@… Owned by: Adrian Holovaty
Component: Validators Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


self.original_object maybe None (e.g. in the case of AddManipulator)

Index: django/db/models/fields/
--- (revision 2051)
+++ (working copy)
@@ -32,7 +32,8 @@
         old_obj = self.manager.get_object(**{lookup_type: field_data})
     except ObjectDoesNotExist:
-    if hasattr(self, 'original_object') and self.original_object._get_pk_val() 
== old_obj._get_pk_val():
+    if hasattr(self, 'original_object') and hasattr(self.original_object, '_get
_pk_val') and \
+           self.original_object._get_pk_val() == old_obj._get_pk_val():
     raise validators.ValidationError, _("%(optname)s with this %(fieldname)s al
ready exists.") % {'optname': capfirst(opts.verbose_name), 'fieldname': f.verbos

Change History (1)

comment:1 Changed 13 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(In [2074]) magic-removal: Fixed #1249 -- Fixed bug in manipulator_validator_unique

Note: See TracTickets for help on using tickets.
Back to Top