Skip to content

[Forwardport] magento/magento2#16184: Fix type error in payment void method#44

Merged
gelanivishal merged 1 commit into
2.3-developfrom
2.3-develop-PR-port-16194
Jul 6, 2018
Merged

[Forwardport] magento/magento2#16184: Fix type error in payment void method#44
gelanivishal merged 1 commit into
2.3-developfrom
2.3-develop-PR-port-16194

Conversation

@gelanivishal
Copy link
Copy Markdown
Owner

Original Pull Request

magento#16194

Description

Fixes TypeError for void() method in Directpost payment method of Authorizenet module.

The method expects an argument of DataObject type but instead none is given. Passing the response object fixes the problem.

Fixed Issues (if relevant)

  1. Argument 1 passed to Magento\Sales\Model\Order\Payment must be an instance of Magento\Framework\DataObject, none given magento/magento2#16184: Argument 1 passed to Magento\Sales\Model\Order\Payment must be an instance of Magento\Framework\DataObject, none given

Manual testing scenarios

  1. Setup authorizenet
  2. Setup a plugin that uses declineOrder method of the Directpost model
  3. Checkout with authorizenet

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@gelanivishal gelanivishal merged commit b96eedd into 2.3-develop Jul 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants