Smarty

[adebruyn]

Hello:

Here is my PHP code. Pretty straightforward, in a test.php file.

...
$smarty = new Smarty;
$smarty->display('file:test.tpl');

test.php and test.tpl are both in the same directory.

When I run this script with a custom error handler (set_error_handler(...)), there is an error triggered in smarty_resource.php, line 313.

filemtime() [<a href='function.filemtime'>function.filemtime</a>]: stat failed for .\templates\test.tpl

I've tested the value of $file in the fileExists(Smarty_Template_Source $source, $file) function. Smarty test the following template locations, sequentially:

.\templates\test.tpl (not found)
test.tpl (correct)

And not finding the first one, throw an error message (E_WARNING). I can't allow systematic E_WARNING messages in my application. Corporate policy.

Now what ? Is it a "bug", or did I do somethig wrong that makes Smarty (v3.1.4) test a file that does not exist ?

[mohrt]

smarty should only check template directories for templates. ie. the current directory will not be tested unless added to the template_dir array. not finding test.tpl in the template_dir array is an expected warning.

you could also employ a default template handler to catch the case when a template is not found and do something with it instead of a warning.

[adebruyn]

Thanks mohrt.

First of all, your answer does not seem to be correct. After running some tests, whether you specifically include the current directory through setTemplateDir or not, it will always be searched, no matter what. So these two seem to be perfectly equivalent on my end:

$smarty->setTemplateDir(array('.', '/mytemplates'));
$smarty->setTemplateDir('/mytemplates');

Second, it doesn't solve my problem. Say my templates can be found in multiple directories (general header and footer in a global directory, and template specific for the current page in current directory), and I set:

$smarty->setTemplateDir(array('.', '/mytemplates'));

Now, if I use 'mytemplates/footer.tpl', smarty will still issue a warning when it doesn't find ./footer.tpl. That's kind of unexpected...

And again, I can't allow systematic warnings in my application, error handling are very strict.

[mohrt]

when you set the template_dir to '/templates' that is a system root directory. If you want it relative to the cwd you want './templates'. is that part of the problem? I'll investigate that automated '.' issue.

[adebruyn]

No, what I have is fine. The recurring templates (header, footer, etc.) are in the root directory, as expected, and they are found.

My problem is not so much that smarty does not find the templates --it does, as expected, despite the automated '.' issue we talked about--.

My problem is that it generates a warning everytime the file is not in one of the listed directories it is supposed to search. So if I list 5 directories (silly, but...), and the template I need is in the last one of them, I get 4 warnings. If it's in the first, I get no warning.

I think it's a weird behavior that might require investigation. After all, if I list multiple directories, Smarty should only trigger one error at the end if the template cannot be found anywhere. Triggering a warning at each search step does not make much sense to me...

By the way, this problem only popped up because I have my own set_error_handler.

Thanks again for your help.

[mohrt]

that doesn't sound right, it should emit a warning only if the template is not found anywhere. will look into it.

[U.Tews]

See www.smarty.net/forums/viewtopic.php?t=20204

See explaination by rodneyrehm


Your cutom error handlers could also check if the error was caused by a statement with @ error suppression. See example in the last post.

[mohrt]

ah yeah. we do a @filemtime() on each file/directory, and that will throw a warning for files no found, which is suppressed. we don't run file_exists() first for both speed and race condition issues. for a custom error handler, you must test if error_reporting() === 0 to correctly handle @suppressed errors.

[mohrt]

and there is no automated '.' search. could you provide a simple test if you see otherwise?