-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[5.1] Fix calendar value #43234
[5.1] Fix calendar value #43234
Conversation
Tested with Joomla 5.1. RC2: this change works and brings back the functionality in non-english backoffice-languages. |
@svenbluege Could you go to the issue tracker and mark your test result so it's properly counted? Each pull request needs 2 human tests, and yours would be one. For this, go to https://issues.joomla.org/tracker/joomla-cms/43234 , use the blue "Test this" button at the top left corner, select your test result (success) and finally submit. Thanks in advance. |
I have tested this item ✅ successfully on 8b94a61 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43234. |
I have tested this item ✅ successfully on 8b94a61 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43234. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43234. |
Thank you @Fedik and for the tests @svenbluege @viocassel ! |
Pull Request for #42709 (comment) .
Summary of Changes
The calendar field was doing extra changes for the value, which is bypassing data[value]
Testing Instructions
Please follow #42709 (comment)
Or edit any existing article date. In this case all should work as before.
Actual result BEFORE applying this Pull Request
Wrong date
Expected result AFTER applying this Pull Request
Correct date
Link to documentations
Please select: