Opened 5 years ago

Closed 5 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 5 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 5 years ago by Aymeric Augustin

Description: modified (diff)

Changed 5 years ago by Preston Holmes

Attachment: attr-fix.diff added

comment:2 Changed 5 years ago by Preston Holmes

Has patch: set
Triage Stage: UnreviewedReady for checkin

comment:3 Changed 5 years ago by Claude Paroz

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

What about a test?

comment:4 Changed 5 years ago by Claude Paroz

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