[Zope-CMF] Re: [CMF-checkins] CVS: Products/CMFCore - DirectoryView.py:1.47.2.7

[Florent Guillaume]

Chris Withers  <chris at simplistix.co.uk> wrote:
> Florent Guillaume wrote:
> > +                warn('DirectoryView %s refers to a non-existing path %s'
> > +                     % (self.id, dirpath), UserWarning)
> 
> Urg. I hate the warn module, anyone object if I change that to a normal 
> logging call?

I deliberately didn't use a LOG call, because this line will get
executed for every request if there is an old directory view present,
and it'll fill your logs pretty quickly, which is not acceptable.

Using warn was the quickest alternative that worked, if you have
something better...

Florent

-- 
Florent Guillaume, Nuxeo (Paris, France)   CTO, Director of R&D
+33 1 40 33 71 59   http://nuxeo.com   fg at nuxeo.com