From d980f26121360ab7dacf7f482c9e85fb5ef43b0b Mon Sep 17 00:00:00 2001
From: Tharre <tharre3@gmail.com>
Date: Wed, 23 May 2018 12:55:24 +0200
Subject: Revert "Fix problem where listing operations would throw exception,
 downgrading h2 to 1.4.196 because of bug in .197  #24993"

This reverts commit d13cee465d399895668a88edc34a30a4d0a380c5.
---
 .../einsatzverwaltung/dao/DBOperationDAO.java          | 18 ++++++------------
 .../einsatzverwaltung/dao/RegistrationDatabaseDAO.java |  2 +-
 2 files changed, 7 insertions(+), 13 deletions(-)

(limited to 'src')

diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java
index 1fcdc2e..bb6fb27 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/DBOperationDAO.java
@@ -221,22 +221,16 @@ public class DBOperationDAO implements OperationDAO {
 
     @Override
     public Set<Operation> list(EnumSet<Status> statuses) throws PersistenceException {
-        StringBuilder listString = new StringBuilder("SELECT * FROM Operation WHERE status IN (");
-        boolean first = true;
-        for (Status status : statuses) {
-            if (first) {
-                first = false;
-            } else {
-                listString.append(", ");
-            }
-            listString.append("\'" + status.name() + "\'");
-        }
-        listString.append(") ;");
+        String sql = "SELECT * FROM Operation WHERE status IN (?)";
         Set<Operation> operations = new HashSet<>();
 
         try {
             Connection con = jdbcConnectionManager.getConnection();
-            try (PreparedStatement pstmt = con.prepareStatement(listString.toString())) {
+
+            try (PreparedStatement pstmt = con.prepareStatement(sql)) {
+                Object[] arr = statuses.stream().map(Enum::ordinal).toArray();
+                pstmt.setArray(1, con.createArrayOf("INTEGER", arr));
+
                 try (ResultSet rs = pstmt.executeQuery()) {
                     while (rs.next()) operations.add(operationFromRS(rs));
                 }
diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/RegistrationDatabaseDAO.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/RegistrationDatabaseDAO.java
index 5c00447..13aeffc 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/RegistrationDatabaseDAO.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/dao/RegistrationDatabaseDAO.java
@@ -56,7 +56,7 @@ public class RegistrationDatabaseDAO implements RegistrationDAO {
                 addRegistration.setLong(1, vehicleId);
                 addRegistration.setLong(2, registration.employee().id());
                 addRegistration.setTimestamp(3, Timestamp.from(registration.start()));
-                addRegistration.setTimestamp(4, Timestamp.from(registration.end()));
+                addRegistration.setObject(4, registration.end());
                 addRegistration.setBoolean(
                         5, true); // ASSUMPTION: Registration gets created as active
                 addRegistration.executeUpdate();
-- 
cgit v1.2.3-70-g09d2