?
avatar ot2sen
ot2sen
28 Apr 2017

Steps to reproduce the issue

Trying to save birthday and date selected in calendar using Danish language in front, which has a different order of %d-%m-%Y than the English %Y-%m-%d set in DATE_FORMAT_CALENDAR_DATE.

Expected result

Birthday saved

Actual result

Getting a: Advarsel
Profilen kunne ikke gemmes: Fødselsdatoen som du angav er ugyldig. Angiv venligst en gyldig dato.
Which is the warning for profile could not be saved, birthday entered is invalid. Please enter valid date.

If I change to English front it works (but only if the field is required, else it fails with same warning in English)

System information (as much as possible)

3.7.0

Additional comments

avatar ot2sen ot2sen - open - 28 Apr 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 28 Apr 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Apr 2017

Tried same in German Lang. (01.01.2000): Write in Field isn't saved, have to select Year by clicking "<", this Date got saved.

avatar infograf768
infograf768 - comment - 28 Apr 2017

I have no issue with French pack.
screen shot 2017-04-28 at 16 08 34

screen shot 2017-04-28 at 16 09 13

avatar infograf768
infograf768 - comment - 28 Apr 2017

My strings are:

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"
avatar infograf768
infograf768 - comment - 28 Apr 2017

german pack uses

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 sure we can use period instead of hyphens...

avatar Bakual
Bakual - comment - 28 Apr 2017

The german one works fine for me. The danish one has a small typo in the DATE_FORMAT_FILTER_DATE filter string: d-m-y instead of d-m-Y. That's why it fails. I've fixed it in Crowdin but it needs a new release of the language pack. Can you take care of that @ot2sen?

avatar Bakual Bakual - change - 28 Apr 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-04-28 19:03:07
Closed_By Bakual
avatar Bakual Bakual - close - 28 Apr 2017
avatar Bakual
Bakual - comment - 28 Apr 2017

Closing as not a core issue.

avatar ot2sen
ot2sen - comment - 28 Apr 2017

Thanks @Bakual good catch. Didn´t see that while testing. Added as an override and now it saves as should :)
Will notify the Danish TT coordinator for him to build a new package

avatar rbuelund
rbuelund - comment - 29 Apr 2017

Have released a new package

Add a Comment

Login with GitHub to post a comment