Smarty

Aryos · Smarty Rookie Joined: 12 Jan 2010 Posts: 8

Just a small nuisance (not a bug), but would be convenient to fix it.

In Smarty 2, syntax of {if tags like:

grmbl · Smarty Rookie Joined: 12 Jan 2010 Posts: 7

I think it takes like 15 minutes to fix wrong templates with mass search and replace.

PHP was aimed to be backwards-compatible with crappy coding practices - look at the mess it is now.

Aryos · Smarty Rookie Joined: 12 Jan 2010 Posts: 8

More like half a minute Smile

This is not a request for me, I just thought there would be some people not knowing what's wrong when they see this exception message, thus not able to fix it right away.

ChrisFah · Posted: Tue Jan 12, 2010 12:45 pm Post subject:

I also think:
Better change template than code becuse of this tiny fact.
Regards, Chris

mohrt · Posted: Tue Jan 12, 2010 3:26 pm Post subject:

Smarty 3 raises an exception because { and } tags with whitespace are ignored so you don't have to escape javacript/css. So when you do this:

{ if foo}

Smarty see it as

if foo}

(no opening tag) and throws the exception. This is a small nuance of Smarty 3, but rids the need for {literal} tags all over the place. You can also revert the behavior with $smarty->auto_literal = false.

Aryos · Smarty Rookie Joined: 12 Jan 2010 Posts: 8

Thanks for the feedback, this makes sense.
One thing I'd like to ask is if the "auto-literal" function makes execution times any faster or if it's added just for convenience in coding templates.

mohrt · Posted: Tue Jan 12, 2010 7:22 pm Post subject:

I doubt there is any significant difference in performance, its just a change in how templates are compiled.

Michael White · Smarty Rookie Joined: 08 Oct 2009 Posts: 20

This error is present in many of our templates. However, I think that being able to drop {literal} is such a bonus that we'll just fix the templates.