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.

Sign up for GitHub

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

Jump to bottom

Issue with OrderItemRepository #1894

Closed
dpotkoc opened this issue Sep 17, 2015 · 3 comments
Closed

Issue with OrderItemRepository #1894

dpotkoc opened this issue Sep 17, 2015 · 3 comments
Assignees
@NadiyaS
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@dpotkoc
Copy link

dpotkoc commented Sep 17, 2015

When I try to use \Magento\Sales\Api\OrderItemRepositoryInterface I got an error, which makes OrderItem service contracts/apis imposible to use.

I seems Magento\Sales\Model\Resource\Order\Item should implement OrderItemResourceInterface

this is error:
Recoverable Error: Argument 1 passed to Magento\Sales\Api\Data\OrderItemInterfacePersistor::__construct() must implement interface Magento\Sales\Model\Spi\OrderItemResourceInterface, instance of Magento\Sales\Model\Resource\Order\Item given, called in /home/dpotkoc/public_html/magento2/lib/internal/Magento/Framework/ObjectManager/Factory/AbstractFactory.php on line 101 and defined in /home/dpotkoc/public_html/magento2/var/generation/Magento/Sales/Api/Data/OrderItemInterfacePersistor.php on line 60
@okorshenko
Copy link
Contributor

please, remove all files from /home/dpotkoc/public_html/magento2/var/generation and try again

@ivanweiler
Copy link

Confirming the problem, exactly like @dpotkoc wrote. /var/generation cleared :)

Tested on merchant beta, but I believe problem still exists.

@udovicic udovicic mentioned this issue Sep 26, 2015
Merged
@NadiyaS NadiyaS added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Jun 21, 2016
@vkorotun vkorotun removed the CS label Aug 4, 2016
@mbrinton01
Copy link

This issue should be resolved now that #1957 has been merged. Closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NadiyaS NadiyaS

Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@dpotkoc @ivanweiler @okorshenko @vkorotun @mbrinton01 @NadiyaS