Opened 12 years ago

Closed 12 years ago

#18002 closed Bug (fixed)

Typo in ReverseSingleRelatedObjectDescriptor

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'

Attachments (1)

attr-fix.diff (652 bytes ) - added by Preston Holmes 12 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by Aymeric Augustin, 12 years ago

Description: modified (diff)

by Preston Holmes, 12 years ago

Attachment: attr-fix.diff added

comment:2 by Preston Holmes, 12 years ago

Has patch: set
Triage Stage: UnreviewedReady for checkin

comment:3 by Claude Paroz, 12 years ago

Needs tests: set
Triage Stage: Ready for checkinAccepted
Type: UncategorizedBug

What about a test?

comment:4 by Claude Paroz, 12 years ago

Resolution: fixed
Status: newclosed

In [17904]:

Fixed #18002 -- Fixed typo in attribute name in ReverseSingleRelatedObjectDescriptor.

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