3176 replies to this topic

[websiteguy]

Sorry in advance if I missed the answer to this question (this thread is long), but I am getting the following error:
"Missing Unique Product Identifiers (50 items)

Some of your items do not have valid unique product identifiers. Please add valid unique product identifiers. Otherwise it's possible that none of your items will appear in Google search results."


This is almost all of my items, is there an easy way around this? I don't have a box in my admin to enter UPC or anything. I am not sure if I need to create one (or how) or if there is some simple way in the code of the mod that I can do it.

Thanks for the help in advance, great of you to keep this up!

[Jack_mcs]

websiteguy, on 17 January 2012, 15:24, said:

Sorry in advance if I missed the answer to this question (this thread is long), but I am getting the following error:
"Missing Unique Product Identifiers (50 items)

Usually, all that is needed is to enalbe the brand and mpn settings.

[Mort-lemur]

My Store is Based in the UK and I sell some items that are taxable and some that are not taxable (Vat at 20%) eg. childrens clothing is not taxable but adults clothing is. Using the Google feeder in its standard form added tax to my non-taxable items resulting in warnings from Google for price missmatches between my data-feed and prices displayed on the store, and threats of impending account suspension.

After several days trying to alter the SQL of admin/googlefeeder.php to recalculate based on the products tax class without success Sakwoya suggested some php code to achieve the desired result. After playing with the code for a day or so (Im no PHP expert) I have managed to get the data-feed to correctly price the taxable and non-taxable products.

The changes required are as below and are probably not well coded - but they do work.

All the changes are ito file: Admin/googlefeeder.php

find:

products.products_quantity AS quantity,

Add After:

products.products_tax_class_id AS taxid,

Find:

		  $output .= $row->product_url . $google_utm . "\t";
	  }

Add After:

// Non Taxable Items Change - strips off tax added above - change division if tax rate changes
$the_price=$row->price;
if ($row->taxid ==0){$the_price=($the_price/1.2);}
// EOF Non Taxable Items Change

Find:

$row->price . "\t" .

Replace With:

//	  $row->price . "\t" .
	  $the_price."\t".

Thats It !

For this to work your taxable items must have a tax class id of 1 and non taxable items must have a tax class of 0.

Hope this helps someone - as it made me pull my hair out !!

[jrthor2]

I just pgraded to version 2.9, and when I try to upload our items, I am getting the following error:

FTP open connection failed to uploads.google.com

I have made sure that the username and password are correct in the googlefeeder.php file. What else could be wrong?

Thanks!

[Jack_mcs]

jrthor2, on 30 January 2012, 00:36, said:

I just pgraded to version 2.9, and when I try to upload our items, I am getting the following error:

FTP open connection failed to uploads.google.com

I have made sure that the username and password are correct in the googlefeeder.php file. What else could be wrong?

Thanks!

There's nothing different in the upload code so if the previous version worked, the login is all it could be, I think. You might want to doublecheck the username/password to be sure they are the same as before.

[sakwoya]

jrthor2, on 30 January 2012, 00:36, said:

I just pgraded to version 2.9, and when I try to upload our items, I am getting the following error:

FTP open connection failed to uploads.google.com

I have made sure that the username and password are correct in the googlefeeder.php file. What else could be wrong?

Thanks!

If you cannot get the ftp to work you can, instead, tell Google to go and collect the file itself - look at the bit under 'Automatic Upload' here:
http://support.google.com/merchants/bin/answer.py?hl=en&answer=188477

Set the feed to run as a cron job and then give Google a time to collect a few hours after that.

Graeme

[NodsDorf]

Hi Jack or anybody that may know..

Google requires us to modify our listing to show the price per minimum order.
So I replaced
FORMAT( IFNULL(specials.specials_new_products_price, products.products_price) * " . $taxCalc . ",2) AS price,
With
FORMAT( IFNULL(specials.specials_new_products_price, products.products_price) * products.minorder * " . $taxCalc . ",2) AS price,

This works great.

But I'm having trouble finding where the title is getting generated, as I would like the output to be `title` && (pack of `minimum order`)

Modifying our actual title wouldn't make sense because they are not sold in packs, you can order 20 or 21, but not 19. For Google display reasons we would like our price we are required to give them to be indicitive of the amount they would receive.

Can you help with this?

Thanks,
Don

[Jack_mcs]

NodsDorf, on 07 February 2012, 18:59, said:

