template variable access

[RichardM]

Hi,

I've been looking through my logs and sometimes I will have a template which may contain code similar to:

Code: Select all

<div class="error">{{errorMsg}}</div>


But in my Controller Action I might not have defined $data['errorMsg'] before calling $this->view()->render('blah', $data); so when the template is compiled and used php will log the warning noting that 'errorMsg was not defined'. Therefore I'm wondering if we should have a check in place for isset(...) ? ... : ''; so it will return an empty string if its not been defined so as to avoide PHP warnings? I know I could avoid having the variables in the template or initialise all of them to '' before hand but I am not sure this is something I want to do in every view...especially if I start updating the templates?


Richard

[leng]

You can check it with if statement.
If you want a shorter code, just write a function in template tags that do that.

Code: Select all

function sample($var){
    return (isset($var)) ? $var : '';
}

[RichardM]

I'd forgotten about the ability to add in extra template functions . I'll see about integrating this into the tutorial later on. Also more KISS friendly that my original idea to force it for every template.


Richard

[RichardM]

Hey,

Just tried this and still getting the error. I think if I'm not mistaken the error is because writing

Code: Select all

<p>{{sample(myUndefinedVar)}}</p>


results in something like

Code: Select all

<p><?php echo sample($data['myUndefinedVar']); ?></p>


So php will try and read the undefined array element before its passed to sample to be processed.

right now I have opted for the following

Code: Select all

<!-- if {{isset(username)}}  -->{{username}}<!-- endif -->


but i find this looks kinda messy in my template file.

Have you any other ideas?


Richard

[leng]

hmm, why would it be undefined? Anyway you can process it in the Controller and set it to null