Google Product Search Feeder II [Support Thread]

[simon1066]

These PHP extension classes are new to me, but as I wanted to create a custom product_type (I used 666 as a test) for category 66 I used this code in a new classes file (includes/classes/gpsf/gpsfKb.php):

<?php
/**
 */
class gpsfKb extends gpsfBase
{
    public function getProductsAttributes(string $products_id, array $product, array $categories_list, $cPath = '66', array $custom_fields): array
    {
        $extension_custom_fields = ['product_type' => '666',];

        return $extension_custom_fields;
    }
}

This works in that it adds the product_type to the feed,

<g:product_type>666</g:product_type>
<g:product_type><![CDATA[Gift Vouchers]]></g:product_type>

I would appreciate some guidance on the code changes needed to replace the already generated product_type rather than add a new one.

I think that this plugin's 'site-specific additions to a product feed' is going to be a stretch for most non-coders, like me, so hopefully this example will get the ball rolling.

[simon1066]

Actually, what I've done hasn't worked as it adds the '666' to all products. Back to the drawing board.

[lat9]

Actually, what I've done hasn't worked as it adds the '666' to all products. Back to the drawing board.

Right, you'd need to check the incoming $products_id value and possibly the $product/$categories_list/$cPath arrays to see if the current product 'qualifies' for the change.

For what I think you're trying to do:

public function getProductsAttributes(string $products_id, array $product, array $categories_list, array $cPath, array $custom_fields): array 
    {
        if (!in_array(66, $cPath)) {
            return [];
        }
        $extension_custom_fields = ['product_type' => '666',]; 

        return $extension_custom_fields; 
    } 

That checks to see if categories-id 66 is present in the current product's category-tree, returning the 'product_type' override if so; otherwise, no override is supplied.

[simon1066]

Right, you'd need to check the incoming $products_id value and possibly the $product/$categories_list/$cPath arrays to see if the current product 'qualifies' for the change.

For what I think you're trying to do:

public function getProductsAttributes(string $products_id, array $product, array $categories_list, array $cPath, array $custom_fields): array 
    {
        if (!in_array(66, $cPath)) {
            return [];
        }
        $extension_custom_fields = ['product_type' => '666',]; 

        return $extension_custom_fields; 
    } 

That checks to see if categories-id 66 is present in the current product's category-tree, returning the 'product_type' override if so; otherwise, no override is supplied.

That's great, thanks so much for the pointer/correction.

[lat9]

These PHP extension classes are new to me, but as I wanted to create a custom product_type (I used 666 as a test) for category 66 I used this code in a new classes file (includes/classes/gpsf/gpsfKb.php):

<?php
/**
 */
class gpsfKb extends gpsfBase
{
    public function getProductsAttributes(string $products_id, array $product, array $categories_list, $cPath = '66', array $custom_fields): array
    {
        $extension_custom_fields = ['product_type' => '666',];

        return $extension_custom_fields;
    }
}

This works in that it adds the product_type to the feed,

<g:product_type>666</g:product_type>
<g:product_type><=!=[=C=D=A=T=A=[Gift Vouchers]=]=></g:product_type>

I would appreciate some guidance on the code changes needed to replace the already generated product_type rather than add a new one.

I think that this plugin's 'site-specific additions to a product feed' is going to be a stretch for most non-coders, like me, so hopefully this example will get the ball rolling.

At this point, the gpsfFeedGenerator class doesn't acknowledge an override for the product_type, using either a fixed value or a selection from the product's master-category's name 'tree'. I've created a change-request (https://github.com/lat9/gpsf/issues/11) on the GPSF GitHub repository.

[simon1066]

At this point, the gpsfFeedGenerator class doesn't acknowledge an override for the product_type, using either a fixed value or a selection from the product's master-category's name 'tree'. I've created a change-request (https://github.com/lat9/gpsf/issues/11) on the GPSF GitHub repository.

Understood.

[Jeff_Mash]

Cindy - Excited to try this one out! I just put it on our site but I'm seeing the following debug error on ALL the products:

Array
(
    [20361] => # SELFIE by Simon R. Stefan & Alex Pandrea - Trick: title cannot be empty
    [25185] => #Augmented (Gimmick and Online Instructions) by Luca Volpe and Renato Cotini - Trick: title cannot be empty
    [21745] => #How by Magic from Holland and Ferry de Riemer - Trick: title cannot be empty
    [35629] => $1.00 Prop Bill (1 bill/$1.00 value) - Realistic Fake Money: title cannot be empty
    [35623] => $10.00 Prop Bill (1 bill/$10.00 value) - Realistic Fake Money: title cannot be empty
    [24025] => $100 bill Silk 36 inch by Magic by Gosh - Trick: title cannot be empty
    [12237] => $100 Bill Switch - Booklet + Gimmick by Royal Magic: title cannot be empty
.....
.....

Any idea what I should be looking at to resolve this?

[lat9]

Cindy - Excited to try this one out! I just put it on our site but I'm seeing the following debug error on ALL the products:

Array
(
    [20361] => # SELFIE by Simon R. Stefan & Alex Pandrea - Trick: title cannot be empty
    [25185] => #Augmented (Gimmick and Online Instructions) by Luca Volpe and Renato Cotini - Trick: title cannot be empty
    [21745] => #How by Magic from Holland and Ferry de Riemer - Trick: title cannot be empty
    [35629] => $1.00 Prop Bill (1 bill/$1.00 value) - Realistic Fake Money: title cannot be empty
    [35623] => $10.00 Prop Bill (1 bill/$10.00 value) - Realistic Fake Money: title cannot be empty
    [24025] => $100 bill Silk 36 inch by Magic by Gosh - Trick: title cannot be empty
    [12237] => $100 Bill Switch - Booklet + Gimmick by Royal Magic: title cannot be empty
.....
.....

Any idea what I should be looking at to resolve this?

Are you using products' meta-tag titles as the g:title? The value's either a product's name (which, for above isn't empty) or the metatags_title.

I might see the issue, darn it. /includes/classes/gpsfFeedGenerator.php, line 280, change from

if (GPSF_META_TITLE === 'true' && $product['metatags_title'] !== '') {

to

if (GPSF_META_TITLE === 'true' && !empty($product['metatags_title'])) {

... and let me know if that corrects the issue. That darn metatags_title can be null.

[Jeff_Mash]

Are you using products' meta-tag titles as the g:title? The value's either a product's name (which, for above isn't empty) or the metatags_title.

I might see the issue, darn it. /includes/classes/gpsfFeedGenerator.php, line 280, change from

if (GPSF_META_TITLE === 'true' && $product['metatags_title'] !== '') {

to

if (GPSF_META_TITLE === 'true' && !empty($product['metatags_title'])) {

... and let me know if that corrects the issue. That darn metatags_title can be null.

BINGO! That fixed the issue!

[lat9]

BINGO! That fixed the issue!

Thanks for the validation; I'll get a GitHub issue logged.

GitHub issue: https://github.com/lat9/gpsf/issues/12