Quote Originally Posted by a_berezin View Post
Thanks. Fixed.
Thanks for updating Andrew.

1. So everyone should now use googlefroogle.sql in Andrew's latest version (disregard my attachment above... forum moderator please delete that post).

UPDATE:
2. However I continue to have issue with the new googlefroogle.sql

Specifically:
Code:
(NULL, 'Expiration Date Base', 'GOOGLE_FROOGLE_EXPIRATION_BASE', 'now', 'Expiration Date Base:<ul><li>now - add Adjust to current date;</li><li>product - add Adjust to product date (max(date_added, last_modified, date_available))</li></ul>', @t4, 2, NOW(), NULL, 'zen_cfg_select_option(array(\'now\', \'product\'),'),
Note that configuration_description above contains two commas and a semi-colon, which produces errors on importing into the database (at least it does for me).

The commas act as delimiters to separate the dB fields, and the semi-colon identifies the end of the SQL command. Maybe these characters need escaped? Maybe others don't have this issue?

I'm not SQL expert so I may be wrong on the above but the new SQL does not work for me.

Code:
'Expiration Date Base:<ul><li>now - add Adjust to current date </li><li>product - add Adjust to product date (max(date_added - last_modified - date_available))</li></ul>'
If I remove the semi-colon and change the two commas to dashes, as in above description, then all is ok. At least it works for me.

Not criticism just want to ensure all is ok with sql patch ;-)

3. Also if installing this new version of mod everyone be sure to upload the new /googlefroogle.php file. Although version id tag in the php file header remains the same, inspection of the file identified a new line of code (line 61):
Code:
echo "Processing: Feed - " . (isset($_GET['feed']) && $_GET['feed'] == "yes" ? "Yes" : "No") . ", Upload - " . (isset($_GET['upload']) && $_GET['upload'] == "yes" ? "Yes" : "No") . NL;
Woody