Hi Jack or anybody that may know..

Google requires us to modify our listing to show the price per minimum order.
So I replaced
FORMAT( IFNULL(specials.specials_new_products_price, products.products_price) * " . $taxCalc . ",2) AS price,
With
FORMAT( IFNULL(specials.specials_new_products_price, products.products_price) * products.minorder * " . $taxCalc . ",2) AS price,

This works great.

But I'm having trouble finding where the title is getting generated, as I would like the output to be `title` && (pack of `minimum order`)

Modifying our actual title wouldn't make sense because they are not sold in packs, you can order 20 or 21, but not 19. For Google display reasons we would like our price we are required to give them to be indicitive of the amount they would receive.

Can you help with this?

Thanks,
Don

If you mean the entries in the column nmaed title, those are generated with this line

	  preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "\t" .

If that is not what you mean, please explain further and I will take another look.

[NodsDorf]

Jack_mcs, on 08 February 2012, 00:15, said:

If you mean the entries in the column nmaed title, those are generated with this line

	  preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "\t" .

If that is not what you mean, please explain further and I will take another look.

That is exactly what I meant, thanks Jack. I'll take a crack a modifing that to include the min-order number.

Thanks,
Don

[NodsDorf]

I tried Jack,

Just can't figure this one out.

I just need to add text to the name title that says Pack of $minorder

But I don't understand your coding enough to make it work.

I tried adding below this

$result=mysql_query( $sql )or die( $FunctionName . ": SQL error " . mysql_error() . "| sql = " . htmlentities($sql) );

while $result=mysql_fetch_array($sql) {
$minorder =$row[minorder]
}
Then changing

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "\t" .

to

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "(Pack of " . $minorder . ")". "\t" .

That just causes a infinite loop.

I tried a few others things but I just can't seem to get that $minorder appended to anything. I'm able to use the products.minorder column in the $sql query but not able to use its value anywhere else.

Any help is appreciated,
Don

[Jack_mcs]

NodsDorf, on 08 February 2012, 17:07, said:

I tried Jack,

Just can't figure this one out.

I just need to add text to the name title that says Pack of $minorder

But I don't understand your coding enough to make it work.

I tried adding below this

$result=mysql_query( $sql )or die( $FunctionName . ": SQL error " . mysql_error() . "| sql = " . htmlentities($sql) );

while $result=mysql_fetch_array($sql) {
$minorder =$row[minorder]
}
Then changing

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "\t" .

to

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "(Pack of " . $minorder . ")". "\t" .

That just causes a infinite loop.

I tried a few others things but I just can't seem to get that $minorder appended to anything. I'm able to use the products.minorder column in the $sql query but not able to use its value anywhere else.

Any help is appreciated,
Don

I don't know think that sql code you are using is correct, from the looks of it. But it shouldn't be necessary if $row[minorder] exists. In that case, this should work

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "(Pack of " . $row[minorder] . ")". "\t" .

[NodsDorf]

Jack_mcs, on 08 February 2012, 18:11, said:

I don't know think that sql code you are using is correct, from the looks of it. But it shouldn't be necessary if $row[minorder] exists. In that case, this should work

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) . "(Pack of " . $row[minorder] . ")". "\t" .

Yep you are correct,
The sql query wasn't formatted correctly


Took a lot of trys but this finally worked

Changed the $sql query to

$sql = "
SELECT concat( '" . $productURL . "' ,products.products_id) AS product_url,
products_model AS prodModel,
manufacturers.manufacturers_id,
products.products_id AS id,
products.minorder AS minorder,
products_description.products_name AS name,
products_description.products_description AS description,
products.products_quantity AS quantity,
products.products_status AS prodStatus,
products.products_weight AS prodWeight, " . $extraFields . "
FORMAT( IFNULL(specials.specials_new_products_price, products.products_price) * products.minorder * " . OPTIONS_TAX_CALC . ",2) AS price,
CONCAT( '" . $imageURL . "' ,products.products_image) AS image_url,
products_to_categories.categories_id AS prodCatID,
categories.parent_id AS catParentID,
categories_description.categories_name AS catName
FROM (categories,
categories_description,
products,
products_description,
products_to_categories)

left join manufacturers on ( manufacturers.manufacturers_id = products.manufacturers_id )
left join specials on ( specials.products_id = products.products_id AND ( ( (specials.expires_date > CURRENT_DATE) OR (specials.expires_date is NULL) OR (specials.expires_date = 0) ) AND ( specials.status = 1 ) ) )

