Smarty

[andrewfo]

Hi,

I am trying to loop through a collection of objects that have a date property and print them to a table, when the date changes I would like to echo out the current date as a heading.

I think to get it done I need to save a variable in a scope higher that the current loop that I can reference in the loop, If the date changes, change the variable and continue the loop until the next time the date changes. here is some example code

[AnrDaemon]

1. You're using wrong syntax for foreach.
The syntax is "{foreach $array as [$key =>] $value}". Yes, same as PHP.
2. You're using wrong formatters for date.
date_format() is date() compatible, not strftime() compatible. (yet another brain fart of PHP developers…)
3. You can not use more than one <caption> per table!
Start reading standards before writing your code.
4. You're using bogus spaces in your template. If you need margins, add margins, but don't try to emulate margins with spaces. Your children will never forgive you.

[andrewfo]

Thanks for the reply,

I will definitely look into all that you have said.

Didn't know about the one caption per table, cheers for that.

I think I have been looking at the old docs, thanks for the new foreach syntax, it's much more natural, I'm sure there are many more improvements that I am yet to discover.

$timestamp|date:"d" & $timestamp|date:"%d" isn't working for me. date_format is though.

spaces were only for the example as there is no syntax highlighting on this forum, so I think it's easier to read when separated out with spaces.

anyway ...turns out it was my fault, I was referencing the incorrect variable identifier. It now works as expected.

[AnrDaemon]

I was only referring to placeholder compatibility, not the modifier name.
Glad you've sorted it now, though.
BTW, it's easier to use "z" instead of "d" modifier, much less chance for collision.