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

Error after install #90

Open
borjadinamic opened this issue Apr 18, 2018 · 6 comments
Open

Error after install #90

borjadinamic opened this issue Apr 18, 2018 · 6 comments

Comments

@borjadinamic
Copy link

I've follow the installation guide, this is that Sylius return's for this request.
/shop-api/products?channel=US_WEB

Type error: Argument 1 passed to Sylius\\ElasticSearchPlugin\\Document\\TaxonDocument::setImages() must be an instance of ONGR\\ElasticsearchBundle\\Collection\\Collection, instance of ONGR\\ElasticsearchBundle\\Result\\ObjectIterator given, called in \/private\/var\/www\/sylius-test\/vendor\/ongr\/elasticsearch-bundle\/Result\/Converter.php on line 134"}

Probably bug?

@borjadinamic borjadinamic changed the title Inconsistencies? Error after install? Apr 18, 2018
@borjadinamic borjadinamic changed the title Error after install? Error after install Apr 18, 2018
@psihius
Copy link

psihius commented Apr 18, 2018

@borjadinamic have you installed the v1.0.0-beta.17 tag or master, or some other version?

If it's the latest version, then would be nice to see the steps you have taken to have this error to reproduce it more easilly.

@jesperveldhuizen
Copy link

jesperveldhuizen commented Apr 19, 2018

I'm having the same problem. Updated an product in the Sylius admin and ran this command:

/opt/plesk/php/7.1/bin/php bin/console sylius:elastic-search:update-product-index

Type error: Argument 1 passed to Sylius\ElasticSearchPlugin\Document\TaxonD
ocument::setImages() must be an instance of ONGR\ElasticsearchBundle\Collec
tion\Collection, instance of ONGR\ElasticsearchBundle\Result\ObjectIterator
given, called in /var/www/vhosts/*/vendor/ongr/elasticsearch-bundle/Result/Converter.php on li
ne 134

I'm on the latest version of this plugin.

@psihius
Copy link

psihius commented Apr 19, 2018

Okay, seems Syliys might have done some changes, I'll take a look

@borjadinamic
Copy link
Author

@psihius some news related?

@MoeVoe
Copy link

MoeVoe commented May 9, 2018

Any news regarding this bug.

@11mb
Copy link

11mb commented May 25, 2018

A workaround for us was to override all Documents and replace the usages of

ONGR\ElasticsearchBundle\Collection\Collection,
with
the Interface Doctrine\Common\Collections\Collection.

In the constructors change new Collection() to new ArrayCollection() (Doctrine\Common\Collections\ArrayCollection)

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

No branches or pull requests

5 participants