From this, it looks like your SQL string contains carrige returns. I don't belive you can do that.

If that's not the case, what happens if you just replace all occurances of $year with a hard coded int?

Rich

Update: Thanks to crazyinsomniac for correcting me. SQL is not delimited by a CR so I was completely off base.