[Zope-dev] bad bare except in PageTemplateFile.py

Shane Hathaway shane@zope.com
Thu, 18 Jul 2002 00:40:34 -0400


Steve Alexander wrote:
> lib/python/Products/PageTemplateFile.py, line 110, method _cook_check
> 
>         try:    mtime=os.stat(self.filename)[8]
>         except: mtime=0
> 
> 
> I've just spent an hour or so tracking down an awkward bug in some 
> unit-tests. The true error was being hidden by this bare except.
> 
> I'd change it, and commit the change, except that I don't know what it 
> is supposed to be catching. Any hints?

os.stat() raises OSError if the file is not found, in which case mtime 
should be set to 0.  There are several ways to fix this; use your 
judgment. :-)

BTW this little idiom is found all over the place, including CMF.

Shane