From d262886a2dccccbc01413c05a302be9044433a76 Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Tue, 29 Jul 2014 10:58:35 -0700 Subject: Fix time zone conversions I know I tested this - no idea how I missed this. Anyway, this was of course utterly bogus. Whenever we get a time from a time_t into a Qt Date or Time datastructure, we need to adjust it by the timezone offset as otherwise Qt will assume it's in local time and helpfully change it to the wrong values for us. See #655 Signed-off-by: Dirk Hohndel --- qt-ui/maintab.cpp | 7 +++++-- qt-ui/maintab.ui | 6 ++++++ 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/qt-ui/maintab.cpp b/qt-ui/maintab.cpp index bf108a9c1..871e9cc41 100644 --- a/qt-ui/maintab.cpp +++ b/qt-ui/maintab.cpp @@ -390,7 +390,10 @@ void MainTab::updateDiveInfo(bool clear) if (!clear) { updateGpsCoordinates(&displayed_dive); + // Subsurface always uses "local time" as in "whatever was the local time at the location" + // so all time stamps have no time zone information and are in UTC QDateTime localTime = QDateTime::fromTime_t(displayed_dive.when - gettimezoneoffset(displayed_dive.when)); + localTime.setTimeSpec(Qt::UTC); ui.dateEdit->setDate(localTime.date()); ui.timeEdit->setTime(localTime.time()); if (MainWindow::instance() && MainWindow::instance()->dive_list()->selectedTrips().count() == 1) { @@ -930,7 +933,7 @@ void MainTab::on_dateEdit_dateChanged(const QDate &date) { if (editMode == IGNORE) return; - QDateTime dateTime = QDateTime::fromTime_t(displayed_dive.when); + QDateTime dateTime = QDateTime::fromTime_t(displayed_dive.when - gettimezoneoffset(displayed_dive.when)); dateTime.setTimeSpec(Qt::UTC); dateTime.setDate(date); displayed_dive.when = dateTime.toTime_t(); @@ -941,7 +944,7 @@ void MainTab::on_timeEdit_timeChanged(const QTime &time) { if (editMode == IGNORE) return; - QDateTime dateTime = QDateTime::fromTime_t(displayed_dive.when); + QDateTime dateTime = QDateTime::fromTime_t(displayed_dive.when - gettimezoneoffset(displayed_dive.when)); dateTime.setTimeSpec(Qt::UTC); dateTime.setTime(time); displayed_dive.when = dateTime.toTime_t(); diff --git a/qt-ui/maintab.ui b/qt-ui/maintab.ui index 2770801a2..ca58ab034 100644 --- a/qt-ui/maintab.ui +++ b/qt-ui/maintab.ui @@ -284,6 +284,9 @@ 0 + + Qt::UTC + @@ -291,6 +294,9 @@ true + + Qt::UTC + -- cgit v1.2.3-70-g09d2