[MGNLUI-4061] info.magnolia.ui.form.field.definition.DateFieldDefinition seems not to work correctly around summertime/wintertime change. Created: 31/Oct/16  Updated: 20/Dec/16  Resolved: 31/Oct/16

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Michiel Meeuwissen Assignee: Unassigned
Resolution: Duplicate Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
duplicate
duplicates MGNLUI-4014 Investigate into corner cases: Wrong ... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled

 Description   

We have the following definition:

 class: info.magnolia.ui.form.field.definition.CompositeFieldDefinition
                fields:
                  - name: date
                    time: false
                    type: Date
                    required: true
                    class: info.magnolia.ui.form.field.definition.DateFieldDefinition

So we use it to indicate it a date.

It does hoever start with indicating a time zone in gray. Currently it displays:

Timezone: GMT + 1

Last week (still in summertime) it indicated: GMT + 2.

If someone last week indicate a date of this week, the date was stored according to GMT+2, and hence actually on the wrong date. (because dates are stored as timestamps at midnight)

I propose:
1. The timezone should be evaluated at the time of the time indicated, not at the current time.
2. For date fields showing a timezone is a bit odd, and could be ommitted.



 Comments   
Comment by Michiel Meeuwissen [ 20/Dec/16 ]

Actually I may add that storing a date as a timestamp is a bit silly too, isn't it?

Generated at Mon Feb 12 09:12:51 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.