[frogr] Rewritten comment explaining the problem with 'date posted' and UTC
- From: Mario Sanchez Prada <msanchez src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [frogr] Rewritten comment explaining the problem with 'date posted' and UTC
- Date: Sat, 13 Sep 2014 23:24:01 +0000 (UTC)
commit ca585b53e477911ea9f600d24a9cf63c1da19d16
Author: Mario Sanchez Prada <msanchez gnome org>
Date: Sun Sep 14 00:05:50 2014 +0100
Rewritten comment explaining the problem with 'date posted' and UTC
In the end, I don't think there's a much better solution than just to go
for UTC, since many pictures won't have the TimeZoneOffset EXIF tag
anyway, and also because for sorting purposes in the photostream just
interpreting the date as UTC (even if it's wrong) should be enough.
https://bugzilla.gnome.org/show_bug.cgi?id=734193
src/frogr-controller.c | 10 ++++++++--
1 files changed, 8 insertions(+), 2 deletions(-)
---
diff --git a/src/frogr-controller.c b/src/frogr-controller.c
index 1ecf0c1..90f71ab 100644
--- a/src/frogr-controller.c
+++ b/src/frogr-controller.c
@@ -1414,8 +1414,14 @@ _set_date_taken_as_posted_for_picture (FrogrController *self, UploadOnePictureDa
if (!g_time_val_from_iso8601 (date_iso8601, &picture_timeval))
return;
- /* FIXME: it's not correct to just do this because the date extracted from
- the EXIF information might not have been stored in UTC */
+ /* It's not correct to just do this because the date extracted from the EXIF information might
+ not have been stored in UTC. However, it seems there's no a much better solution since many
+ cameras don't use the TimeZoneOffset EXIF tag, so we can't rely on that either. And even if
+ it exists, I could not find any picture with that field in the EXIF information so I guess
+ that, all in all, using the 'date taken' as 'posted' as if it was UTC (regardless of whether
+ it's actually UTC) is probably fine anyway. After all, it's not supposed to be shown in
+ flickr and would serve most of the times just to have the photostream sorted by 'date
+ taken', and for that, using the date as UTC would be good enough .*/
picture_date = g_date_time_new_from_timeval_utc (&picture_timeval);
if (!picture_date)
return;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]