#3448: python help() parsing problem for django.core.mail
-----------------------------------------------------------+----------------
Reporter: Michael VanLandingham <[EMAIL PROTECTED]> |
Owner: jacob
Status: new |
Component: Documentation
Version: SVN |
Resolution:
Keywords: django.core.mail, help, docs |
Stage: Accepted
Has_patch: 0 |
Needs_docs: 0
Needs_tests: 0 |
Needs_better_patch: 0
-----------------------------------------------------------+----------------
Changes (by anonymous):
* stage: Design decision needed => Accepted
Comment:
The change suggested by SmileyChris is indeed the correct fix. We should
never refer to {{{settings.*}}} in default arguments because it removes
the ability to manually configure the settings (via
{{{settings.configure()}}}), as well as causing problems as in this bug.
Use {{{None}}} for the default argument (which is read at import time) and
inside the function do test for None and replace it with the
{{{settings.XYZ}}} value; we require settings to have already be
configured in some fashion before actually executing any functions, so
this is safe.
--
Ticket URL: <http://code.djangoproject.com/ticket/3448#comment:2>
Django Code <http://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 post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
