Updated Tests
[staff/due1/sed-hs15-srs-purple.git] / src / main / java / ch / bfh / ti / srs / controller / UserController.java
index 65d8820..6166e91 100644 (file)
@@ -52,7 +52,7 @@ public class UserController extends Controller{
         */\r
        public Map<Reservation, Boolean> getUserReservations(User user) throws Exception {\r
                if (user == null) {\r
-                       if (sc.getLoggedUser().getUserName() == null)\r
+                       if (sc.getLoggedUser() == null)\r
                                throw new Exception("user must be defined, if no user is logged in");\r
                        else\r
                                user = sc.getLoggedUser();\r
@@ -74,12 +74,14 @@ public class UserController extends Controller{
         *             not authorized\r
         */\r
        public boolean changeReservation(Reservation reservation) throws Exception {\r
-               if (sc.getLoggedUser().getUserName() == null)\r
+               if (sc.getLoggedUser() == null)\r
                        throw new Exception("No user is logged in");\r
                if (reservation == null)\r
                        throw new IllegalArgumentException("Reservation must be defined");\r
-               if (!reservation.getUser().equals(super.sc.getLoggedUser().getUserName()))\r
+               if (!reservation.getUser().getUserName().equals(super.sc.getLoggedUser().getUserName())){\r
                        throw new Exception("You are not authorized to make this change");\r
+               }\r
+                       \r
                return super.changeReservation_Super(reservation);\r
        }\r
 \r
@@ -97,7 +99,7 @@ public class UserController extends Controller{
         *             is logged in or logged <code>User</code> is not authorized\r
         */\r
        public boolean deleteReservation(Reservation reservation) throws Exception {\r
-               if (sc.getLoggedUser().getUserName() == null)\r
+               if (sc.getLoggedUser() == null)\r
                        throw new Exception("No user is logged in");\r
                if (reservation == null)\r
                        throw new IllegalArgumentException("Reservation must be defined");\r
@@ -110,7 +112,11 @@ public class UserController extends Controller{
         * {@inheritDoc}\r
         */\r
        public Set<Reservation> makeReservation(User user, Resource resource, List<DateRange> dateRangeList, String remark) throws Exception {\r
-               if (user == null) user = sc.getLoggedUser();\r
+               if (user == null){\r
+                       if (sc.getLoggedUser() == null)\r
+                               throw new IllegalArgumentException("User must be defined if no user is logged in");\r
+                       user = sc.getLoggedUser();\r
+               }\r
                if (!user.equals(sc.getLoggedUser())) throw new Exception("You are not authorized to make this reservation");\r
                return super.makeReservation(user, resource, dateRangeList, remark);\r
        }\r