From: Felix Dörre <felix@dogcraft.de>
Date: Wed, 8 Feb 2017 19:50:28 +0000 (+0100)
Subject: fix: ResultSet.getDate is often wrong as it fetches day-precision times
X-Git-Url: https://code.wpia.club/?p=gigi.git;a=commitdiff_plain;h=c9ed09f0007fc2c813815be927a5a24b23dab83c

fix: ResultSet.getDate is often wrong as it fetches day-precision times

Change-Id: Id9394b12663e78de96a3610590587d3f15096e15
---

diff --git a/src/org/cacert/gigi/dbObjects/Certificate.java b/src/org/cacert/gigi/dbObjects/Certificate.java
index 275aa20e..2aecedf7 100644
--- a/src/org/cacert/gigi/dbObjects/Certificate.java
+++ b/src/org/cacert/gigi/dbObjects/Certificate.java
@@ -471,7 +471,7 @@ public class Certificate implements IdCachable {
                 prep.setInt(1, getId());
                 GigiResultSet res = prep.executeQuery();
                 if (res.next()) {
-                    return new java.util.Date(res.getDate("revoked").getTime());
+                    return new java.util.Date(res.getTimestamp("revoked").getTime());
                 }
             }
         }
diff --git a/tests/org/cacert/gigi/TestUserGroupMembership.java b/tests/org/cacert/gigi/TestUserGroupMembership.java
index 22135574..5ddf2d11 100644
--- a/tests/org/cacert/gigi/TestUserGroupMembership.java
+++ b/tests/org/cacert/gigi/TestUserGroupMembership.java
@@ -48,8 +48,8 @@ public class TestUserGroupMembership extends BusinessTest {
             assertEquals(granter.getId(), rs.getInt("grantedby"));
             assertEquals(ttpGroup.getDBName(), rs.getString("permission"));
 
-            assertNull(rs.getDate("deleted"));
-            assertNotNull(rs.getDate("granted"));
+            assertNull(rs.getTimestamp("deleted"));
+            assertNotNull(rs.getTimestamp("granted"));
 
             assertFalse(rs.next());
         }
@@ -80,8 +80,8 @@ public class TestUserGroupMembership extends BusinessTest {
             assertEquals(granter.getId(), rs.getInt("grantedby"));
             assertEquals(ttpGroup.getDBName(), rs.getString("permission"));
 
-            assertNotNull(rs.getDate("deleted"));
-            assertNotNull(rs.getDate("granted"));
+            assertNotNull(rs.getTimestamp("deleted"));
+            assertNotNull(rs.getTimestamp("granted"));
 
             assertFalse(rs.next());
         }