It seems you are right. I also posted a patch for this problem, but I've gone back to check again today. I'm not sure my patch is the right fix. I changed a lot of stuff that probably didn't need to be changed. Now I think that maybe my bug posting should be pulled from the bugzilla too. On Tue, 2006-09-12 at 08:24 +1200, Chris Cormack wrote:

On Mon, Sep 11, 2006 at 10:17:01PM +0200, Wojciech Zatorski said:

...

Hello MJ,

Monday, September 11, 2006, 8:35:45 PM, you wrote:

MR> Antoine Farnault wrote:

...

...

* on bug #1089 : should be not fix in rel_2_2. koha 2.2.x don't work with mysql 5.

MR> Which is correct on this one? Koha's syntax or the MySQL 5 one? MR> If Koha's SQL is wrong, the problem should be noted in the release MR> notes for 2.2.x, at least.

KOHA SQL is wrong. i have noticed fix for this long times ago...

Yes,

It is an easy fix, adding some ` to the sql install file, and adding some () in the sql in the modules.

There was a patch to the list about it from Dana Huff a little while ago, perhaps we should just do that in rel_2_2, then it will work with mysql 5?

Chris