There should be two kinds of values for "points in time" (with a precision, time zone, calendar, etc) and simple "timestamps" (just an ISO date string). There should be formatters for each, where the formatter for the complex time value uses a formatter for timestamps as a parameter.
Version: unspecified
Severity: minor
Whiteboard: cleanup
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T87764 Bugs related to time datatype (tracking) | |||
| Open | None | T59743 Sort out relationship between TimeFormatter and TimeIsoFormatter |
We more or less constantly worked on this over the past year.
In https://github.com/DataValues/Time/pull/49 I'm trying to make TimeFormatter an actual TimeValue formatter that does as much as possible with no knowledge about MediaWiki and languages at all.
The class that's currently called MwTimeIsoFormatter still needs to be renamed and refactored. Is this what this ticket is about? Otherwise I suggest to close this. Most important: A formatter that gets a timestamp string and nothing else does not make sense to me. Having multiple formatters for TimeValues does make sense.