On Sat, Oct 2, 2010 at 8:59 AM, Kevin Howerton <kevin.hower...@gmail.com> wrote:
> Definitely agree.  Caught the missing nullhandler behavior when
> logging is off too... though didn't make the connection that this was
> likely intended to handle that case.
>
> It might make more sense to put this logic in conf/__init__.py
> following the calls to configure the logger.  Though that doesn't
> really matter as the handler could only be added once anyhow.
>
> ie:
>
> logger = logging.getLogger('django')
> if not logger.handlers:
>   logger.addHandler(NullHandler())

Ok - I've added this code to my RC patch.

Thanks for the clarification, Vinay and Kevin.

Yours,
Russ Magee %-)

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Reply via email to