Opened 13 years ago

Last modified 13 years ago

#18002 closed Bug

Typo in ReverseSingleRelatedObjectDescriptor — at Version 1

Reported by: Bradley Ayers <bradley.ayers@…> Owned by: nobody
Component: Database layer (models, ORM) Version: 1.4
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Aymeric Augustin)

This code uses _field instead of field:

def __set__(self, instance, value):
    if instance is None:
        raise AttributeError("%s must be accessed via instance" % self._field.name)

Example:

>>> from django.contrib.auth.models import Permission
>>> Permission.content_type.__set__(None, "foo")
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/Users/brad/work/foo/.env/lib/python2.7/site-packages/django/db/models/fields/related.py", line 356, in __set__
    raise AttributeError("%s must be accessed via instance" % self._field.name)
AttributeError: 'ReverseSingleRelatedObjectDescriptor' object has no attribute '_field'

Change History (2)

comment:1 by Aymeric Augustin, 13 years ago

Description: modified (diff)

by Preston Holmes, 13 years ago

Attachment: attr-fix.diff​ added
Note: See TracTickets for help on using tickets.
Back to Top