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

Admin Order Reports throw exception in en_GB locale #7888

Closed
JacobDrummond opened this issue Dec 19, 2016 · 8 comments
Closed

Admin Order Reports throw exception in en_GB locale #7888

JacobDrummond opened this issue Dec 19, 2016 · 8 comments
Labels
bug report Component: Customer 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 not valid Gate 1 Failed. Automatic verification of issue format is failed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@JacobDrummond
Copy link

Environment

  1. PHP 5.6.16
  2. MySQL 5.6.21-log
  3. Magento 2.1.3
  4. Apache 2.4.17

Steps to reproduce

  1. In System > All Users > User > [You], set the interface locale to English (United Kingdom):
    blog-3
  2. In Reports > Sales > Orders, execute a query involving days above 12 e.g. 31/11//2016, but not 10/11/2016: hama-1
  3. Click "Show Report"

Expected result

  1. Magento generates Order Report between the dates specified

Actual result

  1. (Exception): Invalid input date format '31/12/2015': hama-2

General date/locale issues

@veloraven
Copy link
Contributor

@JacobDrummond thank you for your report.
This issue was fixed in developer branch but was not fixed in 2.1 yet.
We have internal ticket for this version: MAGETWO-57839

@veloraven veloraven added 2.1.x Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development bug report Component: Customer Component: Sales labels Dec 20, 2016
@JacobDrummond
Copy link
Author

@veloraven, thanks for your feedback.

When will these develop fixes be included in a release?

In general, what is your develop-to-release workflow?

@webdevlabo
Copy link

@JacobDrummond It can happen because of locale: try M/D/Y format instead and check if it works.

@JacobDrummond
Copy link
Author

@webdevlabo, yes that's the known fix :)

The whole point of this bug report is that Magento should be implementing locale consistently across JS and PHP.

@janeblonde
Copy link

@veloraven this has been reported a few times since October, do you have an ETA on when it might be fixed? or is there a way we can see the code that needs to be modified from the developers branch?

@rogerrogerrogerroger
Copy link

We are based in UK with latest Magento 2.1.3, is there an update on when the fix will be released?

@marcosdsdba
Copy link

@veloraven I performed the tests and the error persists in versions 2.1.5 and 2.1.6
I opened an issue #9432

@magento-engcom-team magento-engcom-team added 2.1.x Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development bug report Component: Customer Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed Component: Sales labels Sep 11, 2017
@magento-engcom-team magento-engcom-team added 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 and removed Component: Sales labels Sep 21, 2017
@magento-engcom-team
Copy link
Contributor

@JacobDrummond, thank you for your report.
The fix for this issue is already available in the develop and 2.2-develop branches. See commits with MAGETWO-57835 in the title.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Component: Customer 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 not valid Gate 1 Failed. Automatic verification of issue format is failed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

7 participants