Andrew,
Thank you for the continued development work on this module
This may not be related to the problem BessIsaacola has reported, but isn't there still an unnecessary '\t' on the end of the feed?
For example:
PHP Code:
$output = "title \t" .
"description \t" .
"link \t" .
"price \t" .
"image_link \t" .
"expiration_date \t" .
"label \t" .
"id \t" .
(GOOGLE_FROOGLE_IN_STOCK == 'true' ? "quantity \t" : '') .
(GOOGLE_FROOGLE_SHIPPING != '' ? "shipping \t" : '') .
(GOOGLE_FROOGLE_MANUFACTURER == 'true' ? "brand \t" : '') .
(GOOGLE_FROOGLE_PRODUCT_TYPE_SHOW == 'true' ? "product_type \t" : '') .
(GOOGLE_FROOGLE_LANGUAGE_DISPLAY == 'true' ? "language \t" : '') .
(GOOGLE_FROOGLE_CURRENCY_DISPLAY == 'true' ? "currency \t" : '');
Each line has a '\t' on the end - which means there will always be an extra (unnecessary) tab on the end of the feed, no matter which options are set in admin.
Shouldn't the "id \t" be simply "id", and the following '\t's be moved to the beginning of the optional feed lines, eg:
PHP Code:
$output = "title \t" .
"description \t" .
"link \t" .
"price \t" .
"image_link \t" .
"expiration_date \t" .
"label \t" .
"id" .
(GOOGLE_FROOGLE_IN_STOCK == 'true' ? "\t quantity " : '') .
(GOOGLE_FROOGLE_SHIPPING != '' ? "\t shipping" : '') .
(GOOGLE_FROOGLE_MANUFACTURER == 'true' ? "\t brand " : '') .
(GOOGLE_FROOGLE_PRODUCT_TYPE_SHOW == 'true' ? "\t product_type" : '') .
(GOOGLE_FROOGLE_LANGUAGE_DISPLAY == 'true' ? "\t language" : '') .
(GOOGLE_FROOGLE_CURRENCY_DISPLAY == 'true' ? "\t currency" : '');
(And the same for the block of code from line 146 to line 159)
Bookmarks