Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#22906 closed New feature (fixed)

Give ModelState a __repr__()

Reported by: Collin Anderson Owned by: nobody
Component: Migrations Version: 1.7-beta-2
Severity: Normal Keywords:
Cc: cmawebsite@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I've gotten a number of InvalidBasesError("Cannot resolve bases for %r" % new_unrendered_models) errors. Could we make a repr for ModelState objects?

Change History (5)

comment:1 by Collin Anderson, 10 years ago

Summary: ModelState.__repr__ → Give ModelState a __repr__()
Type: Uncategorized → New feature

​https://github.com/django/django/pull/2852

comment:2 by Claude Paroz, 10 years ago

Needs tests: set
Triage Stage: Unreviewed → Accepted

comment:3 by Loic Bistuer <loic.bistuer@…>, 10 years ago

Resolution: → fixed
Status: new → closed

In 2572c07cc64177e985b453d0f0b819e85fe080cc:

Fixed #22906 -- Added a more helpful repr to migrations' ModelState.

Thanks Collin Anderson for the report and original patch.

comment:4 by Loic Bistuer <loic.bistuer@…>, 10 years ago

In a1ddfe440151c81898b94df4d5ea99696398c3f7:

[1.7.x] Fixed #22906 -- Added a more helpful repr to migrations' ModelState.

Thanks Collin Anderson for the report and original patch.

Backport of 2572c07cc6 from master

comment:5 by Collin Anderson, 10 years ago

This is perfect. Thanks so much all.

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