Revize 88e2ccf3

Fixed a taking vacation for the same day.

server/src/main/java/cz/zcu/yamanager/business/ApiManager.java

@@ -141 +141 @@
         }
 
         try {
+
+            if (vacationRepository.isExistVacationForUser(userId, vacationDay.getDate())) {
+                throw new RESTFullException("Cannot take a double vacation for the same day.", "vacation.double.error");
+            }
+
             User user = userRepository.getUser(userId);
             vacationDay.setStatus(user.getRole() == UserRole.EMPLOYER ? Status.ACCEPTED : Status.PENDING);
 
@@ -159 +164 @@
 
             vacationRepository.insertVacationDay(userId, vacation);
             userRepository.updateUser(user);
+
         } catch (DataAccessException e) {
             log.error(e.getMessage());
             throw new RESTFullException(e.getMessage(), "database.error");

server/src/main/java/cz/zcu/yamanager/repository/VacationRepository.java

@@ -186 +186 @@
             return user;
         }));
     }
+
+    public boolean isExistVacationForUser(Long userId, LocalDate date) {
+        return jdbc.queryForObject("SELECT count(id) AS exist FROM vacation_day WHERE vacation_date = ? AND user_id = ?",
+                new Object[]{date, userId}, (rs, rowNum) -> rs.getBoolean("exist"));
+    }
 }

server/src/main/resources/Message_cs.properties

@@ -23 +23 @@
 vacation.null.error = Po\u010Det hodin dovolen\u00E9 mus\u00ED b\u00FDt zad\u00E1n.
 role.null.error = Role mus\u00ED b\u00FDt zad\u00E1n.
 database.error = Vyskytl se probl\u00E9m s datab\u00E1z\u00ED.
-vacation.past.error = Nelze vybrat volno v minulosti.
+vacation.past.error = Nelze vybrat volno v minulosti.
+vacation.double.error = Nelze vybrat v\u00EDce voln pro stejn\u00FD den.

server/src/main/resources/Message_en.properties

@@ -23 +23 @@
 vacation.null.error = The number of overtime hours has to be filled in.
 role.null.error = The role has to be filled in.
 database.error = There was a problem with the database.
-vacation.past.error = You cannot take a vacation in past.
+vacation.past.error = You cannot take a vacation in past.
+vacation.double.error = You cannot take a vacation at the same day.