MVC, Model should know nothing about View.

Service is in Model layer, it should not be dealing with sessions (View)

diff --git a/TDD/src/main/java/org/craftedsw/tripservicekata/trip/TripService.java b/TDD/src/main/java/org/craftedsw/tripservicekata/trip/TripService.java
index 5a19beb..6a195b8 100644 (file)
--- a/TDD/src/main/java/org/craftedsw/tripservicekata/trip/TripService.java
+++ b/TDD/src/main/java/org/craftedsw/tripservicekata/trip/TripService.java
@@ -29,6 +29,8 @@ public class TripService {
                return tripList;\r
        }\r
 \r
+       // In MVC the Model layer should know nothing about the view.\r
+       // Service is in Model layer, so we have to get rid of this method.\r
        protected User getLoggedInUser() {\r
                User loggedUser = UserSession.getInstance().getLoggedUser();\r
                return loggedUser;\r