[Django] #22906: ModelState.__repr__

46 views
Skip to first unread message

Django

unread,
Jun 25, 2014, 9:14:08 PM6/25/14
to django-...@googlegroups.com
#22906: ModelState.__repr__
--------------------------------+------------------------
Reporter: CollinAnderson | Owner: nobody
Type: Uncategorized | Status: new
Component: Migrations | Version: 1.7-beta-2
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 1
Easy pickings: 0 | UI/UX: 0
--------------------------------+------------------------
I've gotten a number of `InvalidBasesError("Cannot resolve bases for %r" %
new_unrendered_models)` errors. Could we make a repr for `ModelState`
objects?

--
Ticket URL: <https://code.djangoproject.com/ticket/22906>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Jun 25, 2014, 9:16:43 PM6/25/14
to django-...@googlegroups.com
#22906: Give ModelState a __repr__()
--------------------------------+--------------------------------------
Reporter: CollinAnderson | Owner: nobody
Type: New feature | Status: new
Component: Migrations | Version: 1.7-beta-2
Severity: Normal | Resolution:
Keywords: | Triage Stage: Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------
Changes (by CollinAnderson):

* needs_docs: => 0
* needs_better_patch: => 0
* type: Uncategorized => New feature
* needs_tests: => 0


Comment:

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

--
Ticket URL: <https://code.djangoproject.com/ticket/22906#comment:1>

Django

unread,
Jun 26, 2014, 4:05:18 AM6/26/14
to django-...@googlegroups.com
#22906: Give ModelState a __repr__()
--------------------------------+--------------------------------------
Reporter: CollinAnderson | Owner: nobody
Type: New feature | Status: new
Component: Migrations | Version: 1.7-beta-2
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted

Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------
Changes (by claudep):

* needs_tests: 0 => 1
* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/22906#comment:2>

Django

unread,
Jul 6, 2014, 3:58:43 AM7/6/14
to django-...@googlegroups.com
#22906: Give ModelState a __repr__()
--------------------------------+--------------------------------------
Reporter: CollinAnderson | Owner: nobody
Type: New feature | Status: closed
Component: Migrations | Version: 1.7-beta-2
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------
Changes (by Loic Bistuer <loic.bistuer@…>):

* status: new => closed
* resolution: => fixed


Comment:

In [changeset:"2572c07cc64177e985b453d0f0b819e85fe080cc"]:
{{{
#!CommitTicketReference repository=""
revision="2572c07cc64177e985b453d0f0b819e85fe080cc"
Fixed #22906 -- Added a more helpful repr to migrations' ModelState.

Thanks Collin Anderson for the report and original patch.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/22906#comment:3>

Django

unread,
Jul 6, 2014, 4:03:48 AM7/6/14
to django-...@googlegroups.com
#22906: Give ModelState a __repr__()
--------------------------------+--------------------------------------
Reporter: CollinAnderson | Owner: nobody
Type: New feature | Status: closed
Component: Migrations | Version: 1.7-beta-2

Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------

Comment (by Loic Bistuer <loic.bistuer@…>):

In [changeset:"a1ddfe440151c81898b94df4d5ea99696398c3f7"]:
{{{
#!CommitTicketReference repository=""
revision="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
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/22906#comment:4>

Django

unread,
Jul 6, 2014, 11:07:37 AM7/6/14
to django-...@googlegroups.com
#22906: Give ModelState a __repr__()
--------------------------------+--------------------------------------
Reporter: CollinAnderson | Owner: nobody
Type: New feature | Status: closed
Component: Migrations | Version: 1.7-beta-2

Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------+--------------------------------------

Comment (by CollinAnderson):

This is perfect. Thanks so much all.

--
Ticket URL: <https://code.djangoproject.com/ticket/22906#comment:5>

Reply all
Reply to author
Forward
0 new messages