-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
what is the role of $_isGateway in payment ? #1744
Comments
Hi, Actually this flag duplicates https://github.com/magento/magento2/blob/develop/app/code/Magento/Payment/Model/Method/AbstractMethod.php#L88 |
@dkvashninbay when i change Checkmo.php |
@dkvashninbay i use paypal payment then i use admin view order is still error? |
I can't reproduce this issue, also these flags are not something which can cause problems with rendering, especially for Check / Money order |
@dkvashninbay are you change $_isOffline = false |
Yes, i've placed order in several combinations of these flags. |
Do you still have this issue @LifeAsBook ? |
I have the same error when i turned isOffline to false.. So, i think you should to do something with it... |
Public Pull Requests #11594 Fix issue #10347 - Wrong order tax amounts displayed when using specific tax configuration (2.3-develop) by @PieterCappelle Fixed Public Issues #10347 Wrong order tax amounts displayed when using specific tax configuration
* ACQE-7010: [MFTF Tests] Add payment related third party test in pr exclude
No description provided.
The text was updated successfully, but these errors were encountered: