[Sugar-devel] [DESIGN] Absolute time in the journal
Manuel Quiñones
manuq at laptop.org
Tue Dec 3 13:42:27 EST 2013
Hi,
I think this is a good, but hard one. It makes sense to display
absolute time when the entries are sorted by creation time. But James
has two valid points.
2013/12/2 James Cameron <quozl at laptop.org>:
> This exposes a reliance on the absolute time stored in the systems'
> real time clocks.
>
> The cascaded design decisions should include:
>
> 1. provide date and time setting in control panel, #3829,
>
> 2. allow a deployment to disable the date and time setting, as they may
> maintain the real time clock for date limited lease signatures.
Yes, those are needed before going with this, and may not be obvious.
I often forgot that:
- some deployments disable the "date" command
- most times the XOs date and time are incorrect
About the implementation,
- absolute time should be displayed only if the sorting is by creation
time (a quick look tells me that the patch changes it without a
conditional)
- I think that the absolute time should be displayed in the
appropriate representation of the current locale. So I would do:
absolute_datetime.strftime('%c')
instead of:
absolute_datetime.strftime('%Y-%m-%d %H:%M:%S')
--
.. manuq ..
More information about the Sugar-devel
mailing list