#31254: "populate() isn't reentrant" masks underlying error
------------------------------+--------------------------------------
     Reporter:  TK-23         |                    Owner:  nobody
         Type:  Bug           |                   Status:  closed
    Component:  Core (Other)  |                  Version:  2.2
     Severity:  Normal        |               Resolution:  needsinfo
     Keywords:                |             Triage Stage:  Unreviewed
    Has patch:  0             |      Needs documentation:  0
  Needs tests:  0             |  Patch needs improvement:  0
Easy pickings:  0             |                    UI/UX:  0
------------------------------+--------------------------------------

Comment (by Timothy Schilling):

 Moving the `self.loading` comparison and set after the two known cases of
 raising `ImproperlyConfigured` seems to resolve the issue. If I could get
 some direction regarding how to write that test, I'll get this
 implemented. That said, I'm not sure this exception case is covered in the
 tests:
 
https://github.com/django/django/commit/7ed20e015335076fc98ad805eaf241f8a0d872d5

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

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.33a90c22dcc9c00fc104a2418c703faf%40djangoproject.com.

Reply via email to