Replace hardcoded class names in __repr__-methods

[Mads Jensen]

I noticed there are a few places where __repr__ does not use self.__class__.__name__. I think this is a small clean-up that would make sense to do, but perhaps there was a good reason not to do that (probably they're never subclassed?). The following output is from grep:

django/db/models/query.py:        return "<RawQuerySet: %s>" % self.query
django/db/models/sql/query.py:        return "<RawQuery: %s>" % self
django/db/migrations/state.py:        return "<ModelState: '%s.%s'>" % (self.app_label, self.name)
django/utils/baseconv.py:        return "<BaseConverter: base%s (%s)>" % (len(self.digits), self.digits)
django/core/management/commands/makemessages.py:        return "<TranslatableFile: %s>" % os.sep.join([self.dirpath, self.file])
django/core/serializers/base.py:        return "<DeserializedObject: %s(pk=%s)>" % (

[Tim Graham]

Sure, while there it would be nice to add tests if missing.

[Adiyat Mubarak]

Replying to Mads Jensen:

I noticed there are a few places where __repr__ does not use self.__class__.__name__. I think this is a small clean-up that would make sense to do, but perhaps there was a good reason not to do that (probably they're never subclassed?). The following output is from grep:

django/db/models/query.py:        return "<RawQuerySet: %s>" % self.query
django/db/models/sql/query.py:        return "<RawQuery: %s>" % self
django/db/migrations/state.py:        return "<ModelState: '%s.%s'>" % (self.app_label, self.name)
django/utils/baseconv.py:        return "<BaseConverter: base%s (%s)>" % (len(self.digits), self.digits)
django/core/management/commands/makemessages.py:        return "<TranslatableFile: %s>" % os.sep.join([self.dirpath, self.file])
django/core/serializers/base.py:        return "<DeserializedObject: %s(pk=%s)>" % (

Hi, I just learn to contribute on django project by taking this easy task on this ​PR
Do I need to update this ticket to resolve as fixed? or this ticket will closed automatically?

Thank in advance

[Adiyat Mubarak]

​PR

[Mads Jensen]

Ticket will be closed when a maintainer merges the changes and marks it as solved.

[Tim Graham <timograham@…>]

In 794b7d80:

Refs #27546 -- Tested some repr() methods.

[Tim Graham <timograham@…>]

In 48826aa:

Fixed #27546 -- Removed hardcoded class names in repr() methods.

[Tim Graham <timograham@…>]

In 86ae1d51:

Refs #27546 -- Removed hardcoded class names in repr() methods.

[Tim Graham <timograham@…>]

In ee7ab1b6:

Refs #27546 -- Replaced hardcoded class name in ForNode.repr().