reinink
commented
9 years ago The reason is because you are still passing $page_lang to the default() function, which will throw an error in your template already, not in your extension method.
This would be the perfect situation to use the new isset ternary syntax coming in PHP7, which wouldn't even require a custom extension:
<html lang="<?=$page_lang ?: 'en'?>">What you need to do is check if the variable exists within your extension. However, this is currently not possible, since the $template->data parameter is protected, and there is no data access method. This is a bit of an oversight on my part, so I'm going to leave this issue open and actually add this functionality to Plates.
When that is ready, you'll be able to do something like this:
<?php
use League\Plates\Engine;
use League\Plates\Extension\ExtensionInterface;
class MyExtension implements ExtensionInterface
{
public $template;
public function register(Engine $engine)
{
$this->plate->registerFunction('default', [$this, 'extension_default']);
}
function extension_default($var, $def = '')
{
$data = $this->template->data();
if (isset($data[$var])) {
return $data[$var];
}
return $def;
}
}And then in your templates:
<html lang="<?=$this->default('page_lang, 'en')?>">
electricjones
jenolan
I've create an extension of mine to automatically handle missing variables and replacing them with a default value, i.e.
however when using the code in the template i still getting an error message abount the variable being undefined:
avoidable with @:
Is there a better alternative way to fill up my page with all of those @$this->default() calls?