changed the way anonymization works

src/main/java/de/anxietyprime/swajodel/JodelPost.java

@@ -13,6 +13,8 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 public class JodelPost {
     // id of the post
     public Long id;
+    // Parent id
+    public Optional<Long> parent = Optional.empty();
     // id of the author in db
     private final Long authorID;
     // anonymized authorID
@@ -25,14 +27,12 @@ public class JodelPost {
     public Timestamp date;
     // location if the post
     public Location location;
-    // list of all comments for the post
-    public Vector<JodelPost> comments = new Vector<>();
     // the own reaction (null = none, true = positive, false = negative)
     public Optional<Boolean> reaction;
     // all other reactions
     public Reactions reactions;
-    // Parent id
-    public Optional<Long> parent = Optional.empty();
+    // list of all comments for the post
+    public Vector<JodelPost> comments = new Vector<>();
 
     // anonymize function to recursively anonymize the posts
     public void anonymize(Vector<Long> idCache) {
@@ -83,7 +83,7 @@ public class JodelPost {
         this.content = rs.getString("content");
         this.date = rs.getTimestamp("postdate");
         this.location = new Location(rs.getFloat("longitude"), rs.getFloat("latitude"));
-        this.reaction = Optional.of(rs.getBoolean("reaction"));
+        this.reaction = rs.getString("reaction") == null ? Optional.empty() : Optional.of(rs.getBoolean("reaction"));
         this.reactions = new Reactions(rs.getLong("positive"), rs.getLong("negative"));
         this.parent = Optional.of(rs.getLong("parent"));
     }
@@ -99,7 +99,7 @@ public class JodelPost {
 		     @JsonProperty("date") Timestamp date,
 		     @JsonProperty("location") Location location,
              @JsonProperty("parent") Optional<Long> parent) {
-	this.authorID = User.getUserID(); // TODO: getter from Keycloak
+	this.authorID = User.getID(); // TODO: getter from Keycloak
 	this.title = title;
 	this.content = content;
 	this.date = date;

src/main/java/de/anxietyprime/swajodel/Location.java

@@ -4,8 +4,8 @@ import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
 
 public class Location {
-    public float longitude;
     public float latitude;
+    public float longitude;
 
     @JsonCreator
     public Location(@JsonProperty("longitude") float longitude, @JsonProperty("latitude") float latitude ) {

src/main/java/de/anxietyprime/swajodel/Routes.java

@@ -13,13 +13,13 @@ import java.util.Vector;
 @CrossOrigin(origins = "*", allowedHeaders = "*", maxAge = 3600)
 public class Routes {
 
-    @GetMapping("/posts/{longitude}/{latitude}")
+    @GetMapping("/posts/{latitude}/{longitude}")
     public Vector<JodelPost> getPostsByLocation(@PathVariable("longitude") float longitude, @PathVariable("latitude") float latitude) {
         // list of all posts (not comments) in range
         Vector<JodelPost> posts = new Vector<>();
 
         // get UserID
-        Long userID = User.getUserID();
+        Long userID = User.getID();
 
         // DB connection and statement
         Connection c;
@@ -47,8 +47,8 @@ public class Routes {
                         posts.title,
                         posts.content,
                         posts.postdate,
-                        posts.postlocation[0] AS longitude,
-                        posts.postlocation[1] AS latitude,
+                        posts.postlocation[0] AS latitude,
+                        posts.postlocation[1] AS longitude,
                         (SELECT count(*) FROM reactions WHERE reactions.post = posts.id AND reactions.positive = TRUE) AS positive,
                         (SELECT count(*) FROM reactions WHERE reactions.post = posts.id AND reactions.positive = FALSE) AS negative
                       FROM
@@ -78,8 +78,8 @@ public class Routes {
                     )
                     SELECT * FROM targets LEFT JOIN (SELECT post, positive as reaction FROM reactions WHERE userid = (?)) ON post = id;""");
 
-            stmt.setObject(1, longitude);
-            stmt.setObject(2, latitude);
+            stmt.setObject(1, latitude);
+            stmt.setObject(2, longitude);
             stmt.setObject(3, userID);
 
             // query recursively for posts inside a 10km radius
@@ -119,9 +119,9 @@ public class Routes {
         }
 
         // calculate anonymous IDs for the posts
+        Vector<Long> anonymousIDs = new Vector<Long>();
+        anonymousIDs.add(userID);
         posts.forEach(post -> {
-            Vector<Long> anonymousIDs = new Vector<Long>();
-            anonymousIDs.add(userID);
             post.anonymize(anonymousIDs);
         });
 
@@ -135,7 +135,7 @@ public class Routes {
         Optional<JodelPost> root_post = Optional.empty();
 
         // get UserID
-        Long userID = User.getUserID();
+        Long userID = User.getID();
 
         // DB connection and statement
         Connection c;
@@ -163,8 +163,8 @@ public class Routes {
                         posts.title,
                         posts.content,
                         posts.postdate,
-                        posts.postlocation[0] AS longitude,
-                        posts.postlocation[1] AS latitude,
+                        posts.postlocation[0] AS latitude,
+                        posts.postlocation[1] AS longitude,
                         (SELECT count(*) FROM reactions WHERE reactions.post = posts.id AND reactions.positive = TRUE) AS positive,
                         (SELECT count(*) FROM reactions WHERE reactions.post = posts.id AND reactions.positive = FALSE) AS negative
                       FROM
@@ -266,8 +266,8 @@ public class Routes {
             stmt.setObject(2, post.title);
             stmt.setObject(3, post.content);
             stmt.setObject(4, post.date);
-            stmt.setObject(5, post.location.longitude);
-            stmt.setObject(6, post.location.latitude);
+            stmt.setObject(5, post.location.latitude);
+            stmt.setObject(6, post.location.longitude);
 
             // insert post and get its id
             ResultSet rs = stmt.executeQuery();
@@ -316,6 +316,66 @@ public class Routes {
         return post;
     }
 
+    @PatchMapping("/post/{id}")
+    public void reactToPost(@PathVariable long id, @RequestBody JodelPost post) {
+        System.out.println(post.reaction);
+        long userID = User.getID();
+
+        // DB connection and statement
+        Connection c;
+        PreparedStatement stmt;
+
+        // try to get data from db
+        try {
+            // check for the driver
+            Class.forName("org.postgresql.Driver");
+            // get the connection with credentials from env variables
+            c = DriverManager
+                    .getConnection("jdbc:postgresql://"+
+                                    System.getenv("POSTGRES_IP")+"/"+System.getenv("POSTGRES_DB"),
+                            System.getenv("POSTGRES_USER"), System.getenv("POSTGRES_PASSWORD"));
+            // disable auto commits
+            c.setAutoCommit(false);
+
+            // create a new statement
+            stmt = c.prepareStatement("UPDATE Reactions SET positive = (?) WHERE post = (?) AND userid = (?) RETURNING id, positive");
+
+            stmt.setObject(1, post.reaction.orElse(null));
+            stmt.setObject(2, id);
+            stmt.setObject(3, userID);
+
+            // insert post and get its id
+            ResultSet rs = stmt.executeQuery();
+
+            // check if there is a parent
+            if (!rs.next()) {
+                // create a new statement
+                stmt = c.prepareStatement("INSERT INTO Reactions(userid, post, positive) VALUES ((?), (?), (?))");
+
+                // fill statement
+                stmt.setObject(1, userID);
+                stmt.setObject(2, id);
+                stmt.setObject(3, post.reaction.orElse(null));
+
+                // execute statement
+                stmt.execute();
+            }
+
+            // commit the changes
+            c.commit();
+
+            // close all connections to db
+            stmt.close();
+            c.close();
+        }
+
+        // else log the error
+        catch ( Exception e ) {
+            System.err.println( e.getClass().getName()+": "+ e.getMessage() );
+            throw new ResponseStatusException(HttpStatus.SERVICE_UNAVAILABLE, "Database is offline");
+        }
+    }
+
     @DeleteMapping("/post/{id}")
     public void deletePost(@PathVariable long id) {
         // DB connection and statement
@@ -338,7 +398,7 @@ public class Routes {
             stmt = c.prepareStatement("UPDATE Posts SET deleted = now() WHERE id = ? AND author = (?) AND deleted IS NULL");
 
             stmt.setObject(1, id);
-            stmt.setObject(2, User.getUserID());
+            stmt.setObject(2, User.getID());
 
             // insert delete time
             stmt.execute();

src/main/java/de/anxietyprime/swajodel/User.java

@@ -14,9 +14,9 @@ public class User {
         return SecurityContextHolder.getContext().getAuthentication().getName();
     }
 
-    static public Long getUserID () {
+    static public Long getID() {
         String uuid = SecurityContextHolder.getContext().getAuthentication().getName();
-        Long id = -1L;
+        long id;
 
         // DB connection and statement
         Connection c;

src/test/java/de/anxietyprime/swajodel/SwaJodelApplicationTests.java

@@ -8,39 +8,4 @@ import java.util.Vector;
 @SpringBootTest
 class SwaJodelApplicationTests {
 
-    @Test
-    void anonymousTest() {
-        JodelPost post = new JodelPost(1000);
-        post.comments.add(new JodelPost(1001));
-        post.comments.add(new JodelPost(1002));
-        post.comments.get(0).comments.add(new JodelPost(1000));
-        post.comments.get(0).comments.add(new JodelPost(1001));
-        post.comments.get(0).comments.add(new JodelPost(1000));
-        post.comments.add(new JodelPost(1003));
-        post.comments.get(2).comments.add(new JodelPost(1001));
-        post.comments.get(2).comments.add(new JodelPost(1002));
-        post.comments.get(2).comments.add(new JodelPost(1003));
-        post.comments.get(2).comments.add(new JodelPost(1000));
-        post.comments.add(new JodelPost(1000));
-        post.comments.add(new JodelPost(1001));
-        post.comments.add(new JodelPost(1001));
-
-        post.anonymize(new Vector<Long>());
-
-        assert (post.anonymousID == 0);
-        assert (post.comments.get(0).anonymousID == 1);
-        assert (post.comments.get(1).anonymousID == 2);
-        assert (post.comments.get(2).anonymousID == 3);
-        assert (post.comments.get(3).anonymousID == 0);
-        assert (post.comments.get(4).anonymousID == 1);
-        assert (post.comments.get(5).anonymousID == 1);
-        assert (post.comments.get(0).comments.get(0).anonymousID == 0);
-        assert (post.comments.get(0).comments.get(1).anonymousID == 1);
-        assert (post.comments.get(0).comments.get(2).anonymousID == 0);
-        assert (post.comments.get(2).comments.get(0).anonymousID == 1);
-        assert (post.comments.get(2).comments.get(1).anonymousID == 2);
-        assert (post.comments.get(2).comments.get(2).anonymousID == 3);
-        assert (post.comments.get(2).comments.get(3).anonymousID == 0);
-    }
-
 }