Skip to content

Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies #15660

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
Closed
ankurvr opened this issue Jun 1, 2018 · 3 comments
Closed

Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies #15660

ankurvr opened this issue Jun 1, 2018 · 3 comments
Labels
Component: Backend Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release

Comments

@ankurvr
Copy link
Member

ankurvr commented Jun 1, 2018

On Dashboard Last order's table shows wrong table for specific store view dashboard when you have multiple websites and both have different currencies.
Actual Problem is mentioned in 8th point of Steps to Reproduce.

Preconditions

  1. Magneto CE (2.2.3, 2.2.4)
  2. PHP Ver 7.0.24
  3. MySQL Ver (5.7, 8.0.11)
  4. Platform : MacOS High Sierra Ver 10.13.4

Steps to reproduce

  1. Create 2 Websites with 1 store in each
  2. Set Default website/store base currency as INR (Indian Rupee)
  3. Set another website/store base currency as USD (US Dollar)
  4. Place Order from both websites with with their default currencies.
  5. Now go to Dashboard and set Store View as All Store Views
  6. See Last Orders table and check order's total amounts and all mounts will be in INR (USD orders will be converted to INR with exchange rate
  7. Now change Store View to USD currency website's store which have orders
  8. See Last Orders table and check order's total amounts. You will find those amounts converted to again into INR to USD. But at this point orders are already in USD so no need to convert amounts.

Expected result

  1. All Store View Dashboard

screen shot 2018-06-01 at 4 54 03 pm

  1. Second Store View Dashboard which have USD currency

screen shot 2018-06-01 at 5 01 03 pm

Actual result

  1. All Store View Dashboard

screen shot 2018-06-01 at 4 54 03 pm

  1. Second Store View Dashboard which have USD currency

screen shot 2018-06-01 at 4 54 29 pm
@magento-engcom-team magento-engcom-team added the Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed label Jun 1, 2018
@ankurvr ankurvr changed the title Wrong order amount on dashboard recent order listing when having more than one website with different currencies Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies Jun 1, 2018
@ankurvr ankurvr self-assigned this Jun 1, 2018
@ankurvr ankurvr added Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release labels Jun 1, 2018
ankurvr added a commit to ankurvr/magento2 that referenced this issue Jun 1, 2018
@ankurvr
Wrong Last orders amount on dashboard magento#15660
1c1e277 
Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies magento#15660
ankurvr added a commit to ankurvr/magento2 that referenced this issue Jun 1, 2018
@ankurvr
Update Currency.php
a1ac07f 
Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies magento#15660
@ankurvr ankurvr mentioned this issue Jun 1, 2018
Merged
4 tasks
ankurvr added a commit to ankurvr/magento2 that referenced this issue Jun 1, 2018
@ankurvr
Wrong Last orders amount on dashboard magento#15660
d696233 
Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies magento#15660
ankurvr added a commit that referenced this issue Jun 2, 2018
@ankurvr
Wrong Last orders amount on dashboard #15660
08e4eb6 
Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies #15660
@ankurvr ankurvr mentioned this issue Jun 2, 2018
Closed
4 tasks
ankurvr added a commit to ankurvr/magento2 that referenced this issue Jun 2, 2018
@ankurvr
Wrong Last orders amount on dashboard magento#15660
6b0f8d0 
Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies magento#15660
This was referenced Jun 2, 2018
Merged
Merged
Merged
Closed
@ankurvr ankurvr removed their assignment Jun 2, 2018
ankurvr added a commit to ankurvr/magento2 that referenced this issue Jun 2, 2018
@ankurvr
Merge pull request #5 from ankurvr/ankurvr-patch-2.1-develop
3b17646 
Wrong Last orders amount on dashboard magento#15660
@magento-engcom-team magento-engcom-team added the Fixed in 2.3.x The issue has been fixed in 2.3 release line label Jun 2, 2018
@magento-engcom-team
Copy link
Contributor

Hi @ankurvr. Thank you for your report.
The issue has been fixed in #15682 by @ankurvr in 2.3-develop branch
Related commit(s):

The fix will be available with the upcoming 2.3.0 release.

magento-engcom-team added a commit that referenced this issue Jun 2, 2018
@magento-engcom-team
ENGCOM-1770: [Forwardport] Wrong Last orders amount on dashboard #15660
94db7fc 
#15682

 - Merge Pull Request #15682 from ankurvr/magento2:ankurvr-patch-2.3-develop
 - Merged commits:
   1. f5f3c76
   2. d696233
   3. 424324e
   4. eca36b2
   5. 98265ab
   6. ac220b7
   7. 12041e1
magento-engcom-team pushed a commit that referenced this issue Jun 2, 2018
ENGCOM-1770: [Forwardport] Wrong Last orders amount on dashboard #15660
ea1695a 
#15682
@VladimirZaets
Copy link
Contributor

Hi @ankurvr. Thank you for your report.
The issue has been fixed in #15661 by @ankurvr in 2.2-develop branch
Related commit(s):

The fix will be available with the upcoming 2.2.6 release.

@VladimirZaets VladimirZaets added the Fixed in 2.2.x The issue has been fixed in 2.2 release line label Jun 7, 2018
@chirag-wagento chirag-wagento mentioned this issue Jun 10, 2018
Closed
4 tasks
@sidolov sidolov added the Fixed in 2.1.x The issue has been fixed in 2.1 release line label Aug 7, 2018
@sidolov
Copy link
Contributor

sidolov commented Aug 7, 2018

Hi @ankurvr. Thank you for your report.
The issue has been fixed in #15677 by @ankurvr in 2.1-develop branch
Related commit(s):

The fix will be available with the upcoming 2.1.15 release.

magento-engcom-team added a commit that referenced this issue Aug 7, 2018
@magento-engcom-team
ENGCOM-2051: [Backport] Wrong Last orders amount on dashboard #15660 #…
3becc64 
…15677

 - Merge Pull Request #15677 from ankurvr/magento2:ankurvr-patch-2.1-develop
 - Merged commits:
   1. 5a379fc
   2. 0deea0e
   3. 6b0f8d0
   4. 2594853
magento-engcom-team pushed a commit that referenced this issue Aug 7, 2018
ENGCOM-2051: [Backport] Wrong Last orders amount on dashboard #15660 #…
aa54c65 
…15677
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: Backend Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ankurvr @VladimirZaets @sidolov @magento-engcom-team