#14386: Standardize Site/RequestSite access; looser coupling of sites framework
in
contrib
------------------------------------+---------------------------------------
Reporter: gabrielhurley | Owner: gabrielhurley
Status: closed | Milestone: 1.3
Component: Contrib apps | Version: SVN
Resolution: fixed | Keywords:
Stage: Unreviewed | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
------------------------------------+---------------------------------------
Comment (by gabrielhurley):
Thanks for the final cleanup (and commit) on the patch, Luke. The
inconsistent whitespace on comments and the like are what I get when
writing too much code too late at night. Good point on TearDown, too.
As for rolling the other two tickets into it later, I did that simply
because the solution to them relied on the same get_current_site method,
and I wasn't sure if this would get committed soon or days/weeks from now.
Didn't want to write a patch for those that wouldn't apply cleanly until
this one was in and have it end up getting messy. I certainly see your
point though.
Thanks!
--
Ticket URL: <http://code.djangoproject.com/ticket/14386#comment:5>
Django <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.
