Google Base Update

[Guest ITechWest]

Google Base has added a product ID requirement for the upload, this needs to be fixed in export.inc.php, can you guys take a look.

I am using 4.1.1

Current Code

if ($results == true) {

$googleBaseContent = "product_url\tname\tdescription\timage_url\tprice\tcurrency\r\n";

for ($i=0; $i<count($results); $i++) {

$salePrice = salePrice($results[$i]['price'], $results[$i]['sale_price']);

$price = ($salePrice > 0) ? $salePrice : $price = $results[$i]['price'];

$name = str_replace(array("&nbsp;","\t","\r","\n","\0","\x0B","

"),"",strip_tags($results[$i]['name']));

$name = str_replace(" ","",$name);

$desc = str_replace(array("&nbsp;","\t","\r","\n","\0","\x0B","

"),"",strip_tags($results[$i]['description']));

$desc = str_replace(" ","",$desc);

if ($config['sef'] == 0) {

$googleBaseContent .= $glob['storeURL']."/index.php?_a=viewProd&productId=".$results[$i]['productId']."\t".$name."\t".$desc;

} else {

include_once("includes".CC_DS."sef_urls.inc.php");

$googleBaseContent .= $glob['storeURL']. "/" .generateProductUrl($results[$i]['productId'])."\t".$name."\t".$desc;

}

if ($results[$i]['image']) {

$googleBaseContent .= "\t".$glob['storeURL']."/images/uploads/".$results[$i]['image'];

} else {

$googleBaseContent .= "\t".$glob['storeURL']."/skins/".$config['skinDir']."/styleImages/nophoto.gif";

}

$googleBaseContent .= "\t".$price."\t".$config['defaultCurrency']."\r\n";

[Guest xceejayx]

yeah i hope this gets sorted :(

[Guest ITechWest]

It has, if you are running 4.x, down load the file from 4.2.2 and it fixes this

[Guest]

Hi All

Simple change:

if ($results == true) {

$googleBaseContent = "id\tproduct_url\tname\tdescription\timage_url\tprice\tcurrency\r\n";

//ADDED THIS//////////^^^^



for ($i=0; $i<count($results); $i++) {



//ADDED THE BELOW LINE

$googleBaseContent .= $results[$i]['productId']."\t";



$salePrice = salePrice($results[$i]['price'], $results[$i]['sale_price']);

$price = ($salePrice > 0) ? $salePrice : $price = $results[$i]['price'];



$name = str_replace(array("&nbsp;","\t","\r","\n","\0","\x0B","

"),"",strip_tags($results[$i]['name']));

$name = str_replace(" ","",$name);

$desc = str_replace(array("&nbsp;","\t","\r","\n","\0","\x0B","

"),"",strip_tags($results[$i]['description']));

$desc = str_replace(" ","",$desc);



if ($config['sef'] == 0) {

$googleBaseContent .= $glob['storeURL']."/index.php?_a=viewProd&productId=".$results[$i]['productId']."\t".$name."\t".$desc;

} else {

include_once("includes".CC_DS."sef_urls.inc.php");

$googleBaseContent .= $glob['storeURL']. "/" .generateProductUrl($results[$i]['productId'])."\t".$name."\t".$desc;

}



if ($results[$i]['image']) {

$googleBaseContent .= "\t".$glob['storeURL']."/images/uploads/".$results[$i]['image'];

} else {

$googleBaseContent .= "\t".$glob['storeURL']."/skins/".$config['skinDir']."/styleImages/nophoto.gif";

}



$googleBaseContent .= "\t".$price."\t".$config['defaultCurrency']."\r\n";

What I have done here is use the product ID field in the database as this is a unique field and will never be used again if a product is deleted.

Adam

[Guest xceejayx]

thanks :)

[disco_ii_disco]

Also GoogleBase didn't like the fact that some descriptions were blank.

I think I've managed to get around this by adding:

." Click for more info."

to the end of line 55 (or 57 if you are using SEF links):

$googleBaseContent .= $glob['storeURL']."/index.php?_a=viewProd&productId=".$results[$i]['productId']."\t".$name."\t".$desc." Click for more info.";

[Guest xceejayx]

What I have done here is use the product ID field in the database as this is a unique field and will never be used again if a product is deleted.

Adam

I tried this fix.

I get the following errors from google once they have processed the file.

EDIT - Nevermind - disco_ii_disco's post fixed the errors. :)

[Guest jackc]

Will the above mods work on V3??

TIA,

Jack

[Guest cuber1]

Will the above mods work on V3??

TIA,

Jack

Have the same question.

[Guest cuber1]

http://forums.cubecart.com/index.php?showtopic=34146

The above works.

I had to manually shift the cells and include one header, but that's nothing and takes a few secs.