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

#4272: v2.0.4 Credit memos with adjustment fees cannot be fully refunded with a second credit memo #9715

Merged
merged 1 commit into from
May 27, 2017

Conversation

mcspronko
Copy link
Contributor

@mcspronko mcspronko commented May 21, 2017

This is a bug fix for #4272 open issue reported. The problem reported to creditmemo and inability to create second creditmemo for same order in case Adjustment Fee has been used during creation of first creditmemo.

Description

The problem reported in #4272 issue highlights the fact of having 0 (ZERO) Subtotal during creating second creditmemo. However this is by design, neither Adjustment Refund nor Adjustment Fee should in any way reflect modifications in Subtotal, Shipping and Tax Totals.

Magento 2 documentation should be updated in order to highlight purpose of Adjustment Fee/Refund.

Adjustment Fee is used to subtract amount from the total amount refund. It is not subtracted from specific Total e.g. Shipping, Subtotal or Tax, it is additional amount on its own.

Adjustment Refund is used to add additional amount for the refund amount.

Fixed Issues

  1. v2.0.4 Credit memos with adjustment fees cannot be fully refunded with a second credit memo #4272: v2.0.4 Credit memos with adjustment fees cannot be fully refunded with a second credit memo

Manual testing scenarios

Pre-Setup scenario:

  1. Install Magento latest develop version
  2. Go to Admin side
  3. Click "Stores > All Stores"
  4. Click "Add new store view"
  5. Enter name and code, set status "Enabled" and click "Save"
  6. Click "Stores > Configuration"
  7. In header side choose created Store View
  8. Click "General > Currency Setup"
  9. "Default Display Currency" set "British Pound Sterling"
  10. "Allowed Currencies" set "British Pound Sterling" then click "Save"
  11. Click "Stores > Currency rates"
  12. In row GPB enter 0.6 and click "Save"

Scenario 1

  1. Place order on "GB" store view"
  2. Go to Admin side "Sales > Orders"
  3. Receive Invoice and after that click "Credit Memo"
  4. Scroll page down and enter in field "Adjustment Fee" 1$ then click "Refund"

Expected Result:

  • Order marked as "Complete", there is no Credit Memo button for the Order.

Scenario 2

  1. Login as a customer
  2. Place order on "GB" store view"
  3. Go to Admin side "Sales > Orders"
  4. Receive Invoice and after that click "Credit Memo"
  5. Scroll page down and enter in field "Adjustment Fee" 1$ then click "Refund"
  6. Navigate to customer account -> order view and observer Refunds tab.

Expected result:

  • Totals shows refunded amount and Adjustment Fee 0.6 GPB.

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)

@ishakhsuvarov ishakhsuvarov self-assigned this May 21, 2017
@ishakhsuvarov ishakhsuvarov added this to the May 2017 milestone May 21, 2017
@magento-team magento-team merged commit 290cb2a into magento:develop May 27, 2017
magento-team pushed a commit that referenced this pull request May 27, 2017
… be fully refunded with a second credit memo #9715
magento-team pushed a commit that referenced this pull request May 27, 2017
[EngCom] Public Pull Requests
 - MAGETWO-69379 use payment method name to make checkbox of agreements more unique #6207 #9717
 - MAGETWO-69378 #4272: v2.0.4 Credit memos with adjustment fees cannot be fully refunded with a second credit memo #9715
 - MAGETWO-69375 Can't delete last item in cart if Minimum Order is Enable #6151 #9714
 - MAGETWO-69230 #7279 bill-to name and ship-to name truncated to 20 chars #9654
 - MAGETWO-69155 Fix coding standard in Magento AdminNotification module #9627
@magento-team
Copy link
Contributor

@mcspronko thank you for your contribution. Your Pull Request has been successfully merged

@magento-team
Copy link
Contributor

@mcspronko thank you for your contribution

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.

3 participants