Problem/Motivation

Steps to reproduce:

  1. Configure a date format with only date bits and some additional character to be rendered as-is, e.g. l\ท\ี่ j F \C\E Y (Thai date format)
  2. Configure a date field with only date granularity and "Date and Time" formatter using the format above
  3. View the rendered field

Expected result: something like Sundayที่ 10 November CE 2002 is displayed
Actual result: Sundayที่ 10 November CE 2002 ที่ CE is displayed

Proposed resolution

Display the timezone bit only if the date has the time bit.

Remaining tasks

  • Validate the proposed solution
  • Reviews
  • Commit

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 date-tz_strip-3002449-2.patch863 bytesplach

Comments

plach created an issue. See original summary.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Assigned: plach » Unassigned
Status: Active » Needs review
StatusFileSize
new date-tz_strip-3002449-2.patch863 bytes

The root cause is that the timezone string is not computed properly in this case, but this fix should make sense anyway.

Status: Needs review » Needs work

The last submitted patch, 2: date-tz_strip-3002449-2.patch, failed testing. View results

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review

  • DamienMcKenna committed 4d43954 on 7.x-2.x authored by plach 
    Issue #3002449 by plach: Timezone rendered when theming date formats...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed
Parent issue: » #2867810: Plan for Date 7.x-2.11 release

Committed. Thanks.

Status: Fixed » Closed (fixed)

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