[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