WHERE products.products_id=products_description.products_id
AND products.products_id=products_to_categories.products_id
AND products_to_categories.categories_id=categories.categories_id
AND categories.categories_id=categories_description.categories_id " . $quotes . "
AND categories_description.language_id = " . DEFAULT_LANGUAGE_ID . "
AND products_description.language_id = " . DEFAULT_LANGUAGE_ID . "
ORDER BY
products.products_id ASC
";

Added an if statement above the output for the name

if ( $row->minorder > 1) {
             $minorder="(Pack of " . $row->minorder . ")";
        } else {
            $minorder="";
        }

Then added in the variable into the statement

preg_replace($_strip_search, $_strip_replace, strip_tags( strtr($row->name, $_cleaner_array) ) ) .  $minorder . "\t" .

Many thanks for your time and guidance, and knowledge!

[Jack_mcs]

NodsDorf, on 08 February 2012, 19:49, said:

Many thanks for your time and guidance, and knowledge!

You're welcome. I'm glad it worked out.

[altoid]

Jack_mcs, on 09 February 2011, 18:23, said:

Cron jobs are internal and should not be bothered by password protection. You should ask your host how to set it up. If you can't, you can use the Cron Simulator contribution.

Jack, until recently my google feed cron (set up as per your instructions in the install files) worked but something must have changed on my hosts end; meaning the cron ceased to work. I contacted the host support for syntax to pass the username and password along in the cron but the tech guy said it couldn't be done.

I see on the googlebase site I could set up a scheduled upload there, There were input boxes for the username and password, which I entered. A test run last night worked, but I'd have preferred not to put the shops username and password on the google site.

I tried search (google) for the cron syntax but came up with zero, so for now I'll rely on the googlebase site to run automated uploads.

Just FYI, thanks

[Jack_mcs]

altoid, on 10 February 2012, 14:55, said:

Jack, until recently my google feed cron (set up as per your instructions in the install files) worked but something must have changed on my hosts end; meaning the cron ceased to work. I contacted the host support for syntax to pass the username and password along in the cron but the tech guy said it couldn't be done.

I see on the googlebase site I could set up a scheduled upload there, There were input boxes for the username and password, which I entered. A test run last night worked, but I'd have preferred not to put the shops username and password on the google site.

The username/password can be passed via cron but the code is not setup to work that way so it wouldn't make any difference. Plus, I don't understand why you would need that in the first place since the user/pass are in the file. As for the google scheduler, that is safe but you will have to remember to manually run the googlebase script to make sure your products are accurate.

[altoid]

Jack_mcs, on 10 February 2012, 15:05, said:

The username/password can be passed via cron but the code is not setup to work that way so it wouldn't make any difference. Plus, I don't understand why you would need that in the first place since the user/pass are in the file. As for the google scheduler, that is safe but you will have to remember to manually run the googlebase script to make sure your products are accurate.

I am not sure why that quit working.

I have four different crons set up on my sites, the only one not working is the googlefeeder cron. But it did work once.

I was presuming because googlefeeder is under the .htaccess password protected area that might be the cause, especially if the host changed something?

Just to see what happens, I will delete the current googlefeeder cron and set up another to see if that does anything.

I'm just taking a stab in the dark here, as the only other thing that occurred I am aware of on my host is I had my shops consolidated to a VPS acct. I presume (presuming is what I do best) that the host did something that may have effected things.

[Jack_mcs]

altoid, on 10 February 2012, 15:25, said:

I am not sure why that quit working.

I have four different crons set up on my sites, the only one not working is the googlefeeder cron. But it did work once.

I was presuming because googlefeeder is under the .htaccess password protected area that might be the cause, especially if the host changed something?

Just to see what happens, I will delete the current googlefeeder cron and set up another to see if that does anything.

I'm just taking a stab in the dark here, as the only other thing that occurred I am aware of on my host is I had my shops consolidated to a VPS acct. I presume (presuming is what I do best) that the host did something that may have effected things.

It might be that the settings are different on the new server. Cron doesn't need login for the secure areas since it is handled from the server and has access already. You should check, if you haven't already, that the googlebase script can be manually ran from admin. If it can, then the cron job is probably failing due to a server setting, assuming the admin's name hasn't changed. If it is failing, then your host should be able to run it from their side to see why it is failing.