Open SerhiiMandrykin opened 2 weeks ago
Hi @SerhiiMandrykin. Thank you for your report. To speed up processing of this issue, make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:
@magento give me 2.4-develop instance
- upcoming 2.4.x release@magento I am working on this
Join Magento Community Engineering Slack and ask your questions in #github channel. :warning: According to the Magento Contribution requirements, all issues must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting. :clock10: You can find the schedule on the Magento Community Calendar page. :telephone_receiver: The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket.
I prepared a fix in https://github.com/magento/magento2/pull/39138
Hi @engcom-Bravo. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:
Area: XXXXX
label to the ticket, indicating the functional areas it may be related to.2.4-develop
branch@magento give me 2.4-develop instance
to deploy test instance on Magento infrastructure. 2.4-develop
branch, please, add the label Reproduced on 2.4.x
.Issue: Confirmed
once verification is complete. Hi @SerhiiMandrykin does the fix work for the circumstances that were outlined in #36199 ?
@devchris79 works well for me
Hi @engcom-Delta. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:
Area: XXXXX
label to the ticket, indicating the functional areas it may be related to.2.4-develop
branch@magento give me 2.4-develop instance
to deploy test instance on Magento infrastructure. 2.4-develop
branch, please, add the label Reproduced on 2.4.x
.Issue: Confirmed
once verification is complete. Hi @SerhiiMandrykin ,
Thanks for your reporting and collaboration. We have verified the issue in latest 2.4-develop instance and issue is not reproducible . Kindly refer the screenshots.
Steps to reproduce -
Please let us know if any other steps are needed to be performed. Thanks.
Preconditions and environment
Steps to reproduce
sales_order
table)Expected result
Totals in the dashboard bar are calculated correctly
Actual result
Totals are not calculated correctly:
Workaround:
Additional information
This bug was caused by https://github.com/magento/magento2/commit/dd769504d2a56e510b1632e8f1a5f42c9571af52 and this line:
$period = $this->getRequest()->getParam('period', $firstPeriod);
Setting
$firstPeriod
as a default value IS NOT correct because it would betoday
instead of24h
(see\Magento\Backend\Model\Dashboard\Period::getDatePeriods
)BUT by loading the dashboard there is no
period
parameter in the request so24h
is pre-selected:Release note
No response
Triage and priority