Skip to content
This repository has been archived by the owner on Aug 3, 2020. It is now read-only.

Google Shopping default Image & PHP 5.4 #4

Open
Amadeco opened this issue Mar 24, 2015 · 5 comments
Open

Google Shopping default Image & PHP 5.4 #4

Amadeco opened this issue Mar 24, 2015 · 5 comments

Comments

@Amadeco
Copy link

Amadeco commented Mar 24, 2015

Good evening,

First at all, congratulations to make this module compatible with the new Google Shopping Api.
Very useful & hard work.

I just want to warn you about google shopping default image attribute which is not filled by default, I think it will be better to set this attribute check for the first image. When you have many products, it simply impossible to configure this parameter one by one.

Also, I do not understand the use of the shortcut [] to create array, you make your module less accessible.

Thank you for your hard work.

Cheers.

@SchumacherFM
Copy link

Good points! Thanks!

PHP 5.3 is End of Life and PHP 5.4 supports short array syntax. Time to move on.

@Amadeco
Copy link
Author

Amadeco commented Mar 24, 2015

I am not very fan about alternative/shortcut syntax.

I would like to ask you something also, Even when I comment

    $url = $product->getGoogleShoppingImage();
    if ($url && $url != "no_selection") {

        $url = Mage::getModel('catalog/product_media_config')->getMediaUrl($product->getGoogleShoppingImage());
        $shoppingProduct->setImageLink($url);
        return $shoppingProduct;
    }

Google Merchant do not receive the image link.
It not integrate with the others attributes.
Have you a lead ?

A point that could be good is to integrate the shipping method (by default) directly in the API.

Thank you.

@SchumacherFM
Copy link

To be honest: no idea :-( But if you can figure it out and send me a pull request I'm happy to merge it in!

@Amadeco
Copy link
Author

Amadeco commented Mar 25, 2015

Hello,

That is a misunderstanding of mine. The normal behaviour of Google Merchant is to download the image after the product is valid.

It is interesting to implement additionnal image (more queries) ?

$i = 0; 
            foreach ($product->getMediaGalleryImages() as $image) {
                if ($i++ != 1) {
                    $shoppingProduct->setAdditionalImageLink($image->getUrl());
                }
            }

@SchumacherFM
Copy link

Will take care of that. May take a while :-(

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants