Currently date option format has wrong translation because it uses user entered data in t()

This require placeholder to translate this string once

CommentFileSizeAuthor
#2 1676820-date-option-translation-2.patch1.74 KBandypost
#1 1676820-date-option-translation-1.patch746 bytesandypost

Comments

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new 1676820-date-option-translation-1.patch746 bytes

check_plain here is useless because data already passes @ placeholder

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new 1676820-date-option-translation-2.patch1.74 KB

Also there's a problem with url - see example in system_configure_date_formats_form()

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Reviewed & tested by the community

Trivial fixes.

dawehner’s picture

dawehner
Primary language German
commented
Status: Reviewed & tested by the community » Fixed

Even hass the holy gatekeeper of string-freeze posted patches for string changes.

The patch is perfect, committed to 7.x-3.x and 8.x-3.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.