Testimonial Manager Support Thread

[countrycharm]

No worries..

FYI for future reference, the error message number is the hint that will tell you if it's a SQL error versus a PHP error.. 1064 is a mySQL syntax error message. And despite the long winded nature of error messages, the first part of most error messages really do tell you a good deal about an error and it's source: 1064:You have an error in your SQL syntax;

Additional reading on this (in case you're interested):
https://www.google.com/search?q=mySQ...ient=firefox-a

http://dev.mysql.com/doc/refman/5.5/...es-server.html)

In the meantime, and MOST importantly you need to use the updated SQL script I posted for your update submission.. The one lat9 posted still contains code that could inadvertently delete records from the configuration table.. You can simply resubmit it you do not have to wait for the submission you posted today to be approved. Simply reply to the e-mail you got acknowledging your submission and let the admins know you will be submitting a replacement update.. Then re-submit it with the SQL I posted..

Specifically this part is problematic:

#SET @t4=0;
SELECT (@t4:=configuration_group_id) as t4
FROM configuration_group
WHERE configuration_group_title= 'Testimonials Manager';

(The Zen Cart v1.5 threads on updating mods cover this and WHY it should NOT be included in SQL scripts for add-ons. If you recall I took this same kind of code out of the SQL for the Rewards points SQL)

Hi Diva I basically redid sql statement like you said in the first place. The only thing I didn't do was take out the engine part. Thank You for pointing that out. I did some reserch on MyISAM vs InnoDB and found out With the release of MySQL 5.5, InnoDB has become the default storage engine so thank you again. This will maybe cause others not to have the error when installing this module.
And thanks for the other 2 links for the sql errors.

If you'll notice, I had turned the problematic SET statements into comments ... thus also removing the problem.

Yes you did, but I we decided to remove them just in case others might not have understood what that was for.