From 3d10c6a906f985b83cb51e624447d6137e8aa1b3 Mon Sep 17 00:00:00 2001
From: Michael <mdietz@stud.hs-heilbronn.de>
Date: Wed, 17 Apr 2019 07:54:56 +0200
Subject: [PATCH] added some TODO comments/thoughts to methods

---
 .../bugageocaching/controller/Controller.java | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/src/main/java/hhn/labsw/bugageocaching/controller/Controller.java b/src/main/java/hhn/labsw/bugageocaching/controller/Controller.java
index eab8d0f..70710bd 100644
--- a/src/main/java/hhn/labsw/bugageocaching/controller/Controller.java
+++ b/src/main/java/hhn/labsw/bugageocaching/controller/Controller.java
@@ -60,6 +60,8 @@ public class Controller {
     return ResponseEntity.status(200).body(new Gson().toJson(cacheRepository.findAll()));
   }
 
+  // TODO user muss jetzt anders aus dem token geholt werden, da kein subject mehr gesetzt wird und username nichtmehr unique ist
+  // TODO (über der checkAdmin methode steht ein möglicher lösungsvorschlag dafür)
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/startCache")
   @ResponseBody
@@ -142,10 +144,7 @@ public class Controller {
     }
   }
 
-  /**
-   * Checkt, ob die eingescannte Station die Nachfolgestation der zuletzt
-   * besuchten Stationen des aktuell durchgeführten Caches ist
-   */
+  // TODO user muss jetzt anders aus dem token geholt werden, da kein subject mehr gesetzt wird und username nichtmehr unique ist
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/checkStation")
   @ResponseBody
@@ -281,7 +280,8 @@ public class Controller {
     return createCacheUtil(cache);
   }
 
-
+  // TODO claims.get funktioniert denk ich jetzt nurnoch mit claims.get("user") und liefert glaub ein userEntity objekt als JSON zurück
+  // TODO wir könnten doch dann die email aus dem JSON auslesen und mit Hilfe einer findByEmail methode dann den user aus unserer DB finden oder?
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/checkAdmin")
   @ResponseBody
@@ -298,7 +298,6 @@ public class Controller {
     return ResponseEntity.status(200).body(claims.get("admin"));
   }
 
-  //Bis hier
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/getAllStations")
   @ResponseBody
@@ -313,6 +312,7 @@ public class Controller {
     return deleteCacheUtil(cacheID);
   }
 
+  // TODO
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/getMyCaches")
   @ResponseBody
@@ -349,6 +349,9 @@ public class Controller {
     }
   }
 
+  // TODO SQL-Anfrage geschrieben, müssen uns nurnoch überlegen wie wir das ganze zurückgeben.
+  // TODO Da die Daten jetzt über mehrere Klassen verteilt sind, können wir nicht nur ein einzelnes Objekt zurückliefern.
+  // TODO Wäre eigentlich am Besten, wenn wir einfach das array zurückgeben, das frontend sollte ja ohne probleme daraus dann die Daten rausholen können.
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/getRankingList")
   @ResponseBody
@@ -364,9 +367,13 @@ public class Controller {
       sendBackUsers.add(u);
     }
 
+    /**
+     * return ResponseEntity.status(200).body(new Gson().toJson(userRepository.getRankingList()));
+     */
     return ResponseEntity.status(200).body(new Gson().toJson(sendBackUsers));
   }
 
+  // TODO ebenfalls userfindung ändern
   @CrossOrigin(origins = "http://localhost:8081") // only for dev purpose
   @RequestMapping("/api/getUser")
   @ResponseBody