Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix invalid @method phpdoc to prevent prophecy mocking error #1792

Conversation

tkotosz
Copy link
Contributor

@tkotosz tkotosz commented Sep 4, 2015

Issue - Invalid docblock prevent using Prophecy

I am trying to use Magento\Catalog\Model\Product with Prophecy (using phpspec) and I get the following error :

could not double `Magento\Catalog\Model\Product` and implement interfaces: [Prophecy\Prophecy\ProphecySubjectInterface, Prophecy\Doubler\Generator\ReflectionInterface].

Fix

After I found this it turned out the Product class has invalid @method phpdoc here. So in order to fix this issue only this phpdoc need to be fixed.

@okorshenko
Copy link
Contributor

@tkotosz, thank you for contribution!

CR: passed
Builds: red (known issue)
Resolution: OK to merge
Ticket: MAGETWO-42614

@tkotosz
Copy link
Contributor Author

tkotosz commented Sep 14, 2015

@okorshenko I have updated the PR since this issue appeared in other classes as well. (I also rebased my branch in order to not conflict with the current develop)

@tkotosz tkotosz force-pushed the fix/prophecy-doubler-error-for-product-model branch from 30df492 to 8141174 Compare September 14, 2015 10:24
@tkotosz tkotosz force-pushed the fix/prophecy-doubler-error-for-product-model branch from 8141174 to b15d950 Compare September 14, 2015 10:26
@magento-cicd2
Copy link
Contributor

We have automated a Magento Contributor License Agreement verifier for contributions sent to our GitHub projects.
Please see the CLA agreement in the Pull Request comments below.

@okorshenko
Copy link
Contributor

@tkotosz , could you please merge latest changes from develop branch? Thank you

Conflicts:
	app/code/Magento/Catalog/Model/Product.php
	app/code/Magento/Reports/Block/Adminhtml/Shopcart/Abandoned/Grid.php
@tkotosz
Copy link
Contributor Author

tkotosz commented Oct 19, 2015

@okorshenko Done

@magento-team magento-team merged commit 120247c into magento:develop Oct 23, 2015
@tkotosz tkotosz deleted the fix/prophecy-doubler-error-for-product-model branch April 15, 2021 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants