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

FieldtypeDatetime translation breaks saving field #2018

Open
BernhardBaumrock opened this issue Jan 6, 2025 · 1 comment
Open

FieldtypeDatetime translation breaks saving field #2018

BernhardBaumrock opened this issue Jan 6, 2025 · 1 comment

Comments

@BernhardBaumrock
Copy link

Short description of the issue

I'm working on a Datetime related module and think I found a bug!

My user's language is german, I set the field input to this:

image

When showing an already populated field, it looks fine:

image

But when I change the time to this:

image

Which means "PM" in english and I save the page:

image

---> Data lost!

Expected behavior

Translations should not break the field functionality.

Actual behavior

All times without translated strings like "PM --> NACHM." work.

Setup/Environment

Field Settings

label Von
name from
id 102
type FieldtypeDatetime
flags 0
dateOutputFormat 'd/m/Y H:i:s'
inputType 'text'
dateSelectFormat 'yMd'
datepicker 3
dateInputFormat 'Y-m-d'
timeInputFormat 'h:i A'
columnWidth 33
collapsed 0
yearFrom 1925
yearTo 2045
yearLock 0
timeInputSelect 0
size 25
showAnim 'fade'
numberOfMonths 1
changeMonth 1
changeYear 1
showButtonPanel 0
showMonthAfterYear 0
showOtherMonths 0
htmlType 'date'
_lazy '1'
tags ''

PW Info

SERVER DETAILS
ProcessWire: 3.0.243
PHP: 8.3.12
Webserver: Apache/2.4.62 (Debian)
MySQL Server: 10.6.19-MariaDB-ubu2004-log
MySQL Client: mysqlnd 8.3.12

@BernhardBaumrock
Copy link
Author

BernhardBaumrock commented Jan 6, 2025

Actually @jmartsch shouldn't "AM" also be translated to "VORM." ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant