Steve Alexander wrote: > Shane Hathaway wrote: > >> >> os.stat() raises OSError if the file is not found, in which case mtime >> should be set to 0. > > > Surely if the file is not found, that's an error because the > PageTemplateFile is pointing at a source file that doesn't exist. > > I cannot think of any reason I'd want not to be informed that the source > file for a PageTemplateFile isn't there. That's clearly a programming > error. > > So, I suggest removing the try: except: clause entirely, and letting > os.stat() raise its error. > > Or, am I missing something here?
I vaguely recall having a similar discussion with someone regarding DTMLFile, and we decided it had to ignore missing files, but I don't remember why. Also, the open() call just below that line will raise an equivalent exception. Use your judgment--mine is clouded. :-) Shane _______________________________________________ Zope-Dev maillist - [EMAIL PROTECTED] http://lists.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://lists.zope.org/mailman/listinfo/zope-announce http://lists.zope.org/mailman/listinfo/zope )
