?
avatar jjnxpct
jjnxpct
1 Nov 2018

Steps to reproduce the issue

Create a custom field of the type 'calendar' (datepicker). Set the option 'Show time' to No. Create an article and fill in the date and save the article.

Expected result

The date is saved.

Actual result

An error appears saying the date field is not valid. Article is not saved. This does not happen when the option 'Show time' is set to 'Yes'

System information (as much as possible)

Joomla 3.9, PHP 7.2.10, MySQL 5.5.61

avatar jjnxpct jjnxpct - open - 1 Nov 2018
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 1 Nov 2018
avatar jjnxpct jjnxpct - change - 1 Nov 2018
The description was changed
avatar jjnxpct jjnxpct - edited - 1 Nov 2018
avatar Quy
Quy - comment - 2 Nov 2018

Unable to reproduce. What is the date entered and the exact error message?

avatar jjnxpct
jjnxpct - comment - 2 Nov 2018

I narrowed it down a bit more. It seems this error only appeared when my backend language is set to Dutch. When I set it to the default English the problem does not exist. I have some screenshots to illustrate:

https://inxpactdrive.nl/s/npXCWKOU5hvHBzl

Not sure how or why a backend language would cause this.

avatar jjnxpct
jjnxpct - comment - 2 Nov 2018

I tried to disable all admin language overrides. That did not help.

avatar jjnxpct
jjnxpct - comment - 2 Nov 2018

Well, the problem is solved for me by adding this to the language overide for the backend Dutch language:

DATE_FORMAT_CALENDAR_DATE="%d-%m-%Y"
DATE_FORMAT_CALENDAR_DATETIME="%d-%m-%Y %H:%M:%S"
DATE_FORMAT_FILTER_DATE="d-m-Y"
DATE_FORMAT_FILTER_DATETIME="d-m-Y H:i:s"

Not the date is displayed in the Dutch format (better for us...) and the article gets saved OK, And as far as I can see this has no further impact on the site. So this solves this issue for me.

But maybe this problem should not occur at all? I will leave this assessment to others for now.
Thanks for listening so far ;-)

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Nov 2018

@jjnxpct can you please get in touch with one Dutch-Translator at https://docs.joomla.org/JDOC:Documentation_Translators#nl_-_Nederlands_-_Dutch?

avatar brianteeman
brianteeman - comment - 2 Nov 2018

NOTE: not the first time there has been an issue with the dutch date format iirc - maybe @Bakual can confirm

avatar wojsmol
wojsmol - comment - 2 Nov 2018

@jjnxpct There is small typo in Dutch language package for Joomla! 3.9.0 both site and admin in nl-NL.ini there is DATE_FORMAT_FILTER_DATE="y-m-d" instead of DATE_FORMAT_FILTER_DATE="Y-m-d".

avatar infograf768
infograf768 - comment - 2 Nov 2018
avatar jjnxpct
jjnxpct - comment - 2 Nov 2018

@wojsmol Excelent find. Thanks!

avatar jjnxpct
jjnxpct - comment - 2 Nov 2018

I noticed the same error in these language strings:
DATE_FORMAT_LC4="y-m-d" (admin nl-NL.ini)
DATE_FORMAT_JS1="y-m-d" (site nl-NL.ini)

avatar wojsmol
wojsmol - comment - 2 Nov 2018

cc @MartijnMaandag (this is Dutch TT github nick) Please check Dutch language pack regarding date formats.

avatar Bakual
Bakual - comment - 2 Nov 2018

I'm going to close this issue then as it is indeed a bug in the language.

I think @brianteeman was referring to the PR #22373 which tries to solve an issue that is already solved if the language pack would be corrected (from how I understand that PR) ?

avatar MartijnMaandag
MartijnMaandag - comment - 3 Nov 2018

The date format in the Dutch language files is like this for years. I changed it and am testing this with @jjnxpct
When it works I will release a new Dutch package.

avatar jjnxpct
jjnxpct - comment - 5 Nov 2018

@MartijnMaandag I tested the new language files you send me. I cleared all the overrides and there are no issues as far as I can tell. The problem I had described in this thread is solved. Thanks!

avatar Bakual Bakual - change - 5 Nov 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-11-05 09:06:37
Closed_By Bakual
avatar Bakual
Bakual - comment - 5 Nov 2018

I'm closing this issue then.

avatar Bakual Bakual - close - 5 Nov 2018

Add a Comment

Login with GitHub to post a comment