From d00fd7705cb1a4b085eed7f34df2f1c6d9a69f76 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Sat, 24 Oct 2015 18:09:15 +0300
Subject: moving to Gradle

---
 .../com/juick/server/SubscriptionsQueries.java     | 168 +++++++++++++++++++++
 1 file changed, 168 insertions(+)
 create mode 100644 src/main/java/com/juick/server/SubscriptionsQueries.java

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
new file mode 100644
index 00000000..d0f5f308
--- /dev/null
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -0,0 +1,168 @@
+/*
+ * To change this template, choose Tools | Templates
+ * and open the template in the editor.
+ */
+package com.juick.server;
+
+import java.sql.Connection;
+import java.sql.PreparedStatement;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.util.ArrayList;
+
+/**
+ *
+ * @author ugnich
+ */
+public class SubscriptionsQueries {
+
+    public static ArrayList<String> getJIDSubscribedToUser(Connection sql, int uid, boolean friendsonly) {
+        ArrayList<String> jids = new ArrayList<String>();
+
+        PreparedStatement stmt = null;
+        ResultSet rs = null;
+        try {
+            if (friendsonly == false) {
+                stmt = sql.prepareStatement("SELECT jids.jid FROM subscr_users INNER JOIN jids ON (subscr_users.user_id=? AND subscr_users.suser_id=jids.user_id) WHERE jids.active=1");
+                stmt.setInt(1, uid);
+            } else {
+                stmt = sql.prepareStatement("SELECT jids.jid FROM subscr_users INNER JOIN jids ON (subscr_users.user_id=? AND subscr_users.suser_id=jids.user_id) WHERE jids.active=1 AND jids.user_id IN (SELECT wl_user_id FROM wl_users WHERE user_id=?)");
+                stmt.setInt(1, uid);
+                stmt.setInt(2, uid);
+            }
+            rs = stmt.executeQuery();
+            rs.beforeFirst();
+            while (rs.next()) {
+                jids.add(rs.getString(1));
+            }
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(rs, stmt);
+        }
+        return jids;
+    }
+
+    public static ArrayList<String> getJIDSubscribedToUserAndTags(Connection sql, int uid, int mid) {
+        ArrayList<String> jids = new ArrayList<String>();
+
+        PreparedStatement stmt = null;
+        ResultSet rs = null;
+
+        String tbl = "subscr_jids_" + mid;
+        ArrayList<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
+
+        try {
+            stmt = sql.prepareStatement("CREATE TEMPORARY TABLE " + tbl + "(user_id INT UNSIGNED NOT NULL) ENGINE=MEMORY");
+            stmt.executeUpdate();
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(null, stmt);
+        }
+
+        try {
+            String query = "INSERT INTO " + tbl + " SELECT suser_id FROM subscr_users WHERE user_id=" + uid;
+            if (!tags.isEmpty()) {
+                query += " UNION DISTINCT SELECT suser_id FROM subscr_tags WHERE tag_id IN (" + Utils.convertArrayInt2String(tags) + ") AND suser_id!=" + uid;
+            }
+            stmt = sql.prepareStatement(query);
+            stmt.executeUpdate();
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(null, stmt);
+        }
+
+
+        try {
+            String query = "SELECT jids.jid FROM " + tbl + " INNER JOIN jids ON (" + tbl + ".user_id=jids.user_id) WHERE jids.active=1 AND " + tbl + ".user_id NOT IN (SELECT user_id FROM bl_users WHERE bl_user_id=" + uid + ")";
+            if (!tags.isEmpty()) {
+                query += " AND " + tbl + ".user_id NOT IN (SELECT user_id FROM bl_tags WHERE tag_id IN (" + Utils.convertArrayInt2String(tags) + "))";
+            }
+            stmt = sql.prepareStatement(query);
+            rs = stmt.executeQuery();
+            rs.beforeFirst();
+            while (rs.next()) {
+                jids.add(rs.getString(1));
+            }
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(rs, stmt);
+        }
+
+        try {
+            stmt = sql.prepareStatement("DROP TABLE " + tbl);
+            stmt.executeUpdate();
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(null, stmt);
+        }
+
+        return jids;
+    }
+
+    public static ArrayList<String> getJIDSubscribedToComments(Connection sql, int mid, int ignore_uid) {
+        ArrayList<String> jids = new ArrayList<String>();
+
+        PreparedStatement stmt = null;
+        ResultSet rs = null;
+        try {
+            stmt = sql.prepareStatement("SELECT jids.jid FROM subscr_messages INNER JOIN jids ON (subscr_messages.message_id=? AND subscr_messages.suser_id=jids.user_id) WHERE jids.user_id!=? AND jids.active=1");
+            stmt.setInt(1, mid);
+            stmt.setInt(2, ignore_uid);
+            rs = stmt.executeQuery();
+            rs.beforeFirst();
+            while (rs.next()) {
+                jids.add(rs.getString(1));
+            }
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(rs, stmt);
+        }
+        return jids;
+    }
+
+    public static ArrayList<String> getJIDSubscribedToUserRecommendations(Connection sql, int uid, int mid, int muid) {
+        ArrayList<String> jids = new ArrayList<String>();
+
+        ArrayList<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
+
+        PreparedStatement stmt = null;
+        ResultSet rs = null;
+        try {
+            String query = "SELECT jid FROM jids WHERE active=1 AND user_id!=" + uid;
+            query += " AND user_id IN (SELECT suser_id FROM subscr_users WHERE user_id=" + uid + ")";
+            query += " AND user_id NOT IN (SELECT user_id FROM bl_users WHERE bl_user_id=" + muid + ")";
+            query += " AND user_id NOT IN (SELECT suser_id FROM subscr_users WHERE user_id=" + muid + ")";
+            query += " AND user_id NOT IN (SELECT suser_id FROM subscr_messages WHERE message_id=" + mid + ")";
+            query += " AND user_id NOT IN (SELECT user_id FROM favorites WHERE message_id=" + mid + ")";
+            query += " AND user_id NOT IN (SELECT subscr_users.suser_id FROM subscr_users INNER JOIN favorites ON (favorites.message_id=" + mid + " AND subscr_users.user_id=favorites.user_id AND favorites.user_id!=" + uid + "))";
+            if (!tags.isEmpty()) {
+                String tagsStr = Utils.convertArrayInt2String(tags);
+                query += " AND user_id NOT IN (SELECT suser_id FROM subscr_tags WHERE tag_id IN (" + tagsStr + "))";
+                query += " AND user_id NOT IN (SELECT user_id FROM bl_tags WHERE tag_id IN (" + tagsStr + "))";
+            }
+
+            stmt = sql.prepareStatement(query);
+            rs = stmt.executeQuery();
+            rs.beforeFirst();
+            while (rs.next()) {
+                jids.add(rs.getString(1));
+            }
+        } catch (SQLException e) {
+            System.err.println(e);
+        } finally {
+            Utils.finishSQL(rs, stmt);
+        }
+
+        return jids;
+    }
+
+    public static boolean subscribeMessage(Connection sql, int mid, int vuid) {
+        return SQLHelpers.execute(sql, "INSERT IGNORE INTO subscr_messages(suser_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
+    }
+}
-- 
cgit v1.2.3


From c37ed56f884661666b964e47efd23a36be9b24d4 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Sun, 10 Jan 2016 16:56:51 +0300
Subject: logger, diamond and arraylist refactoring

---
 src/main/java/com/juick/server/AdsQueries.java     |   6 +-
 .../java/com/juick/server/CrosspostQueries.java    |   8 +-
 .../java/com/juick/server/MessagesQueries.java     | 173 +++++++++++----------
 src/main/java/com/juick/server/PMQueries.java      |  54 ++++---
 src/main/java/com/juick/server/PushQueries.java    |  20 ++-
 src/main/java/com/juick/server/SQLHelpers.java     |  25 +--
 .../com/juick/server/SubscriptionsQueries.java     |  39 +++--
 src/main/java/com/juick/server/TagQueries.java     |  37 +++--
 src/main/java/com/juick/server/UserQueries.java    |  79 +++++-----
 src/main/java/com/juick/server/Utils.java          |   6 +-
 10 files changed, 244 insertions(+), 203 deletions(-)

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/AdsQueries.java b/src/main/java/com/juick/server/AdsQueries.java
index 0fb4c0a6..ad9b7149 100644
--- a/src/main/java/com/juick/server/AdsQueries.java
+++ b/src/main/java/com/juick/server/AdsQueries.java
@@ -20,6 +20,8 @@ package com.juick.server;
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.SQLException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -27,6 +29,8 @@ import java.sql.SQLException;
  */
 public class AdsQueries {
 
+    private static final Logger logger = Logger.getLogger(AdsQueries.class.getName());
+
     public static int getAdMID(Connection sql, int uid) {
         if (uid > 0) {
             return SQLHelpers.getInt(sql, "SELECT message_id FROM ads_messages WHERE message_id NOT IN (SELECT message_id FROM ads_messages_log WHERE user_id=? AND ts>UNIX_TIMESTAMP()-60*60*24 GROUP BY message_id HAVING COUNT(*)>2) ORDER BY RAND() LIMIT 1", uid, 0);
@@ -43,7 +47,7 @@ public class AdsQueries {
             stmt.setInt(2, mid);
             stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
diff --git a/src/main/java/com/juick/server/CrosspostQueries.java b/src/main/java/com/juick/server/CrosspostQueries.java
index fd677048..b44b6b59 100644
--- a/src/main/java/com/juick/server/CrosspostQueries.java
+++ b/src/main/java/com/juick/server/CrosspostQueries.java
@@ -21,6 +21,8 @@ import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -28,6 +30,8 @@ import java.sql.SQLException;
  */
 public class CrosspostQueries {
 
+    private static final Logger logger = Logger.getLogger(CrosspostQueries.class.getName());
+
     public static String[] getTwitterTokens(Connection sql, int uid) {
         String tokens[] = null;
 
@@ -43,7 +47,7 @@ public class CrosspostQueries {
                 tokens[1] = rs.getString(2);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -70,7 +74,7 @@ public class CrosspostQueries {
                 tokens[1] = rs.getString(2);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
diff --git a/src/main/java/com/juick/server/MessagesQueries.java b/src/main/java/com/juick/server/MessagesQueries.java
index 82fef672..4d5683bd 100644
--- a/src/main/java/com/juick/server/MessagesQueries.java
+++ b/src/main/java/com/juick/server/MessagesQueries.java
@@ -25,6 +25,9 @@ import java.sql.Statement;
 import java.sql.Types;
 import java.util.ArrayList;
 import java.util.Collections;
+import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -32,11 +35,13 @@ import java.util.Collections;
  */
 public class MessagesQueries {
 
-    public static int createMessage(Connection sql, int uid, String txt, String attachment, ArrayList<com.juick.Tag> tags) {
+    private static final Logger logger = Logger.getLogger(MessagesQueries.class.getName());
+
+    public static int createMessage(Connection sql, int uid, String txt, String attachment, List<com.juick.Tag> tags) {
         int mid = 0;
 
         PreparedStatement stmt = null;
-        ResultSet rs = null;
+        ResultSet rs;
         try {
             stmt = sql.prepareStatement("INSERT INTO messages(user_id,attach) VALUES (?,?)", Statement.RETURN_GENERATED_KEYS);
             stmt.setInt(1, uid);
@@ -51,7 +56,7 @@ public class MessagesQueries {
                 mid = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -73,7 +78,7 @@ public class MessagesQueries {
                 stmt = sql.prepareStatement("INSERT INTO messages_tags(message_id,tag_id) VALUES " + tagsIDs);
                 stmt.executeUpdate();
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, stmt);
             }
@@ -81,7 +86,7 @@ public class MessagesQueries {
             try {
                 stmt = sql.prepareStatement("INSERT INTO messages_txt(message_id,tags,txt) VALUES (?,?,?)");
                 stmt.setInt(1, mid);
-                if (tagsNames == "") {
+                if (tagsNames.isEmpty()) {
                     stmt.setNull(2, Types.VARCHAR);
                 } else {
                     stmt.setString(2, tagsNames);
@@ -89,7 +94,7 @@ public class MessagesQueries {
                 stmt.setString(3, txt);
                 stmt.executeUpdate();
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, stmt);
             }
@@ -118,7 +123,7 @@ public class MessagesQueries {
                 ridnew = 0;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -166,7 +171,7 @@ public class MessagesQueries {
                 privacy = rs.getInt(2);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -189,7 +194,7 @@ public class MessagesQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -210,7 +215,7 @@ public class MessagesQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -255,7 +260,7 @@ public class MessagesQueries {
                 msg.Hidden = rs.getBoolean(13);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -276,7 +281,7 @@ public class MessagesQueries {
                 msg.Text = rs.getString(3);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -307,7 +312,7 @@ public class MessagesQueries {
                 msg.Text = rs.getString(6);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -330,15 +335,15 @@ public class MessagesQueries {
                 user.UName = rs.getString(2);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return user;
     }
 
-    public static ArrayList<com.juick.Tag> getMessageTags(Connection sql, int mid) {
-        ArrayList<com.juick.Tag> tags = new ArrayList<com.juick.Tag>();
+    public static List<com.juick.Tag> getMessageTags(Connection sql, int mid) {
+        List<com.juick.Tag> tags = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -356,7 +361,7 @@ public class MessagesQueries {
                 tags.add(t);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -364,8 +369,8 @@ public class MessagesQueries {
         return tags;
     }
 
-    public static ArrayList<Integer> getMessageTagsIDs(Connection sql, int mid) {
-        ArrayList<Integer> tids = new ArrayList<Integer>();
+    public static List<Integer> getMessageTagsIDs(Connection sql, int mid) {
+        List<Integer> tids = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -378,7 +383,7 @@ public class MessagesQueries {
                 tids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -387,7 +392,7 @@ public class MessagesQueries {
     }
 
     public static ArrayList<String> getMessageRecommendations(Connection sql, int mid) {
-        ArrayList<String> users = new ArrayList<String>();
+        ArrayList<String> users = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -400,7 +405,7 @@ public class MessagesQueries {
                 users.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -408,8 +413,8 @@ public class MessagesQueries {
         return users;
     }
 
-    public static ArrayList<Integer> getAll(Connection sql, int visitor_uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getAll(Connection sql, int visitor_uid, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -448,15 +453,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getTag(Connection sql, int tid, int visitor_uid, int before, int cnt) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getTag(Connection sql, int tid, int visitor_uid, int before, int cnt) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -483,15 +488,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getTags(Connection sql, String tids, int visitor_uid, int before, int cnt) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getTags(Connection sql, String tids, int visitor_uid, int before, int cnt) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -512,15 +517,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getPlace(Connection sql, int place_id, int visitor_uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getPlace(Connection sql, int place_id, int visitor_uid, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -541,15 +546,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getMyFeed(Connection sql, int uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(40);
+    public static List<Integer> getMyFeed(Connection sql, int uid, int before) {
+        List<Integer> mids = new ArrayList<>(40);
         PreparedStatement stmt = null;
         ResultSet rs = null;
         try {
@@ -569,7 +574,7 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -589,7 +594,7 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -603,8 +608,8 @@ public class MessagesQueries {
         return mids;
     }
 
-    public static ArrayList<Integer> getPrivate(Connection sql, int uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getPrivate(Connection sql, int uid, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -623,15 +628,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getDiscussions(Connection sql, int uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getDiscussions(Connection sql, int uid, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -650,7 +655,7 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -658,8 +663,8 @@ public class MessagesQueries {
         return mids;
     }
 
-    public static ArrayList<Integer> getRecommended(Connection sql, int uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getRecommended(Connection sql, int uid, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -678,7 +683,7 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -687,8 +692,8 @@ public class MessagesQueries {
         return mids;
     }
 
-    public static ArrayList<Integer> getPopular(Connection sql, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getPopular(Connection sql, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -705,15 +710,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getPhotos(Connection sql, int visitor_uid, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getPhotos(Connection sql, int visitor_uid, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -732,15 +737,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getSearch(Connection sql, Connection sqlSearch, String search, int before) {
-        ArrayList<Integer> mids0 = new ArrayList<Integer>(20);
+    public static List<Integer> getSearch(Connection sql, Connection sqlSearch, String search, int before) {
+        List<Integer> mids0 = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -760,12 +765,12 @@ public class MessagesQueries {
                 mids0.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
 
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+        List<Integer> mids = new ArrayList<>(20);
         if (mids0.size() > 0) {
             try {
                 stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id IN (" + Utils.convertArrayInt2String(mids0) + ") AND privacy>0 ORDER BY message_id DESC LIMIT 20");
@@ -775,7 +780,7 @@ public class MessagesQueries {
                     mids.add(rs.getInt(1));
                 }
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(rs, stmt);
             }
@@ -784,8 +789,8 @@ public class MessagesQueries {
         return mids;
     }
 
-    public static ArrayList<Integer> getUserBlog(Connection sql, int UID, int privacy, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getUserBlog(Connection sql, int UID, int privacy, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -804,15 +809,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getUserTag(Connection sql, int UID, int TID, int privacy, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getUserTag(Connection sql, int UID, int TID, int privacy, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -833,15 +838,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getUserRecommendations(Connection sql, int UID, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getUserRecommendations(Connection sql, int UID, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -860,15 +865,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getUserPhotos(Connection sql, int UID, int privacy, int before) {
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+    public static List<Integer> getUserPhotos(Connection sql, int UID, int privacy, int before) {
+        List<Integer> mids = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -887,15 +892,15 @@ public class MessagesQueries {
                 mids.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return mids;
     }
 
-    public static ArrayList<Integer> getUserSearch(Connection sql, Connection sqlSearch, int UID, String search, int privacy, int before) {
-        ArrayList<Integer> mids0 = new ArrayList<Integer>(20);
+    public static List<Integer> getUserSearch(Connection sql, Connection sqlSearch, int UID, String search, int privacy, int before) {
+        List<Integer> mids0 = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -916,12 +921,12 @@ public class MessagesQueries {
                 mids0.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
 
-        ArrayList<Integer> mids = new ArrayList<Integer>(20);
+        List<Integer> mids = new ArrayList<>(20);
         if (mids0.size() > 0) {
             try {
                 stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id IN (" + Utils.convertArrayInt2String(mids0) + ") AND privacy>=" + privacy + " ORDER BY message_id DESC");
@@ -931,7 +936,7 @@ public class MessagesQueries {
                     mids.add(rs.getInt(1));
                 }
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(rs, stmt);
             }
@@ -940,8 +945,8 @@ public class MessagesQueries {
         return mids;
     }
 
-    public static ArrayList<com.juick.Message> getMessages(Connection sql, ArrayList<Integer> mids) {
-        ArrayList<com.juick.Message> msgs = new ArrayList<com.juick.Message>(20);
+    public static List<com.juick.Message> getMessages(Connection sql, List<Integer> mids) {
+        List<com.juick.Message> msgs = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -978,7 +983,7 @@ public class MessagesQueries {
                 msgs.add(msg);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -986,8 +991,8 @@ public class MessagesQueries {
         return msgs;
     }
 
-    public static ArrayList<com.juick.Message> getReplies(Connection sql, int mid) {
-        ArrayList<com.juick.Message> replies = new ArrayList<com.juick.Message>();
+    public static List<com.juick.Message> getReplies(Connection sql, int mid) {
+        List<com.juick.Message> replies = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -1013,7 +1018,7 @@ public class MessagesQueries {
                 replies.add(msg);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -1039,7 +1044,7 @@ public class MessagesQueries {
             }
             ret = stmt.executeUpdate() > 0;
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -1050,7 +1055,7 @@ public class MessagesQueries {
                 stmt.setInt(1, mid);
                 stmt.executeUpdate();
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, stmt);
             }
@@ -1068,7 +1073,7 @@ public class MessagesQueries {
             stmt.setInt(1, mid);
             ret = stmt.executeUpdate() > 0;
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
diff --git a/src/main/java/com/juick/server/PMQueries.java b/src/main/java/com/juick/server/PMQueries.java
index 56b91abf..c4b4fa2d 100644
--- a/src/main/java/com/juick/server/PMQueries.java
+++ b/src/main/java/com/juick/server/PMQueries.java
@@ -17,11 +17,16 @@
  */
 package com.juick.server;
 
+import com.juick.User;
+
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
+import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -29,6 +34,8 @@ import java.util.ArrayList;
  */
 public class PMQueries {
 
+    private static final Logger logger = Logger.getLogger(PMQueries.class.getName());
+
     public static boolean createPM(Connection sql, int uid_from, int uid_to, String body) {
         boolean ret = false;
         PreparedStatement stmt = null;
@@ -39,7 +46,7 @@ public class PMQueries {
             stmt.setString(3, body);
             ret = stmt.executeUpdate() > 0;
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -52,7 +59,7 @@ public class PMQueries {
                 stmt2.setInt(2, uid_to);
                 stmt2.executeUpdate();
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, stmt2);
             }
@@ -70,7 +77,7 @@ public class PMQueries {
             stmt.setString(2, jid);
             ret = stmt.executeUpdate() > 0;
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -86,7 +93,7 @@ public class PMQueries {
             stmt.setString(2, jid);
             ret = stmt.executeUpdate() > 0;
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -107,7 +114,7 @@ public class PMQueries {
                 ret = true;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -128,15 +135,15 @@ public class PMQueries {
                 ret = rs.getString(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return ret;
     }
 
-    public static ArrayList<com.juick.User> getPMLastConversationsUsers(Connection sql, int uid, int cnt) {
-        ArrayList<com.juick.User> users = new ArrayList<com.juick.User>();
+    public static List<com.juick.User> getPMLastConversationsUsers(Connection sql, int uid, int cnt) {
+        List<User> users = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -152,7 +159,7 @@ public class PMQueries {
                 users.add(u);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -177,7 +184,7 @@ public class PMQueries {
                     }
                 }
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(rs, stmt);
             }
@@ -190,18 +197,17 @@ public class PMQueries {
         return users;
     }
 
-    public static boolean haveUserInArray(ArrayList<com.juick.User> arr, int uid) {
-        int s = arr.size();
-        for (int i = 0; i < s; i++) {
-            if (arr.get(i).UID == uid) {
+    public static boolean haveUserInArray(List<com.juick.User> arr, int uid) {
+        for (User user : arr) {
+            if (user.UID == uid) {
                 return true;
             }
         }
         return false;
     }
 
-    public static ArrayList<com.juick.Message> getPMMessages(Connection sql, int uid, int uid_to) {
-        ArrayList<com.juick.Message> msgs = new ArrayList<com.juick.Message>(20);
+    public static List<com.juick.Message> getPMMessages(Connection sql, int uid, int uid_to) {
+        List<com.juick.Message> msgs = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -223,7 +229,7 @@ public class PMQueries {
                 msgs.add(0, msg);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -235,7 +241,7 @@ public class PMQueries {
             stmt2.setInt(2, uid_to);
             stmt2.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt2);
         }
@@ -243,8 +249,8 @@ public class PMQueries {
         return msgs;
     }
 
-    public static ArrayList<com.juick.Message> getLastPMInbox(Connection sql, int uid) {
-        ArrayList<com.juick.Message> msgs = new ArrayList<com.juick.Message>(20);
+    public static List<com.juick.Message> getLastPMInbox(Connection sql, int uid) {
+        List<com.juick.Message> msgs = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -264,7 +270,7 @@ public class PMQueries {
                 msgs.add(0, msg);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -272,8 +278,8 @@ public class PMQueries {
         return msgs;
     }
 
-    public static ArrayList<com.juick.Message> getLastPMSent(Connection sql, int uid) {
-        ArrayList<com.juick.Message> msgs = new ArrayList<com.juick.Message>(20);
+    public static List<com.juick.Message> getLastPMSent(Connection sql, int uid) {
+        List<com.juick.Message> msgs = new ArrayList<>(20);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -293,7 +299,7 @@ public class PMQueries {
                 msgs.add(0, msg);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
diff --git a/src/main/java/com/juick/server/PushQueries.java b/src/main/java/com/juick/server/PushQueries.java
index bf7505ca..c17972b0 100644
--- a/src/main/java/com/juick/server/PushQueries.java
+++ b/src/main/java/com/juick/server/PushQueries.java
@@ -10,6 +10,8 @@ import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -17,12 +19,14 @@ import java.util.List;
  */
 public class PushQueries {
 
+    private static final Logger logger = Logger.getLogger(PushQueries.class.getName());
+
     public static String getAndroidRegID(Connection sql, int uid) {
         return SQLHelpers.getString(sql, "SELECT regid FROM android WHERE user_id=?", uid);
     }
 
-    public static ArrayList<String> getAndroidSubscribers(Connection sql, int uid) {
-        ArrayList<String> regids = new ArrayList<String>();
+    public static List<String> getAndroidSubscribers(Connection sql, int uid) {
+        List<String> regids = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -35,7 +39,7 @@ public class PushQueries {
                 regids.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -46,8 +50,8 @@ public class PushQueries {
         return SQLHelpers.getString(sql, "SELECT url FROM winphone WHERE user_id=?", uid);
     }
 
-    public static ArrayList<String> getWinPhoneSubscribers(Connection sql, int uid) {
-        ArrayList<String> urls = new ArrayList<String>();
+    public static List<String> getWinPhoneSubscribers(Connection sql, int uid) {
+        List<String> urls = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -60,7 +64,7 @@ public class PushQueries {
                 urls.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -72,7 +76,7 @@ public class PushQueries {
     }
 
     public static List<String> getAPNSSubscribers(Connection sql, int uid) {
-        List<String> urls = new ArrayList<String>();
+        List<String> urls = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -85,7 +89,7 @@ public class PushQueries {
                 urls.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
diff --git a/src/main/java/com/juick/server/SQLHelpers.java b/src/main/java/com/juick/server/SQLHelpers.java
index f5569993..13ebe98a 100644
--- a/src/main/java/com/juick/server/SQLHelpers.java
+++ b/src/main/java/com/juick/server/SQLHelpers.java
@@ -9,6 +9,9 @@ import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
+import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -16,6 +19,8 @@ import java.util.ArrayList;
  */
 public class SQLHelpers {
 
+    private static final Logger logger = Logger.getLogger(SQLHelpers.class.getName());
+
     public static int execute(Connection sql, String query) {
         int ret = -1;
         PreparedStatement stmt = null;
@@ -23,7 +28,7 @@ public class SQLHelpers {
             stmt = sql.prepareStatement(query);
             ret = stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -38,7 +43,7 @@ public class SQLHelpers {
             stmt.setInt(1, param);
             ret = stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -56,7 +61,7 @@ public class SQLHelpers {
                 ret = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -75,7 +80,7 @@ public class SQLHelpers {
                 ret = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -94,7 +99,7 @@ public class SQLHelpers {
                 ret = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -113,7 +118,7 @@ public class SQLHelpers {
                 ret = rs.getString(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -132,15 +137,15 @@ public class SQLHelpers {
                 ret = rs.getString(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return ret;
     }
 
-    public static ArrayList<Integer> getArrayInteger(Connection sql, String query, int param) {
-        ArrayList<Integer> ret = new ArrayList<Integer>();
+    public static List<Integer> getArrayInteger(Connection sql, String query, int param) {
+        List<Integer> ret = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -153,7 +158,7 @@ public class SQLHelpers {
                 ret.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index d0f5f308..637c2bcf 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -9,6 +9,9 @@ import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
+import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -16,8 +19,10 @@ import java.util.ArrayList;
  */
 public class SubscriptionsQueries {
 
-    public static ArrayList<String> getJIDSubscribedToUser(Connection sql, int uid, boolean friendsonly) {
-        ArrayList<String> jids = new ArrayList<String>();
+    private static final Logger logger = Logger.getLogger(SubscriptionsQueries.class.getName());
+
+    public static List<String> getJIDSubscribedToUser(Connection sql, int uid, boolean friendsonly) {
+        List<String> jids = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -36,27 +41,27 @@ public class SubscriptionsQueries {
                 jids.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return jids;
     }
 
-    public static ArrayList<String> getJIDSubscribedToUserAndTags(Connection sql, int uid, int mid) {
-        ArrayList<String> jids = new ArrayList<String>();
+    public static List<String> getJIDSubscribedToUserAndTags(Connection sql, int uid, int mid) {
+        List<String> jids = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
 
         String tbl = "subscr_jids_" + mid;
-        ArrayList<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
+        List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
 
         try {
             stmt = sql.prepareStatement("CREATE TEMPORARY TABLE " + tbl + "(user_id INT UNSIGNED NOT NULL) ENGINE=MEMORY");
             stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -69,7 +74,7 @@ public class SubscriptionsQueries {
             stmt = sql.prepareStatement(query);
             stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -87,7 +92,7 @@ public class SubscriptionsQueries {
                 jids.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -96,7 +101,7 @@ public class SubscriptionsQueries {
             stmt = sql.prepareStatement("DROP TABLE " + tbl);
             stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -104,8 +109,8 @@ public class SubscriptionsQueries {
         return jids;
     }
 
-    public static ArrayList<String> getJIDSubscribedToComments(Connection sql, int mid, int ignore_uid) {
-        ArrayList<String> jids = new ArrayList<String>();
+    public static List<String> getJIDSubscribedToComments(Connection sql, int mid, int ignore_uid) {
+        List<String> jids = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -119,17 +124,17 @@ public class SubscriptionsQueries {
                 jids.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return jids;
     }
 
-    public static ArrayList<String> getJIDSubscribedToUserRecommendations(Connection sql, int uid, int mid, int muid) {
-        ArrayList<String> jids = new ArrayList<String>();
+    public static List<String> getJIDSubscribedToUserRecommendations(Connection sql, int uid, int mid, int muid) {
+        List<String> jids = new ArrayList<>();
 
-        ArrayList<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
+        List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -154,7 +159,7 @@ public class SubscriptionsQueries {
                 jids.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
diff --git a/src/main/java/com/juick/server/TagQueries.java b/src/main/java/com/juick/server/TagQueries.java
index 493e8675..1c5e27d1 100644
--- a/src/main/java/com/juick/server/TagQueries.java
+++ b/src/main/java/com/juick/server/TagQueries.java
@@ -17,12 +17,17 @@
  */
 package com.juick.server;
 
+import com.juick.Tag;
+
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.ArrayList;
+import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  *
@@ -30,6 +35,8 @@ import java.util.ArrayList;
  */
 public class TagQueries {
 
+    private static final Logger logger = Logger.getLogger(TagQueries.class.getName());
+
     public static com.juick.Tag getTag(Connection sql, int tid) {
         com.juick.Tag ret = null;
 
@@ -46,7 +53,7 @@ public class TagQueries {
                 ret.Name = rs.getString(2);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -70,7 +77,7 @@ public class TagQueries {
                 ret.Name = rs.getString(3);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -84,12 +91,12 @@ public class TagQueries {
         return ret;
     }
 
-    public static ArrayList<com.juick.Tag> getTags(Connection sql, String[] tags, boolean autoCreate) {
-        ArrayList<com.juick.Tag> ret = new ArrayList<com.juick.Tag>();
+    public static List<com.juick.Tag> getTags(Connection sql, String[] tags, boolean autoCreate) {
+        List<Tag> ret = new ArrayList<>();
 
-        for (int i = 0; i < tags.length; i++) {
-            if (!tags[i].isEmpty()) {
-                com.juick.Tag t = getTag(sql, tags[i], autoCreate);
+        for (String tag : tags) {
+            if (!tag.isEmpty()) {
+                Tag t = getTag(sql, tag, autoCreate);
                 if (t != null) {
                     ret.add(t);
                 }
@@ -112,7 +119,7 @@ public class TagQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -134,7 +141,7 @@ public class TagQueries {
                 ret = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -142,8 +149,8 @@ public class TagQueries {
         return ret;
     }
 
-    public static ArrayList<com.juick.Tag> getUserTagsAll(Connection sql, int uid) {
-        ArrayList<com.juick.Tag> tags = new ArrayList<com.juick.Tag>();
+    public static List<com.juick.Tag> getUserTagsAll(Connection sql, int uid) {
+        List<com.juick.Tag> tags = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -159,15 +166,15 @@ public class TagQueries {
                 tags.add(t);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return tags;
     }
 
-    public static ArrayList<String> getUserBLTags(Connection sql, int uid) {
-        ArrayList<String> tags = new ArrayList<String>();
+    public static List<String> getUserBLTags(Connection sql, int uid) {
+        List<String> tags = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -180,7 +187,7 @@ public class TagQueries {
                 tags.add(rs.getString(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
diff --git a/src/main/java/com/juick/server/UserQueries.java b/src/main/java/com/juick/server/UserQueries.java
index 767a68d2..98573bfb 100644
--- a/src/main/java/com/juick/server/UserQueries.java
+++ b/src/main/java/com/juick/server/UserQueries.java
@@ -25,6 +25,7 @@ import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.ArrayList;
+import java.util.List;
 import java.util.Random;
 import java.util.UUID;
 import java.util.logging.Level;
@@ -52,7 +53,7 @@ public class UserQueries {
                 stmt.setString(2, hash);
                 stmt.executeUpdate();
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, stmt);
             }
@@ -76,7 +77,7 @@ public class UserQueries {
                 uid = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -103,7 +104,7 @@ public class UserQueries {
                 user.Banned = rs.getBoolean(2);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -126,7 +127,7 @@ public class UserQueries {
                 user.Banned = rs.getBoolean(3);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -149,15 +150,15 @@ public class UserQueries {
                 user.JID = jid;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return user;
     }
 
-    public static ArrayList<com.juick.User> getUsersByName(Connection sql, ArrayList<String> unames) {
-        ArrayList<com.juick.User> users = new ArrayList<com.juick.User>();
+    public static List<com.juick.User> getUsersByName(Connection sql, List<String> unames) {
+        List<com.juick.User> users = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -172,15 +173,15 @@ public class UserQueries {
                 users.add(user);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return users;
     }
 
-    public static ArrayList<com.juick.User> getUsersByID(Connection sql, ArrayList<Integer> uids) {
-        ArrayList<com.juick.User> users = new ArrayList<com.juick.User>();
+    public static List<com.juick.User> getUsersByID(Connection sql, List<Integer> uids) {
+        List<com.juick.User> users = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -195,7 +196,7 @@ public class UserQueries {
                 users.add(u);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -203,7 +204,7 @@ public class UserQueries {
         return users;
     }
 
-    public static boolean fillUsersByID(Connection sql, ArrayList<com.juick.User> users) {
+    public static boolean fillUsersByID(Connection sql, List<com.juick.User> users) {
         boolean ret = false;
 
         String uids = "";
@@ -231,7 +232,7 @@ public class UserQueries {
                 }
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -239,8 +240,8 @@ public class UserQueries {
         return ret;
     }
 
-    public static ArrayList<com.juick.User> getUsersByJID(Connection sql, ArrayList<String> jids) {
-        ArrayList<com.juick.User> users = new ArrayList<com.juick.User>();
+    public static List<com.juick.User> getUsersByJID(Connection sql, List<String> jids) {
+        List<com.juick.User> users = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -256,7 +257,7 @@ public class UserQueries {
                 users.add(user);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -296,7 +297,7 @@ public class UserQueries {
                 user.AuthHash = hash;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -315,7 +316,7 @@ public class UserQueries {
                 stmt.setString(2, hash);
                 stmt.executeUpdate();
             } catch (SQLException e) {
-                System.err.println(e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, stmt);
             }
@@ -353,7 +354,7 @@ public class UserQueries {
                 }
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -373,7 +374,7 @@ public class UserQueries {
                 ret = rs.getInt(1);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -388,7 +389,7 @@ public class UserQueries {
             stmt.setInt(2, uid);
             stmt.executeUpdate();
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(null, stmt);
         }
@@ -407,7 +408,7 @@ public class UserQueries {
                 ret = rs.getInt(1) < 3600;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -428,7 +429,7 @@ public class UserQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -449,7 +450,7 @@ public class UserQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -472,15 +473,15 @@ public class UserQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return ret;
     }
 
-    public static ArrayList<Integer> checkBL(Connection sql, int visitor, ArrayList<Integer> uids) {
-        ArrayList<Integer> ret = new ArrayList<Integer>();
+    public static List<Integer> checkBL(Connection sql, int visitor, List<Integer> uids) {
+        List<Integer> ret = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -493,7 +494,7 @@ public class UserQueries {
                 ret.add(rs.getInt(1));
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -515,19 +516,19 @@ public class UserQueries {
                 ret = rs.getInt(1) == 1;
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
         return ret;
     }
 
-    public static ArrayList<Integer> getUserRead(Connection sql, int uid) {
+    public static List<Integer> getUserRead(Connection sql, int uid) {
         return SQLHelpers.getArrayInteger(sql, "SELECT user_id FROM subscr_users WHERE suser_id=?", uid);
     }
 
-    public static ArrayList<com.juick.User> getUserReadLeastPopular(Connection sql, int uid, int cnt) {
-        ArrayList<com.juick.User> users = new ArrayList<com.juick.User>(cnt);
+    public static List<com.juick.User> getUserReadLeastPopular(Connection sql, int uid, int cnt) {
+        List<com.juick.User> users = new ArrayList<>(cnt);
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -544,7 +545,7 @@ public class UserQueries {
                 users.add(u);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -552,12 +553,12 @@ public class UserQueries {
         return users;
     }
 
-    public static ArrayList<Integer> getUserReaders(Connection sql, int uid) {
+    public static List<Integer> getUserReaders(Connection sql, int uid) {
         return SQLHelpers.getArrayInteger(sql, "SELECT suser_id FROM subscr_users WHERE user_id=?", uid);
     }
 
-    public static ArrayList<com.juick.User> getUserBLUsers(Connection sql, int uid) {
-        ArrayList<com.juick.User> users = new ArrayList<com.juick.User>();
+    public static List<com.juick.User> getUserBLUsers(Connection sql, int uid) {
+        List<com.juick.User> users = new ArrayList<>();
 
         PreparedStatement stmt = null;
         ResultSet rs = null;
@@ -573,7 +574,7 @@ public class UserQueries {
                 users.add(u);
             }
         } catch (SQLException e) {
-            System.err.println(e);
+            logger.log(Level.SEVERE, "sql exception", e);
         } finally {
             Utils.finishSQL(rs, stmt);
         }
@@ -603,7 +604,7 @@ public class UserQueries {
                 return stmt2.execute();
             }
         } catch (SQLException e) {
-            logger.throwing(UserQueries.class.getName(), "linkTwitterAccount", e);
+            logger.log(Level.SEVERE, "sql exception", e);
         }
         return false;
     }
@@ -642,7 +643,7 @@ public class UserQueries {
                 preparedStatement.setString(3, JID);
                 return preparedStatement.executeUpdate() >= 0;
             } catch (SQLException e) {
-                logger.log(Level.SEVERE, "sql error", e);
+                logger.log(Level.SEVERE, "sql exception", e);
             } finally {
                 Utils.finishSQL(null, preparedStatement);
             }
diff --git a/src/main/java/com/juick/server/Utils.java b/src/main/java/com/juick/server/Utils.java
index f97797d7..1968150e 100644
--- a/src/main/java/com/juick/server/Utils.java
+++ b/src/main/java/com/juick/server/Utils.java
@@ -21,7 +21,7 @@ import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Statement;
-import java.util.ArrayList;
+import java.util.List;
 
 /**
  *
@@ -29,7 +29,7 @@ import java.util.ArrayList;
  */
 public class Utils {
 
-    public static String convertArrayInt2String(ArrayList<Integer> mids) {
+    public static String convertArrayInt2String(List<Integer> mids) {
         String q = "";
         for (int i = 0; i < mids.size(); i++) {
             if (i > 0) {
@@ -40,7 +40,7 @@ public class Utils {
         return q;
     }
 
-    public static String convertArrayString2String(ArrayList<String> unames) {
+    public static String convertArrayString2String(List<String> unames) {
         String q = "";
         for (int i = 0; i < unames.size(); i++) {
             if (i > 0) {
-- 
cgit v1.2.3


From de32a6e9e994b830fdf0768d0825559a85d19654 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Sat, 16 Jan 2016 23:36:20 +0300
Subject: message queries almost done

---
 .../java/com/juick/server/MessagesQueries.java     | 807 ++++++---------------
 .../com/juick/server/SubscriptionsQueries.java     | 127 +---
 src/main/java/com/juick/server/UserQueries.java    |  42 +-
 .../java/com/juick/server/helpers/PrivacyOpts.java |  29 +
 4 files changed, 303 insertions(+), 702 deletions(-)
 create mode 100644 src/main/java/com/juick/server/helpers/PrivacyOpts.java

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/MessagesQueries.java b/src/main/java/com/juick/server/MessagesQueries.java
index 1d8eceb3..bcb2d08d 100644
--- a/src/main/java/com/juick/server/MessagesQueries.java
+++ b/src/main/java/com/juick/server/MessagesQueries.java
@@ -19,21 +19,18 @@ package com.juick.server;
 
 import com.juick.Message;
 import com.juick.User;
+import com.juick.server.helpers.PrivacyOpts;
 import org.springframework.dao.EmptyResultDataAccessException;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.jdbc.support.GeneratedKeyHolder;
 import org.springframework.jdbc.support.KeyHolder;
 
-import java.sql.Connection;
 import java.sql.PreparedStatement;
-import java.sql.ResultSet;
-import java.sql.SQLException;
 import java.sql.Statement;
 import java.sql.Types;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 
 /**
@@ -125,83 +122,53 @@ public class MessagesQueries {
         return sql.update("UPDATE messages SET maxreplyid=maxreplyid+1 WHERE message_id=?", mid);
     }
 
-    public static boolean recommendMessage(Connection sql, int mid, int vuid) {
-        boolean res = SQLHelpers.execute(sql, "INSERT IGNORE INTO favorites(user_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
+    public static boolean recommendMessage(JdbcTemplate sql, int mid, int vuid) {
+        boolean res = sql.update("INSERT IGNORE INTO favorites(user_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
         if (res) {
-            SQLHelpers.executeInt(sql, "UPDATE messages SET likes=likes+1 WHERE message_id=?", mid);
+            sql.update("UPDATE messages SET likes=likes+1 WHERE message_id=?", mid);
         }
         return res;
     }
 
-    public static boolean canViewThread(Connection sql, int mid, int uid) {
+    public static boolean canViewThread(JdbcTemplate sql, int mid, int uid) {
 
-        int privacy = 0;
-        int owner_uid = 0;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT user_id,privacy FROM messages WHERE messages.message_id=?");
-            stmt.setInt(1, mid);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                owner_uid = rs.getInt(1);
-                privacy = rs.getInt(2);
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
+        PrivacyOpts privacyOpts = sql.queryForObject("SELECT user_id,privacy FROM messages WHERE messages.message_id=?",
+                new Object[] {mid},
+                (rs, rowNum) -> {
+                    PrivacyOpts res = new PrivacyOpts();
+                    res.setUid(rs.getInt(1));
+                    res.setPrivacy(rs.getInt(2));
+                    return res;
+                });
 
-        return privacy >= 0
-                || uid == owner_uid
-                || ((privacy == -1 || privacy == -2) && uid > 0 && UserQueries.isInWL(sql, owner_uid, uid));
+        return privacyOpts.getPrivacy() >= 0
+                || uid == privacyOpts.getUid()
+                || ((privacyOpts.getPrivacy() == -1 || privacyOpts.getPrivacy() == -2) && uid > 0
+                && UserQueries.isInWL(sql, privacyOpts.getUid(), uid));
     }
 
-    public static boolean isReadOnly(Connection sql, int mid) {
-        boolean ret = false;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static boolean isReadOnly(JdbcTemplate sql, int mid) {
         try {
-            stmt = sql.prepareStatement("SELECT readonly FROM messages WHERE message_id=?");
-            stmt.setInt(1, mid);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                ret = rs.getInt(1) == 1;
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return sql.queryForObject("SELECT readonly FROM messages WHERE message_id=?", new Object[]{mid}, Integer.class) == 1;
+        } catch (EmptyResultDataAccessException e) {
+            return false;
         }
-        return ret;
     }
 
-    public static boolean isSubscribed(Connection sql, int uid, int mid) {
-        boolean ret = false;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static boolean isSubscribed(JdbcTemplate sql, int uid, int mid) {
         try {
-            stmt = sql.prepareStatement("SELECT 1 FROM subscr_messages WHERE suser_id=? AND message_id=?");
-            stmt.setInt(1, uid);
-            stmt.setInt(2, mid);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                ret = rs.getInt(1) == 1;
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return sql.queryForObject("SELECT 1 FROM subscr_messages WHERE suser_id=? AND message_id=?", new Object[]{uid, mid}, Integer.class) == 1;
+        } catch (EmptyResultDataAccessException e) {
+            return false;
         }
-        return ret;
     }
 
-    public static int getMessagePrivacy(Connection sql, int mid) {
-        return SQLHelpers.getInt(sql, "SELECT privacy FROM messages WHERE message_id=?", mid, -4);
+    public static int getMessagePrivacy(JdbcTemplate sql, int mid) {
+        try {
+            return sql.queryForObject("SELECT privacy FROM messages WHERE message_id=?", new Object[]{mid}, Integer.class);
+        } catch (EmptyResultDataAccessException e) {
+            return -4;
+        }
     }
 
 
@@ -252,24 +219,28 @@ public class MessagesQueries {
     }
 
     public static com.juick.Message getReply(JdbcTemplate sql, int mid, int rid) {
-        return sql.queryForObject("SELECT replies.user_id,users.nick,"
-                + "replies.replyto,replies.ts,"
-                + "replies.attach,replies.txt FROM replies INNER JOIN users "
-                + "ON replies.user_id=users.id "
-                + "WHERE replies.message_id=? AND replies.reply_id=?",
-                new Object[]{mid, rid}, (rs, num) -> {
-                    Message msg = new Message();
-                    msg.setMID(mid);
-                    msg.setRID(rid);
-                    msg.setUser(new User());
-                    msg.getUser().setUID(rs.getInt(1));
-                    msg.getUser().setUName(rs.getString(2));
-                    msg.ReplyTo = rs.getInt(3);
-                    msg.setDate(rs.getTimestamp(4));
-                    msg.AttachmentType = rs.getString(5);
-                    msg.setText(rs.getString(6));
-                    return msg;
-                });
+        try {
+            return sql.queryForObject("SELECT replies.user_id,users.nick,"
+                            + "replies.replyto,replies.ts,"
+                            + "replies.attach,replies.txt FROM replies INNER JOIN users "
+                            + "ON replies.user_id=users.id "
+                            + "WHERE replies.message_id=? AND replies.reply_id=?",
+                    new Object[]{mid, rid}, (rs, num) -> {
+                        Message msg = new Message();
+                        msg.setMID(mid);
+                        msg.setRID(rid);
+                        msg.setUser(new User());
+                        msg.getUser().setUID(rs.getInt(1));
+                        msg.getUser().setUName(rs.getString(2));
+                        msg.ReplyTo = rs.getInt(3);
+                        msg.setDate(rs.getTimestamp(4));
+                        msg.AttachmentType = rs.getString(5);
+                        msg.setText(rs.getString(6));
+                        return msg;
+                    });
+        } catch (EmptyResultDataAccessException e) {
+            return null;
+        }
     }
 
     public static User getMessageAuthor(JdbcTemplate sql, int mid) {
@@ -284,215 +255,96 @@ public class MessagesQueries {
                 });
     }
 
-    public static List<com.juick.Tag> getMessageTags(Connection sql, int mid) {
-        List<com.juick.Tag> tags = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT tags.tag_id,synonym_id,name,stat_messages FROM tags INNER JOIN messages_tags ON (messages_tags.message_id=? AND messages_tags.tag_id=tags.tag_id)");
-            stmt.setInt(1, mid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                com.juick.Tag t = new com.juick.Tag();
-                t.TID = rs.getInt(1);
-                t.SynonymID = rs.getInt(2);
-                t.Name = rs.getString(3);
-                t.UsageCnt = rs.getInt(4);
-                tags.add(t);
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return tags;
+    public static List<com.juick.Tag> getMessageTags(JdbcTemplate sql, int mid) {
+        return sql.query("SELECT tags.tag_id,synonym_id,name,stat_messages FROM tags " +
+                        "INNER JOIN messages_tags ON (messages_tags.message_id=? AND messages_tags.tag_id=tags.tag_id)",
+                new Object[]{mid}, (rs, num) -> {
+                    com.juick.Tag t = new com.juick.Tag();
+                    t.TID = rs.getInt(1);
+                    t.SynonymID = rs.getInt(2);
+                    t.Name = rs.getString(3);
+                    t.UsageCnt = rs.getInt(4);
+                    return t;
+                });
     }
 
-    public static List<Integer> getMessageTagsIDs(Connection sql, int mid) {
-        List<Integer> tids = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT tag_id FROM messages_tags WHERE message_id=?");
-            stmt.setInt(1, mid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                tids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return tids;
+    public static List<Integer> getMessageTagsIDs(JdbcTemplate sql, int mid) {
+        return sql.queryForList("SELECT tag_id FROM messages_tags WHERE message_id=?", new Object[] {mid}, Integer.class);
     }
 
-    public static ArrayList<String> getMessageRecommendations(Connection sql, int mid) {
-        ArrayList<String> users = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT users.nick FROM favorites INNER JOIN users ON (favorites.message_id=? AND favorites.user_id=users.id)");
-            stmt.setInt(1, mid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                users.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return users;
+    public static List<String> getMessageRecommendations(JdbcTemplate sql, int mid) {
+        return sql.queryForList("SELECT users.nick FROM favorites INNER JOIN users " +
+                "ON (favorites.message_id=? AND favorites.user_id=users.id)",
+                new Object[] {mid}, String.class);
     }
 
-    public static List<Integer> getAll(Connection sql, int visitor_uid, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (visitor_uid > 1) {
-                if (before > 0) {
-                    stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id<? AND hidden=0" +
-                            " AND (privacy>0 OR user_id=?) AND user_id NOT IN (SELECT bl_user_id FROM bl_users WHERE user_id=?)" +
-                            " AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20");
-                    stmt.setInt(1, before);
-                    stmt.setInt(2, visitor_uid);
-                    stmt.setInt(3, visitor_uid);
-                } else {
-                    stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE hidden=0" +
-                            " AND (privacy>0 OR user_id=?)" +
-                            " AND user_id NOT IN (SELECT bl_user_id FROM bl_users WHERE user_id=?)" +
-                            " AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20");
-                    stmt.setInt(1, visitor_uid);
-                    stmt.setInt(2, visitor_uid);
-                }
+    public static List<Integer> getAll(JdbcTemplate sql, int visitor_uid, int before) {
+        if (visitor_uid > 1) {
+            if (before > 0) {
+                return sql.queryForList("SELECT message_id FROM messages WHERE message_id<? AND hidden=0" +
+                                " AND (privacy>0 OR user_id=?) AND user_id NOT IN (SELECT bl_user_id FROM bl_users WHERE user_id=?)" +
+                                " AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20",
+                        new Object[]{before, visitor_uid, visitor_uid}, Integer.class);
             } else {
-                if (before > 0) {
-                    stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id<?" +
-                            " AND hidden=0 AND privacy>0 AND user_id NOT IN (SELECT id from users WHERE banned=1) " +
-                            " ORDER BY message_id DESC LIMIT 20");
-                    stmt.setInt(1, before);
-                } else {
-                    stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE hidden=0 AND privacy>0" +
-                            " AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20");
-                }
-
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
+                return sql.queryForList("SELECT message_id FROM messages WHERE hidden=0" +
+                                " AND (privacy>0 OR user_id=?)" +
+                                " AND user_id NOT IN (SELECT bl_user_id FROM bl_users WHERE user_id=?)" +
+                                " AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20",
+                        new Object[]{visitor_uid, visitor_uid}, Integer.class);
             }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-        return mids;
-    }
-
-    public static List<Integer> getTag(Connection sql, int tid, int visitor_uid, int before, int cnt) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
+        } else {
             if (before > 0) {
-//                stmt = sql.prepareStatement("SELECT messages.message_id FROM messages_tags INNER JOIN messages USING(message_id) WHERE messages_tags.tag_id IN (SELECT tag_id FROM tags WHERE tag_id=? OR synonym_id=?) AND messages.message_id<? AND (messages.privacy>0 OR messages.user_id=?) ORDER BY messages.message_id DESC LIMIT ?");
-                stmt = sql.prepareStatement("SELECT message_id FROM (tags INNER JOIN messages_tags ON ((tags.synonym_id=? OR tags.tag_id=?) AND tags.tag_id=messages_tags.tag_id)) INNER JOIN messages USING(message_id) WHERE messages.message_id<? AND (messages.privacy>0 OR messages.user_id=?) ORDER BY message_id DESC LIMIT ?");
-                stmt.setInt(1, tid);
-                stmt.setInt(2, tid);
-                stmt.setInt(3, before);
-                stmt.setInt(4, visitor_uid);
-                stmt.setInt(5, cnt);
+                return sql.queryForList("SELECT message_id FROM messages WHERE message_id<?" +
+                                " AND hidden=0 AND privacy>0 AND user_id NOT IN (SELECT id from users WHERE banned=1) " +
+                                " ORDER BY message_id DESC LIMIT 20",
+                        new Object[]{before}, Integer.class);
             } else {
-                //stmt = sql.prepareStatement("SELECT messages.message_id FROM messages_tags INNER JOIN messages USING(message_id) WHERE messages_tags.tag_id IN (SELECT tag_id FROM tags WHERE tag_id=? OR synonym_id=?) AND (messages.privacy>0 OR messages.user_id=?) ORDER BY messages.message_id DESC LIMIT ?");
-                stmt = sql.prepareStatement("SELECT message_id FROM (tags INNER JOIN messages_tags ON ((tags.synonym_id=? OR tags.tag_id=?) AND tags.tag_id=messages_tags.tag_id)) INNER JOIN messages USING(message_id) WHERE messages.privacy>0 OR messages.user_id=? ORDER BY message_id DESC LIMIT ?");
-                stmt.setInt(1, tid);
-                stmt.setInt(2, tid);
-                stmt.setInt(3, visitor_uid);
-                stmt.setInt(4, cnt);
+                return sql.queryForList("SELECT message_id FROM messages WHERE hidden=0 AND privacy>0" +
+                                " AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20",
+                        Integer.class);
             }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
         }
-        return mids;
     }
 
-    public static List<Integer> getTags(Connection sql, String tids, int visitor_uid, int before, int cnt) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT messages.message_id FROM messages_tags INNER JOIN messages USING(message_id) WHERE messages_tags.tag_id IN (" + tids + ") AND messages.message_id<? AND (messages.privacy>0 OR messages.user_id=?) ORDER BY messages.message_id DESC LIMIT ?");
-                stmt.setInt(1, before);
-                stmt.setInt(2, visitor_uid);
-                stmt.setInt(3, cnt);
-            } else {
-                stmt = sql.prepareStatement("SELECT messages.message_id FROM messages_tags INNER JOIN messages USING(message_id) WHERE messages_tags.tag_id IN (" + tids + ") AND (messages.privacy>0 OR messages.user_id=?) ORDER BY messages.message_id DESC LIMIT ?");
-                stmt.setInt(1, visitor_uid);
-                stmt.setInt(2, cnt);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getTag(JdbcTemplate sql, int tid, int visitor_uid, int before, int cnt) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM (tags INNER JOIN messages_tags " +
+                            "ON ((tags.synonym_id=? OR tags.tag_id=?) AND tags.tag_id=messages_tags.tag_id)) " +
+                            "INNER JOIN messages USING(message_id) WHERE messages.message_id<? " +
+                            "AND (messages.privacy>0 OR messages.user_id=?) ORDER BY message_id DESC LIMIT ?",
+                    new Object[]{tid, tid, before, visitor_uid, cnt}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM (tags INNER JOIN messages_tags " +
+                            "ON ((tags.synonym_id=? OR tags.tag_id=?) AND tags.tag_id=messages_tags.tag_id)) " +
+                            "INNER JOIN messages USING(message_id) WHERE messages.privacy>0 OR messages.user_id=? " +
+                            "ORDER BY message_id DESC LIMIT ?",
+                    new Object[]{tid, tid, visitor_uid, cnt}, Integer.class);
         }
-        return mids;
     }
 
-    public static List<Integer> getPlace(Connection sql, int place_id, int visitor_uid, int before) {
-        List<Integer> mids = new ArrayList<>(20);
+    public static List<Integer> getTags(JdbcTemplate sql, String tids, int visitor_uid, int before, int cnt) {
+        if (before > 0) {
+            return sql.queryForList("SELECT messages.message_id FROM messages_tags " +
+                    "INNER JOIN messages USING(message_id) WHERE messages_tags.tag_id IN (" + tids + ") " +
+                    "AND messages.message_id<? AND (messages.privacy>0 OR messages.user_id=?) " +
+                    "ORDER BY messages.message_id DESC LIMIT ?", new Object[]{before, visitor_uid, cnt}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT messages.message_id FROM messages_tags " +
+                    "INNER JOIN messages USING(message_id) WHERE messages_tags.tag_id IN (" + tids + ") " +
+                    "AND (messages.privacy>0 OR messages.user_id=?) " +
+                    "ORDER BY messages.message_id DESC LIMIT ?", new Object[]{visitor_uid, cnt}, Integer.class);
+        }
+    }
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE place_id=? AND message_id<? AND (privacy>0 OR user_id=?) ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, place_id);
-                stmt.setInt(2, before);
-                stmt.setInt(3, visitor_uid);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE place_id=? AND (privacy>0 OR user_id=?) ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, place_id);
-                stmt.setInt(2, visitor_uid);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getPlace(JdbcTemplate sql, int place_id, int visitor_uid, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM messages WHERE place_id=? AND message_id<? " +
+                            "AND (privacy>0 OR user_id=?) ORDER BY message_id DESC LIMIT 20",
+                    new Object[]{place_id, before, visitor_uid}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM messages WHERE place_id=? AND (privacy>0 OR user_id=?) " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{place_id, visitor_uid}, Integer.class);
         }
-        return mids;
     }
 
     public static List<Integer> getMyFeed(JdbcTemplate sql, int uid, int before) {
@@ -532,241 +384,103 @@ public class MessagesQueries {
         return mids;
     }
 
-    public static List<Integer> getPrivate(Connection sql, int uid, int before) {
-        List<Integer> mids = new ArrayList<>(20);
+    public static List<Integer> getPrivate(JdbcTemplate sql, int uid, int before) {
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE user_id=? AND privacy<0 AND message_id<? ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, uid);
-                stmt.setInt(2, before);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE user_id=? AND privacy<0 ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, uid);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM messages WHERE user_id=? AND privacy<0 AND message_id<? " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{uid, before}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM messages WHERE user_id=? AND privacy<0 " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{uid}, Integer.class);
         }
-        return mids;
     }
 
-    public static List<Integer> getDiscussions(Connection sql, int uid, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM subscr_messages WHERE suser_id=? AND message_id<? ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, uid);
-                stmt.setInt(2, before);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM subscr_messages WHERE suser_id=? ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, uid);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getDiscussions(JdbcTemplate sql, int uid, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM subscr_messages WHERE suser_id=? AND message_id<? " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{uid, before}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM subscr_messages WHERE suser_id=? " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{uid}, Integer.class);
         }
-
-        return mids;
     }
 
-    public static List<Integer> getRecommended(Connection sql, int uid, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM favorites WHERE user_id IN (SELECT user_id FROM subscr_users WHERE suser_id=?) AND message_id<? ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, uid);
-                stmt.setInt(2, before);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM favorites WHERE user_id IN (SELECT user_id FROM subscr_users WHERE suser_id=?) ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, uid);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getRecommended(JdbcTemplate sql, int uid, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM favorites WHERE user_id " +
+                    "IN (SELECT user_id FROM subscr_users WHERE suser_id=?) AND message_id<? " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{uid, before}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM favorites WHERE user_id " +
+                    "IN (SELECT user_id FROM subscr_users WHERE suser_id=?) " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{uid}, Integer.class);
         }
-
-
-        return mids;
     }
 
-    public static List<Integer> getPopular(Connection sql, int before) {
-        List<Integer> mids = new ArrayList<>(20);
+    public static List<Integer> getPopular(JdbcTemplate sql, int before) {
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
             if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id<? AND privacy>0 AND popular>0 ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, before);
+                return sql.queryForList("SELECT message_id FROM messages WHERE message_id<? AND privacy>0 " +
+                        "AND popular>0 ORDER BY message_id DESC LIMIT 20", new Object[] {before}, Integer.class);
             } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE privacy>0 AND popular>0 ORDER BY message_id DESC LIMIT 20");
+                return sql.queryForList("SELECT message_id FROM messages WHERE privacy>0 " +
+                        "AND popular>0 ORDER BY message_id DESC LIMIT 20", Integer.class);
             }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-        return mids;
     }
 
-    public static List<Integer> getPhotos(Connection sql, int visitor_uid, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id<? AND (privacy>0 OR user_id=?) AND attach IS NOT NULL AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, before);
-                stmt.setInt(2, visitor_uid);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE (privacy>0 OR user_id=?) AND attach IS NOT NULL AND user_id NOT IN (SELECT id from users WHERE banned=1) ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, visitor_uid);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getPhotos(JdbcTemplate sql, int visitor_uid, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM messages WHERE message_id<? AND (privacy>0 OR user_id=?) " +
+                    "AND attach IS NOT NULL AND user_id NOT IN (SELECT id from users WHERE banned=1) " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{before, visitor_uid}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM messages WHERE (privacy>0 OR user_id=?) " +
+                    "AND attach IS NOT NULL AND user_id NOT IN (SELECT id from users WHERE banned=1) " +
+                    "ORDER BY message_id DESC LIMIT 20", new Object[]{visitor_uid}, Integer.class);
         }
-        return mids;
     }
 
-    public static List<Integer> getSearch(Connection sql, Connection sqlSearch, String search, int before) {
-        List<Integer> mids0 = new ArrayList<>(20);
+    public static List<Integer> getSearch(JdbcTemplate sql, JdbcTemplate sqlSearch, String search, int before) {
+        List<Integer> mids;
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-
-            if (before > 0) {
-                stmt = sqlSearch.prepareStatement("SELECT id AS message_id FROM messages WHERE MATCH(?) AND id<? ORDER BY id DESC LIMIT 25");
-                stmt.setString(1, search);
-                stmt.setInt(2, before);
-            } else {
-                stmt = sqlSearch.prepareStatement("SELECT id AS message_id FROM messages WHERE MATCH(?) ORDER BY id DESC LIMIT 25");
-                stmt.setString(1, search);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids0.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+        if (before > 0) {
+            mids = sqlSearch.queryForList("SELECT id AS message_id FROM messages WHERE MATCH(?) AND id<? " +
+                    "ORDER BY id DESC LIMIT 25", new Object[]{search, before}, Integer.class);
+        } else {
+            mids = sqlSearch.queryForList("SELECT id AS message_id FROM messages WHERE MATCH(?) " +
+                    "ORDER BY id DESC LIMIT 25", new Object[]{search}, Integer.class);
         }
-
-        List<Integer> mids = new ArrayList<>(20);
-        if (mids0.size() > 0) {
-            try {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id IN (" + Utils.convertArrayInt2String(mids0) + ") AND privacy>0 ORDER BY message_id DESC LIMIT 20");
-                rs = stmt.executeQuery();
-                rs.beforeFirst();
-                while (rs.next()) {
-                    mids.add(rs.getInt(1));
-                }
-            } catch (SQLException e) {
-                LOGGER.log(Level.SEVERE, "sql exception", e);
-            } finally {
-                Utils.finishSQL(rs, stmt);
-            }
+        if (mids.size() > 0) {
+            return sql.queryForList("SELECT message_id FROM messages WHERE message_id " +
+                    "IN (" + Utils.convertArrayInt2String(mids) + ") AND privacy>0 ORDER BY message_id DESC LIMIT 20",
+                    Integer.class);
         }
-
         return mids;
     }
 
-    public static List<Integer> getUserBlog(Connection sql, int UID, int privacy, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE user_id=? AND message_id<? AND privacy>=" + privacy + " ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-                stmt.setInt(2, before);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE user_id=? AND privacy>=" + privacy + " ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getUserBlog(JdbcTemplate sql, int UID, int privacy, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM messages WHERE user_id=? AND message_id<? AND privacy>=" +
+                    privacy + " ORDER BY message_id DESC LIMIT 20", new Object[]{UID, before}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM messages WHERE user_id=? AND privacy>=" + privacy +
+                    " ORDER BY message_id DESC LIMIT 20", new Object[]{UID}, Integer.class);
         }
-        return mids;
     }
 
-    public static List<Integer> getUserTag(Connection sql, int UID, int TID, int privacy, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT messages.message_id FROM messages_tags INNER JOIN messages USING(message_id) WHERE messages.user_id=? AND messages_tags.tag_id=? AND messages.message_id<? AND messages.privacy>=" + privacy + " ORDER BY messages.message_id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-                stmt.setInt(2, TID);
-                stmt.setInt(3, before);
-            } else {
-                stmt = sql.prepareStatement("SELECT messages.message_id FROM messages_tags INNER JOIN messages USING(message_id) WHERE messages.user_id=? AND messages_tags.tag_id=? AND messages.privacy>=" + privacy + " ORDER BY messages.message_id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-                stmt.setInt(2, TID);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getUserTag(JdbcTemplate sql, int UID, int TID, int privacy, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT messages.message_id FROM messages_tags INNER JOIN messages " +
+                    "USING(message_id) WHERE messages.user_id=? AND messages_tags.tag_id=? " +
+                    "AND messages.message_id<? AND messages.privacy>=" + privacy +
+                    " ORDER BY messages.message_id DESC LIMIT 20",
+                    new Object[] {UID, TID, before}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT messages.message_id FROM messages_tags INNER JOIN messages " +
+                    "USING(message_id) WHERE messages.user_id=? AND messages_tags.tag_id=? " +
+                    "AND messages.privacy>=" + privacy + " ORDER BY messages.message_id DESC LIMIT 20",
+                    new Object[]{UID, TID}, Integer.class);
         }
-        return mids;
     }
 
     public static List<Integer> getUserRecommendations(JdbcTemplate sql, int UID, int before) {
@@ -783,76 +497,35 @@ public class MessagesQueries {
         }
     }
 
-    public static List<Integer> getUserPhotos(Connection sql, int UID, int privacy, int before) {
-        List<Integer> mids = new ArrayList<>(20);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE user_id=? AND message_id<? AND privacy>=" + privacy + " AND attach IS NOT NULL ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-                stmt.setInt(2, before);
-            } else {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE user_id=? AND privacy>=" + privacy + " AND attach IS NOT NULL ORDER BY message_id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<Integer> getUserPhotos(JdbcTemplate sql, int UID, int privacy, int before) {
+        if (before > 0) {
+            return sql.queryForList("SELECT message_id FROM messages WHERE user_id=? AND message_id<? AND privacy>=" +
+                    privacy + " AND attach IS NOT NULL ORDER BY message_id DESC LIMIT 20",
+                    new Object[] {UID, before}, Integer.class);
+        } else {
+            return sql.queryForList("SELECT message_id FROM messages WHERE user_id=? AND privacy>=" + privacy +
+                    " AND attach IS NOT NULL ORDER BY message_id DESC LIMIT 20",
+                    new Object[] {UID}, Integer.class);
         }
-        return mids;
     }
 
-    public static List<Integer> getUserSearch(Connection sql, Connection sqlSearch, int UID, String search, int privacy, int before) {
-        List<Integer> mids0 = new ArrayList<>(20);
+    public static List<Integer> getUserSearch(JdbcTemplate sql, JdbcTemplate sqlSearch, int UID, String search, int privacy, int before) {
+        List<Integer> mids;
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (before > 0) {
-                stmt = sqlSearch.prepareStatement("SELECT id AS message_id FROM messages WHERE user_id=? AND MATCH(?) AND id<? ORDER BY id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-                stmt.setString(2, search);
-                stmt.setInt(3, before);
-            } else {
-                stmt = sqlSearch.prepareStatement("SELECT id AS message_id FROM messages WHERE user_id=? AND MATCH(?) ORDER BY id DESC LIMIT 20");
-                stmt.setInt(1, UID);
-                stmt.setString(2, search);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                mids0.add(rs.getInt(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
 
-        List<Integer> mids = new ArrayList<>(20);
-        if (mids0.size() > 0) {
-            try {
-                stmt = sql.prepareStatement("SELECT message_id FROM messages WHERE message_id IN (" + Utils.convertArrayInt2String(mids0) + ") AND privacy>=" + privacy + " ORDER BY message_id DESC");
-                rs = stmt.executeQuery();
-                rs.beforeFirst();
-                while (rs.next()) {
-                    mids.add(rs.getInt(1));
-                }
-            } catch (SQLException e) {
-                LOGGER.log(Level.SEVERE, "sql exception", e);
-            } finally {
-                Utils.finishSQL(rs, stmt);
-            }
+        if (before > 0) {
+            mids = sqlSearch.queryForList("SELECT id AS message_id FROM messages WHERE user_id=? AND MATCH(?) AND id<? " +
+                    "ORDER BY id DESC LIMIT 20", new Object[] {UID, search, before}, Integer.class);
+        } else {
+            mids = sqlSearch.queryForList("SELECT id AS message_id FROM messages WHERE user_id=? AND MATCH(?) " +
+                    "ORDER BY id DESC LIMIT 20", new Object[] {UID, search}, Integer.class);
         }
 
+        if (mids.size() > 0) {
+           return sql.queryForList("SELECT message_id FROM messages WHERE message_id IN (" +
+                   Utils.convertArrayInt2String(mids) + ") AND privacy>=" + privacy + " ORDER BY message_id DESC",
+                   Integer.class);
+        }
         return mids;
     }
 
@@ -899,39 +572,25 @@ public class MessagesQueries {
         });
     }
 
-    public static List<com.juick.Message> getReplies(Connection sql, int mid) {
-        List<com.juick.Message> replies = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT replies.reply_id,replies.replyto,replies.user_id,users.nick,users.banned,replies.txt,TIMESTAMPDIFF(MINUTE,replies.ts,NOW()),replies.ts,replies.attach FROM replies INNER JOIN users ON replies.user_id=users.id WHERE replies.message_id=? ORDER BY replies.reply_id ASC");
-            stmt.setInt(1, mid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                com.juick.Message msg = new com.juick.Message();
-                msg.setMID(mid);
-                msg.setRID(rs.getInt(1));
-                msg.ReplyTo = rs.getInt(2);
-                msg.setUser(new User());
-                msg.getUser().setUID(rs.getInt(3));
-                msg.getUser().setUName(rs.getString(4));
-                msg.getUser().Banned = rs.getBoolean(5);
-                msg.setText(rs.getString(6));
-                msg.TimeAgo = rs.getInt(7);
-                msg.setDate(rs.getTimestamp(8));
-                msg.AttachmentType = rs.getString(9);
-
-                replies.add(msg);
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return replies;
+    public static List<com.juick.Message> getReplies(JdbcTemplate sql, int mid) {
+        return sql.query("SELECT replies.reply_id,replies.replyto,replies.user_id,users.nick,users.banned,replies.txt," +
+                        "TIMESTAMPDIFF(MINUTE,replies.ts,NOW()),replies.ts,replies.attach FROM replies INNER JOIN users " +
+                        "ON replies.user_id=users.id WHERE replies.message_id=? ORDER BY replies.reply_id ASC",
+                new Object[]{mid}, (rs, num) -> {
+                    com.juick.Message msg = new com.juick.Message();
+                    msg.setMID(mid);
+                    msg.setRID(rs.getInt(1));
+                    msg.ReplyTo = rs.getInt(2);
+                    msg.setUser(new User());
+                    msg.getUser().setUID(rs.getInt(3));
+                    msg.getUser().setUName(rs.getString(4));
+                    msg.getUser().Banned = rs.getBoolean(5);
+                    msg.setText(rs.getString(6));
+                    msg.TimeAgo = rs.getInt(7);
+                    msg.setDate(rs.getTimestamp(8));
+                    msg.AttachmentType = rs.getString(9);
+                    return msg;
+                });
     }
 
     public static boolean setMessagePopular(JdbcTemplate sql, int mid, int popular) {
diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index 637c2bcf..8207f499 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -4,12 +4,17 @@
  */
 package com.juick.server;
 
+import com.juick.User;
+import org.springframework.jdbc.core.JdbcTemplate;
+
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -48,99 +53,31 @@ public class SubscriptionsQueries {
         return jids;
     }
 
-    public static List<String> getJIDSubscribedToUserAndTags(Connection sql, int uid, int mid) {
-        List<String> jids = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-
-        String tbl = "subscr_jids_" + mid;
+    public static List<User> getSubscribedUsers(JdbcTemplate sql, int uid, int mid) {
+        List<Integer> userids = UserQueries.getUserRead(sql, uid);
         List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
-
-        try {
-            stmt = sql.prepareStatement("CREATE TEMPORARY TABLE " + tbl + "(user_id INT UNSIGNED NOT NULL) ENGINE=MEMORY");
-            stmt.executeUpdate();
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(null, stmt);
-        }
-
-        try {
-            String query = "INSERT INTO " + tbl + " SELECT suser_id FROM subscr_users WHERE user_id=" + uid;
-            if (!tags.isEmpty()) {
-                query += " UNION DISTINCT SELECT suser_id FROM subscr_tags WHERE tag_id IN (" + Utils.convertArrayInt2String(tags) + ") AND suser_id!=" + uid;
-            }
-            stmt = sql.prepareStatement(query);
-            stmt.executeUpdate();
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(null, stmt);
-        }
-
-
-        try {
-            String query = "SELECT jids.jid FROM " + tbl + " INNER JOIN jids ON (" + tbl + ".user_id=jids.user_id) WHERE jids.active=1 AND " + tbl + ".user_id NOT IN (SELECT user_id FROM bl_users WHERE bl_user_id=" + uid + ")";
-            if (!tags.isEmpty()) {
-                query += " AND " + tbl + ".user_id NOT IN (SELECT user_id FROM bl_tags WHERE tag_id IN (" + Utils.convertArrayInt2String(tags) + "))";
-            }
-            stmt = sql.prepareStatement(query);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                jids.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        try {
-            stmt = sql.prepareStatement("DROP TABLE " + tbl);
-            stmt.executeUpdate();
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(null, stmt);
-        }
-
-        return jids;
+        List<Integer> tagUsers = sql.queryForList("SELECT suser_id FROM subscr_tags " +
+                "WHERE tag_id IN (" + Utils.convertArrayInt2String(tags) + ") AND suser_id!=" + uid, Integer.class);
+        Set<Integer> set = new HashSet<>();
+        set.addAll(userids);
+        set.addAll(tagUsers);
+        return UserQueries.getUsersByID(sql, new ArrayList<>(set));
     }
 
-    public static List<String> getJIDSubscribedToComments(Connection sql, int mid, int ignore_uid) {
-        List<String> jids = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT jids.jid FROM subscr_messages INNER JOIN jids ON (subscr_messages.message_id=? AND subscr_messages.suser_id=jids.user_id) WHERE jids.user_id!=? AND jids.active=1");
-            stmt.setInt(1, mid);
-            stmt.setInt(2, ignore_uid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                jids.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<User> getUsersSubscribedToComments(JdbcTemplate sql, int mid, int ignore_uid) {
+        List<Integer> userids = sql.queryForList("SELECT suser_id FROM subscr_messages WHERE message_id=? AND suser_id!=?",
+                new Object[] {mid, ignore_uid}, Integer.class);
+        if (userids.size() > 0) {
+            return UserQueries.getUsersByID(sql, userids);
+        } else {
+            return new ArrayList<>();
         }
-        return jids;
     }
 
-    public static List<String> getJIDSubscribedToUserRecommendations(Connection sql, int uid, int mid, int muid) {
-        List<String> jids = new ArrayList<>();
-
+    public static List<User> getUsersSubscribedToUserRecommendations(JdbcTemplate sql, int uid, int mid, int muid) {
         List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            String query = "SELECT jid FROM jids WHERE active=1 AND user_id!=" + uid;
-            query += " AND user_id IN (SELECT suser_id FROM subscr_users WHERE user_id=" + uid + ")";
+            String query = "SELECT user_id IN (SELECT suser_id FROM subscr_users WHERE user_id=" + uid + ")";
             query += " AND user_id NOT IN (SELECT user_id FROM bl_users WHERE bl_user_id=" + muid + ")";
             query += " AND user_id NOT IN (SELECT suser_id FROM subscr_users WHERE user_id=" + muid + ")";
             query += " AND user_id NOT IN (SELECT suser_id FROM subscr_messages WHERE message_id=" + mid + ")";
@@ -151,23 +88,11 @@ public class SubscriptionsQueries {
                 query += " AND user_id NOT IN (SELECT suser_id FROM subscr_tags WHERE tag_id IN (" + tagsStr + "))";
                 query += " AND user_id NOT IN (SELECT user_id FROM bl_tags WHERE tag_id IN (" + tagsStr + "))";
             }
-
-            stmt = sql.prepareStatement(query);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                jids.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return jids;
+        List<Integer> userids = sql.queryForList(query, Integer.class);
+        return UserQueries.getUsersByID(sql, userids);
     }
 
-    public static boolean subscribeMessage(Connection sql, int mid, int vuid) {
-        return SQLHelpers.execute(sql, "INSERT IGNORE INTO subscr_messages(suser_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
+    public static boolean subscribeMessage(JdbcTemplate sql, int mid, int vuid) {
+        return sql.update("INSERT INTO subscr_messages(suser_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
     }
 }
diff --git a/src/main/java/com/juick/server/UserQueries.java b/src/main/java/com/juick/server/UserQueries.java
index 5588d8e4..21373b7d 100644
--- a/src/main/java/com/juick/server/UserQueries.java
+++ b/src/main/java/com/juick/server/UserQueries.java
@@ -18,6 +18,7 @@
 package com.juick.server;
 
 import com.juick.User;
+import org.springframework.dao.EmptyResultDataAccessException;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.jdbc.core.RowMapper;
 import org.springframework.jdbc.support.GeneratedKeyHolder;
@@ -46,9 +47,9 @@ public class UserQueries {
         @Override
         public User mapRow(ResultSet rs, int rowNum) throws SQLException {
             User user = new User();
-            user.setUID(rs.getInt("id"));
-            user.setUName(rs.getString("nick"));
-            user.Banned = rs.getBoolean("banned");
+            user.setUID(rs.getInt(1));
+            user.setUName(rs.getString(2));
+            user.Banned = rs.getBoolean(3);
             return user;
         }
     }
@@ -88,9 +89,13 @@ public class UserQueries {
     }
 
     public static User getUserByName(JdbcTemplate sql, String username) {
-        return sql.queryForObject("SELECT id,nick,banned FROM users WHERE nick=?",
-                new Object[] {username},
-                new UserMapper());
+        try {
+            return sql.queryForObject("SELECT id,nick,banned FROM users WHERE nick=?",
+                    new Object[]{username},
+                    new UserMapper());
+        } catch (EmptyResultDataAccessException e) {
+            return null;
+        }
     }
 
     public static User getUserByJID(JdbcTemplate sql, String jid) {
@@ -107,8 +112,7 @@ public class UserQueries {
 
     public static List<User> getUsersByID(JdbcTemplate sql, List<Integer> uids) {
         return sql.query("SELECT id,nick,banned FROM users WHERE id IN (" + Utils.convertArrayInt2String(uids) + ")",
-                new UserMapper(),
-                uids.toArray());
+                new UserMapper());
     }
 
     public static boolean fillUsersByID(JdbcTemplate sql, List<User> users) {
@@ -262,25 +266,9 @@ public class UserQueries {
         return ret;
     }
 
-    public static boolean isInWL(Connection sql, int uid, int check) {
-        boolean ret = false;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT 1 FROM wl_users WHERE user_id=? AND wl_user_id=?");
-            stmt.setInt(1, uid);
-            stmt.setInt(2, check);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                ret = rs.getInt(1) == 1;
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-        return ret;
+    public static boolean isInWL(JdbcTemplate sql, int uid, int check) {
+        return sql.queryForObject("SELECT 1 FROM wl_users WHERE user_id=? AND wl_user_id=?",
+                new Object[] {uid, check}, Integer.class) == 1;
     }
 
     public static boolean isInBL(Connection sql, int uid, int check) {
diff --git a/src/main/java/com/juick/server/helpers/PrivacyOpts.java b/src/main/java/com/juick/server/helpers/PrivacyOpts.java
new file mode 100644
index 00000000..66cf9410
--- /dev/null
+++ b/src/main/java/com/juick/server/helpers/PrivacyOpts.java
@@ -0,0 +1,29 @@
+package com.juick.server.helpers;
+
+/**
+ * Created by vt on 16/01/16.
+ */
+public class PrivacyOpts {
+    private int uid;
+    private int privacy;
+
+    public PrivacyOpts() {
+
+    }
+
+    public int getUid() {
+        return uid;
+    }
+
+    public void setUid(int uid) {
+        this.uid = uid;
+    }
+
+    public int getPrivacy() {
+        return privacy;
+    }
+
+    public void setPrivacy(int privacy) {
+        this.privacy = privacy;
+    }
+}
-- 
cgit v1.2.3


From 8ae5d9e3ce643bf0b8488c4566eff133b2cb2f11 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Sun, 17 Jan 2016 01:36:57 +0300
Subject: fix replyidincrement

---
 src/main/java/com/juick/server/MessagesQueries.java      | 3 ++-
 src/main/java/com/juick/server/SubscriptionsQueries.java | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/MessagesQueries.java b/src/main/java/com/juick/server/MessagesQueries.java
index bcb2d08d..422cfe7d 100644
--- a/src/main/java/com/juick/server/MessagesQueries.java
+++ b/src/main/java/com/juick/server/MessagesQueries.java
@@ -119,7 +119,8 @@ public class MessagesQueries {
     }
 
     public static int getReplyIDIncrement(JdbcTemplate sql, int mid) {
-        return sql.update("UPDATE messages SET maxreplyid=maxreplyid+1 WHERE message_id=?", mid);
+        sql.update("UPDATE messages SET maxreplyid=maxreplyid+1 WHERE message_id=?", mid);
+        return sql.queryForObject("SELECT maxreplyid FROM messages WHERE message_id=?", new Object[] {mid}, Integer.class);
     }
 
     public static boolean recommendMessage(JdbcTemplate sql, int mid, int vuid) {
diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index 8207f499..3a76e23e 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -93,6 +93,6 @@ public class SubscriptionsQueries {
     }
 
     public static boolean subscribeMessage(JdbcTemplate sql, int mid, int vuid) {
-        return sql.update("INSERT INTO subscr_messages(suser_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
+        return sql.update("INSERT IGNORE INTO subscr_messages(suser_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
     }
 }
-- 
cgit v1.2.3


From eca30236892175d7071e6d0d39f474148d655ffa Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Fri, 24 Jun 2016 16:22:20 +0300
Subject: fix tags queries

---
 src/main/java/com/juick/server/SubscriptionsQueries.java | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index 3a76e23e..b271435b 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -6,6 +6,7 @@ package com.juick.server;
 
 import com.juick.User;
 import org.springframework.jdbc.core.JdbcTemplate;
+import org.springframework.util.StringUtils;
 
 import java.sql.Connection;
 import java.sql.PreparedStatement;
@@ -55,12 +56,14 @@ public class SubscriptionsQueries {
 
     public static List<User> getSubscribedUsers(JdbcTemplate sql, int uid, int mid) {
         List<Integer> userids = UserQueries.getUserRead(sql, uid);
-        List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
-        List<Integer> tagUsers = sql.queryForList("SELECT suser_id FROM subscr_tags " +
-                "WHERE tag_id IN (" + Utils.convertArrayInt2String(tags) + ") AND suser_id!=" + uid, Integer.class);
         Set<Integer> set = new HashSet<>();
         set.addAll(userids);
-        set.addAll(tagUsers);
+        List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
+        if (tags.size() > 0) {
+            List<Integer> tagUsers = sql.queryForList("SELECT suser_id FROM subscr_tags " +
+                    "WHERE tag_id IN (" + StringUtils.arrayToCommaDelimitedString(tags.toArray()) + ") AND suser_id!=" + uid, Integer.class);
+            set.addAll(tagUsers);
+        }
         return UserQueries.getUsersByID(sql, new ArrayList<>(set));
     }
 
-- 
cgit v1.2.3


From 6362eab34390b187944c7e3c12eb444e2fb2c455 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Mon, 27 Jun 2016 13:15:57 +0300
Subject: more spring-jdbc

---
 src/main/java/com/juick/server/PushQueries.java    | 108 ++++------
 .../com/juick/server/SubscriptionsQueries.java     |  38 +---
 src/main/java/com/juick/server/TagQueries.java     |  23 +--
 src/main/java/com/juick/server/UserQueries.java    | 220 +++++++--------------
 4 files changed, 128 insertions(+), 261 deletions(-)

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/PushQueries.java b/src/main/java/com/juick/server/PushQueries.java
index c17972b0..54528993 100644
--- a/src/main/java/com/juick/server/PushQueries.java
+++ b/src/main/java/com/juick/server/PushQueries.java
@@ -4,14 +4,12 @@
  */
 package com.juick.server;
 
-import java.sql.Connection;
-import java.sql.PreparedStatement;
-import java.sql.ResultSet;
-import java.sql.SQLException;
-import java.util.ArrayList;
+import org.springframework.dao.EmptyResultDataAccessException;
+import org.springframework.jdbc.core.JdbcTemplate;
+import org.springframework.util.StringUtils;
+
 import java.util.List;
-import java.util.logging.Level;
-import java.util.logging.Logger;
+import java.util.Optional;
 
 /**
  *
@@ -19,80 +17,58 @@ import java.util.logging.Logger;
  */
 public class PushQueries {
 
-    private static final Logger logger = Logger.getLogger(PushQueries.class.getName());
-
-    public static String getAndroidRegID(Connection sql, int uid) {
-        return SQLHelpers.getString(sql, "SELECT regid FROM android WHERE user_id=?", uid);
-    }
-
-    public static List<String> getAndroidSubscribers(Connection sql, int uid) {
-        List<String> regids = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static Optional<String> getAndroidRegID(JdbcTemplate sql, int uid) {
         try {
-            stmt = sql.prepareStatement("SELECT regid FROM android INNER JOIN subscr_users ON (subscr_users.user_id=? AND android.user_id=subscr_users.suser_id)");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                regids.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return Optional.of(sql.queryForObject("SELECT regid FROM android WHERE user_id=?", String.class, uid));
+        } catch (EmptyResultDataAccessException e) {
+            return Optional.empty();
         }
-        return regids;
+
     }
 
-    public static String getWinPhoneURL(Connection sql, int uid) {
-        return SQLHelpers.getString(sql, "SELECT url FROM winphone WHERE user_id=?", uid);
+    public static  List<String> getAndroidSubscribers(JdbcTemplate sql, int uid) {
+        return sql.queryForList("SELECT regid FROM android INNER JOIN subscr_users " +
+                "ON (subscr_users.user_id=? AND android.user_id=subscr_users.suser_id)", String.class, uid);
     }
 
-    public static List<String> getWinPhoneSubscribers(Connection sql, int uid) {
-        List<String> urls = new ArrayList<>();
+    public static  List<String> getAndroidTokens(JdbcTemplate sql, List<Integer> uids) {
+        return sql.queryForList("SELECT regid FROM android INNER JOIN users " +
+                "ON (users.id=android.user_id) WHERE users.id IN (" + StringUtils.collectionToCommaDelimitedString(uids) + ")", String.class);
+    }
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static  Optional<String> getWinPhoneURL(JdbcTemplate sql, int uid) {
         try {
-            stmt = sql.prepareStatement("SELECT url FROM winphone INNER JOIN subscr_users ON (subscr_users.user_id=? AND winphone.user_id=subscr_users.suser_id)");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                urls.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return Optional.of(sql.queryForObject("SELECT url FROM winphone WHERE user_id=?", String.class, uid));
+        } catch (EmptyResultDataAccessException e) {
+            return Optional.empty();
         }
-        return urls;
     }
 
-    public static String getAPNSToken(Connection sql, int uid) {
-        return SQLHelpers.getString(sql, "SELECT token from ios WHERE user_id=?", uid);
+    public static  List<String> getWinPhoneSubscribers(JdbcTemplate sql, int uid) {
+        return sql.queryForList("SELECT url FROM winphone INNER JOIN subscr_users " +
+                "ON (subscr_users.user_id=? AND winphone.user_id=subscr_users.suser_id)", String.class, uid);
     }
 
-    public static List<String> getAPNSSubscribers(Connection sql, int uid) {
-        List<String> urls = new ArrayList<>();
+    public static  List<String> getWindowsTokens(JdbcTemplate sql, List<Integer> uids) {
+        return sql.queryForList("SELECT url FROM winphone INNER JOIN users " +
+                "ON (users.id=winphone.user_id) WHERE users.id IN (" + StringUtils.collectionToCommaDelimitedString(uids) + ")", String.class);
+    }
 
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static  Optional<String> getAPNSToken(JdbcTemplate sql, int uid) {
         try {
-            stmt = sql.prepareStatement("SELECT token FROM ios INNER JOIN subscr_users ON (subscr_users.user_id=? AND ios.user_id=subscr_users.suser_id)");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                urls.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return Optional.of(sql.queryForObject("SELECT token from ios WHERE user_id=?", String.class, uid));
+        } catch (EmptyResultDataAccessException e) {
+            return Optional.empty();
         }
-        return urls;
     }
+
+    public static  List<String> getAPNSSubscribers(JdbcTemplate sql, int uid) {
+        return sql.queryForList("SELECT token FROM ios INNER JOIN subscr_users " +
+                "ON (subscr_users.user_id=? AND ios.user_id=subscr_users.suser_id)", String.class, uid);
+    }
+    public static  List<String> getAPNSTokens(JdbcTemplate sql, List<Integer> uids) {
+        return sql.queryForList("SELECT token FROM ios INNER JOIN users " +
+                "ON (users.id=ios.user_id) WHERE users.id IN (" + StringUtils.collectionToCommaDelimitedString(uids) + ")", String.class);
+    }
+
 }
diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index b271435b..feebfb22 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -8,15 +8,10 @@ import com.juick.User;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.util.StringUtils;
 
-import java.sql.Connection;
-import java.sql.PreparedStatement;
-import java.sql.ResultSet;
-import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 
 /**
@@ -27,31 +22,16 @@ public class SubscriptionsQueries {
 
     private static final Logger logger = Logger.getLogger(SubscriptionsQueries.class.getName());
 
-    public static List<String> getJIDSubscribedToUser(Connection sql, int uid, boolean friendsonly) {
-        List<String> jids = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            if (friendsonly == false) {
-                stmt = sql.prepareStatement("SELECT jids.jid FROM subscr_users INNER JOIN jids ON (subscr_users.user_id=? AND subscr_users.suser_id=jids.user_id) WHERE jids.active=1");
-                stmt.setInt(1, uid);
-            } else {
-                stmt = sql.prepareStatement("SELECT jids.jid FROM subscr_users INNER JOIN jids ON (subscr_users.user_id=? AND subscr_users.suser_id=jids.user_id) WHERE jids.active=1 AND jids.user_id IN (SELECT wl_user_id FROM wl_users WHERE user_id=?)");
-                stmt.setInt(1, uid);
-                stmt.setInt(2, uid);
-            }
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                jids.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            logger.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+    public static List<String> getJIDSubscribedToUser(JdbcTemplate sql, int uid, boolean friendsonly) {
+        if (friendsonly == false) {
+            return sql.queryForList("SELECT jids.jid FROM subscr_users INNER JOIN jids " +
+                            "ON (subscr_users.user_id=? AND subscr_users.suser_id=jids.user_id) WHERE jids.active=1",
+                    String.class, uid);
+        } else {
+            return sql.queryForList("SELECT jids.jid FROM subscr_users INNER JOIN jids " +
+                    "ON (subscr_users.user_id=? AND subscr_users.suser_id=jids.user_id) WHERE jids.active=1 " +
+                    "AND jids.user_id IN (SELECT wl_user_id FROM wl_users WHERE user_id=?)", String.class, uid, uid);
         }
-        return jids;
     }
 
     public static List<User> getSubscribedUsers(JdbcTemplate sql, int uid, int mid) {
diff --git a/src/main/java/com/juick/server/TagQueries.java b/src/main/java/com/juick/server/TagQueries.java
index d2159b5a..0094fc22 100644
--- a/src/main/java/com/juick/server/TagQueries.java
+++ b/src/main/java/com/juick/server/TagQueries.java
@@ -123,25 +123,10 @@ public class TagQueries {
                 }, uid);
     }
 
-    public static List<String> getUserBLTags(Connection sql, int uid) {
-        List<String> tags = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT tags.name FROM tags INNER JOIN bl_tags ON (bl_tags.user_id=? AND bl_tags.tag_id=tags.tag_id) ORDER BY tags.name");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                tags.add(rs.getString(1));
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-        return tags;
+    public static List<String> getUserBLTags(JdbcTemplate sql, int uid) {
+        return sql.queryForList("SELECT tags.name FROM tags INNER JOIN bl_tags " +
+                        "ON (bl_tags.user_id=? AND bl_tags.tag_id=tags.tag_id) ORDER BY tags.name",
+                String.class, uid);
     }
 
     public static List<String> getPopularTags(JdbcTemplate sql) {
diff --git a/src/main/java/com/juick/server/UserQueries.java b/src/main/java/com/juick/server/UserQueries.java
index 0d6a3485..e7aee0f5 100644
--- a/src/main/java/com/juick/server/UserQueries.java
+++ b/src/main/java/com/juick/server/UserQueries.java
@@ -25,14 +25,11 @@ import org.springframework.jdbc.support.GeneratedKeyHolder;
 import org.springframework.jdbc.support.KeyHolder;
 import org.springframework.util.StringUtils;
 
-import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.*;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 /**
  *
@@ -42,8 +39,6 @@ public class UserQueries {
 
     static final String ABCDEF = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ";
 
-    private static final Logger LOGGER = Logger.getLogger(UserQueries.class.getName());
-
     public static class UserMapper implements RowMapper<User> {
         @Override
         public User mapRow(ResultSet rs, int rowNum) throws SQLException {
@@ -163,8 +158,8 @@ public class UserQueries {
                 });
     }
 
-    public static String getJIDbyUID(JdbcTemplate sql, int uid) {
-        return sql.queryForObject("SELECT jid FROM jids WHERE user_id=? AND active=1", String.class, uid);
+    public static List<String> getJIDsbyUID(JdbcTemplate sql, int uid) {
+        return sql.queryForList("SELECT jid FROM jids WHERE user_id=? AND active=1", String.class, uid);
     }
 
     public static int getUIDbyJID(JdbcTemplate sql, String jid) {
@@ -241,58 +236,26 @@ public class UserQueries {
         }
     }
 
-    public static int getUserOptionInt(Connection sql, int uid, String option, int defaultValue) {
-        int ret = defaultValue;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static int getUserOptionInt(JdbcTemplate sql, int uid, String option, int defaultValue) {
         try {
-            stmt = sql.prepareStatement("SELECT " + option + " FROM useroptions WHERE user_id=?");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                ret = rs.getInt(1);
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return sql.queryForObject("SELECT " + option + " FROM useroptions WHERE user_id=?", Integer.class, uid);
+        } catch (EmptyResultDataAccessException e) {
+            return defaultValue;
         }
-        return ret;
     }
 
-    public static void setUserOptionInt(Connection sql, int uid, String option, int value) {
-        PreparedStatement stmt = null;
-        try {
-            stmt = sql.prepareStatement("UPDATE useroptions SET " + option + "=? WHERE user_id=?");
-            stmt.setInt(1, value);
-            stmt.setInt(2, uid);
-            stmt.executeUpdate();
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(null, stmt);
-        }
+    public static void setUserOptionInt(JdbcTemplate sql, int uid, String option, int value) {
+        sql.update("UPDATE useroptions SET " + option + "=? WHERE user_id=?", value, uid);
     }
 
-    public static boolean getCanMedia(Connection sql, int uid) {
-        boolean ret = false;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static boolean getCanMedia(JdbcTemplate sql, int uid) {
         try {
-            stmt = sql.prepareStatement("SELECT users.lastphoto-UNIX_TIMESTAMP() FROM users WHERE id=?");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                ret = rs.getInt(1) < 3600;
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            int res = sql.queryForObject("SELECT users.lastphoto-UNIX_TIMESTAMP() FROM users WHERE id=?",
+                    Integer.class, uid);
+            return res < 3600;
+        } catch (EmptyResultDataAccessException e) {
+            return false;
         }
-        return ret;
     }
 
     public static boolean isInWL(JdbcTemplate sql, int uid, int check) {
@@ -300,25 +263,13 @@ public class UserQueries {
                 new Object[] {uid, check}, Integer.class) == 1;
     }
 
-    public static boolean isInBL(Connection sql, int uid, int check) {
-        boolean ret = false;
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
+    public static boolean isInBL(JdbcTemplate sql, int uid, int check) {
         try {
-            stmt = sql.prepareStatement("SELECT 1 FROM bl_users WHERE user_id=? AND bl_user_id=?");
-            stmt.setInt(1, uid);
-            stmt.setInt(2, check);
-            rs = stmt.executeQuery();
-            if (rs.first()) {
-                ret = rs.getInt(1) == 1;
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
+            return sql.queryForObject("SELECT 1 FROM bl_users WHERE user_id=? AND bl_user_id=?",
+                    Integer.class, uid, check) == 1;
+        } catch (EmptyResultDataAccessException e) {
+            return false;
         }
-        return ret;
     }
 
     public static boolean isInBLAny(JdbcTemplate sql, int uid, int uid2) {
@@ -353,33 +304,20 @@ public class UserQueries {
         return sql.queryForList("SELECT user_id FROM subscr_users WHERE suser_id=?", Integer.class, uid);
     }
 
-    public static List<com.juick.User> getUserReadLeastPopular(Connection sql, int uid, int cnt) {
-        List<com.juick.User> users = new ArrayList<>(cnt);
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT users.id,users.nick FROM (subscr_users INNER JOIN users_subscr ON (subscr_users.suser_id=? AND subscr_users.user_id=users_subscr.user_id)) INNER JOIN users ON subscr_users.user_id=users.id ORDER BY cnt LIMIT ?");
-            stmt.setInt(1, uid);
-            stmt.setInt(2, cnt);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                com.juick.User u = new com.juick.User();
-                u.setUID(rs.getInt(1));
-                u.setUName(rs.getString(2));
-                users.add(u);
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return users;
+    public static List<com.juick.User> getUserReadLeastPopular(JdbcTemplate sql, int uid, int cnt) {
+        return sql.query("SELECT users.id,users.nick FROM (subscr_users " +
+                        "INNER JOIN users_subscr ON (subscr_users.suser_id=? " +
+                        "AND subscr_users.user_id=users_subscr.user_id)) INNER JOIN users " +
+                        "ON subscr_users.user_id=users.id ORDER BY cnt LIMIT ?",
+                (rs, num) -> {
+                    com.juick.User u = new com.juick.User();
+                    u.setUID(rs.getInt(1));
+                    u.setUName(rs.getString(2));
+                    return u;
+                }, uid, cnt);
     }
 
-    public List<User> getUserReaders(JdbcTemplate sql, int uid) {
+    public static List<User> getUserReaders(JdbcTemplate sql, int uid) {
         return sql.query("SELECT users.id, users.nick FROM subscr_users " +
                         "INNER JOIN users ON subscr_users.suser_id=users.id " +
                         "WHERE subscr_users.user_id=? ORDER BY users.nick",
@@ -391,7 +329,7 @@ public class UserQueries {
                 }, uid);
     }
 
-    public List<User> getUserFriends(JdbcTemplate sql, int uid) {
+    public static List<User> getUserFriends(JdbcTemplate sql, int uid) {
         return sql.query("SELECT users.id,users.nick FROM subscr_users " +
                         "INNER JOIN users ON subscr_users.user_id=users.id " +
                         "WHERE subscr_users.suser_id=? AND users.id!=? " +
@@ -404,72 +342,60 @@ public class UserQueries {
                 }, uid, uid);
     }
 
-    public static List<com.juick.User> getUserBLUsers(Connection sql, int uid) {
-        List<com.juick.User> users = new ArrayList<>();
-
-        PreparedStatement stmt = null;
-        ResultSet rs = null;
-        try {
-            stmt = sql.prepareStatement("SELECT users.id,users.nick FROM users INNER JOIN bl_users ON(bl_users.bl_user_id=users.id) WHERE bl_users.user_id=? ORDER BY users.nick");
-            stmt.setInt(1, uid);
-            rs = stmt.executeQuery();
-            rs.beforeFirst();
-            while (rs.next()) {
-                com.juick.User u = new com.juick.User();
-                u.setUID(rs.getInt(1));
-                u.setUName(rs.getString(2));
-                users.add(u);
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
-        } finally {
-            Utils.finishSQL(rs, stmt);
-        }
-
-        return users;
+    public static List<com.juick.User> getUserBLUsers(JdbcTemplate sql, int uid) {
+        return sql.query("SELECT users.id,users.nick FROM users INNER JOIN bl_users " +
+                        "ON(bl_users.bl_user_id=users.id) WHERE bl_users.user_id=? ORDER BY users.nick",
+                (rs, num) -> {
+                    com.juick.User u = new com.juick.User();
+                    u.setUID(rs.getInt(1));
+                    u.setUName(rs.getString(2));
+                    return u;
+                }, uid);
     }
 
-    public static boolean linkTwitterAccount(Connection sql, User user, String accessToken,
+    public static boolean linkTwitterAccount(JdbcTemplate sql, User user, String accessToken,
                                              String accessTokenSecret, String screenName) {
-        try {
-            PreparedStatement stmt =
-                    sql.prepareStatement("INSERT INTO twitter(user_id,access_token,access_token_secret,uname) " +
-                            "VALUES (?,?,?,?)" +
-                            " ON DUPLICATE KEY UPDATE access_token=?,access_token_secret=?,uname=?");
-            stmt.setInt(1, user.getUID());
-            stmt.setString(2, accessToken);
-            stmt.setString(3, accessTokenSecret);
-            stmt.setString(4, screenName);
-            stmt.setString(5, accessToken);
-            stmt.setString(6, accessTokenSecret);
-            stmt.setString(7, screenName);
-            if (stmt.execute()) {
-                PreparedStatement stmt2 =
-                        sql.prepareStatement("INSERT INTO subscr_users(user_id,suser_id,jid) " +
-                                "VALUES (?,1741,'juick\\@twitter.juick.com')");
-                stmt2.setInt(1, user.getUID());
-                return stmt2.execute();
-            }
-        } catch (SQLException e) {
-            LOGGER.log(Level.SEVERE, "sql exception", e);
+        if (sql.update("INSERT INTO twitter(user_id,access_token,access_token_secret,uname) " +
+                        "VALUES (?,?,?,?)" +
+                        " ON DUPLICATE KEY UPDATE access_token=?,access_token_secret=?,uname=?",
+                user.getUID(), accessToken, accessTokenSecret, screenName, accessToken, accessTokenSecret, screenName) > 0) {
+            return sql.update("INSERT INTO subscr_users(user_id,suser_id,jid) " +
+                    "VALUES (?,1741,'juick\\@twitter.juick.com')", user.getUID()) > 0;
         }
         return false;
+
     }
 
-    public static int getStatsIRead(Connection sql, int uid) {
-        return SQLHelpers.getInt(sql, "SELECT COUNT(*) FROM subscr_users WHERE suser_id=?", uid, 0);
+    public static int getStatsIRead(JdbcTemplate sql, int uid) {
+        try {
+            return sql.queryForObject("SELECT COUNT(*) FROM subscr_users WHERE suser_id=?", Integer.class, uid);
+        } catch (EmptyResultDataAccessException e) {
+            return 0;
+        }
     }
 
-    public static int getStatsMyReaders(Connection sql, int uid) {
-        return SQLHelpers.getInt(sql, "SELECT COUNT(*) FROM subscr_users WHERE user_id=?", uid, 0);
+    public static int getStatsMyReaders(JdbcTemplate sql, int uid) {
+        try {
+            return sql.queryForObject("SELECT COUNT(*) FROM subscr_users WHERE user_id=?", Integer.class, uid);
+        } catch (EmptyResultDataAccessException e) {
+            return 0;
+        }
     }
 
-    public static int getStatsMessages(Connection sql, int uid) {
-        return SQLHelpers.getInt(sql, "SELECT COUNT(*) FROM messages WHERE user_id=?", uid, 0);
+    public static int getStatsMessages(JdbcTemplate sql, int uid) {
+        try {
+            return sql.queryForObject("SELECT COUNT(*) FROM messages WHERE user_id=?", Integer.class, uid);
+        } catch (EmptyResultDataAccessException e) {
+            return 0;
+        }
     }
 
-    public static int getStatsReplies(Connection sql, int uid) {
-        return SQLHelpers.getInt(sql, "SELECT COUNT(*) FROM replies WHERE user_id=?", uid, 0);
+    public static int getStatsReplies(JdbcTemplate sql, int uid) {
+        try {
+            return sql.queryForObject("SELECT COUNT(*) FROM replies WHERE user_id=?", Integer.class, uid);
+        } catch (EmptyResultDataAccessException e) {
+            return 0;
+        }
     }
 
     public enum ActiveStatus {
-- 
cgit v1.2.3


From 797942249b0d5e38285e806bc0e73e755e6ffede Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Wed, 29 Jun 2016 15:09:47 +0300
Subject: add more bot queries

---
 .../java/com/juick/server/MessagesQueries.java     | 11 ++++
 .../com/juick/server/SubscriptionsQueries.java     | 22 ++++++++
 src/main/java/com/juick/server/TagQueries.java     | 65 ++++++++++++++++------
 3 files changed, 81 insertions(+), 17 deletions(-)

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/MessagesQueries.java b/src/main/java/com/juick/server/MessagesQueries.java
index cd098ec6..880e94e2 100644
--- a/src/main/java/com/juick/server/MessagesQueries.java
+++ b/src/main/java/com/juick/server/MessagesQueries.java
@@ -645,4 +645,15 @@ public class MessagesQueries {
         return sql.update("UPDATE messages SET privacy=1 WHERE message_id=?", 
                 mid) > 0;            
     }
+    public static boolean deleteMessage(JdbcTemplate sql, int uid, int mid) {
+        if (sql.update("DELETE FROM messages WHERE message_id=? AND user_id=?", mid, uid) > 0) {
+            sql.update("DELETE FROM messages_txt WHERE message_id=?", mid);
+            sql.update("DELETE FROM replies WHERE message_id=?", mid);
+            sql.update("DELETE FROM subscr_messages WHERE message_id=?", mid);
+            sql.update("DELETE FROM messages_tags WHERE message_id=?", mid);
+            return true;
+        }
+        return false;
+    }
+
 }
diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index feebfb22..9f0372ff 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -4,6 +4,7 @@
  */
 package com.juick.server;
 
+import com.juick.Tag;
 import com.juick.User;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.util.StringUtils;
@@ -78,4 +79,25 @@ public class SubscriptionsQueries {
     public static boolean subscribeMessage(JdbcTemplate sql, int mid, int vuid) {
         return sql.update("INSERT IGNORE INTO subscr_messages(suser_id,message_id) VALUES (" + vuid + "," + mid + ")") == 1;
     }
+    public static boolean unSubscribeMessage(JdbcTemplate sql, int mid, int vuid) {
+        return sql.update("DELETE FROM subscr_messages WHERE message_id=? AND suser_id=?",
+                mid, vuid) > 0;
+    }
+    public static boolean subscribeUser(JdbcTemplate sql, User user, User toUser) {
+        return sql.update("INSERT IGNORE INTO subscr_users(user_id,suser_id) VALUES (?,?)",
+                user.getUID(), toUser.getUID()) == 1;
+    }
+    public static boolean unSubscribeUser(JdbcTemplate sql, User user, User fromUser) {
+        return sql.update("DELETE FROM subscr_users WHERE suser_id=? AND user_id=?",
+                user.getUID(), fromUser.getUID()) > 0;
+    }
+    public static boolean subscribeTag(JdbcTemplate sql, User user, Tag toTag) {
+        return sql.update("INSERT IGNORE INTO subscr_tags(tag_id,suser_id) VALUES (?,?)",
+                toTag.TID, user.getUID()) == 1;
+    }
+    public static boolean unSubscribeTag(JdbcTemplate sql, User user, Tag toTag) {
+        return sql.update("DELETE FROM subscr_tags WHERE tag_id=? AND suser_id=?",
+                toTag.TID, user.getUID()) > 0;
+    }
+
 }
diff --git a/src/main/java/com/juick/server/TagQueries.java b/src/main/java/com/juick/server/TagQueries.java
index 0094fc22..2258a8ea 100644
--- a/src/main/java/com/juick/server/TagQueries.java
+++ b/src/main/java/com/juick/server/TagQueries.java
@@ -20,18 +20,17 @@ package com.juick.server;
 import com.juick.Tag;
 import org.springframework.dao.EmptyResultDataAccessException;
 import org.springframework.jdbc.core.JdbcTemplate;
+import org.springframework.jdbc.support.GeneratedKeyHolder;
+import org.springframework.jdbc.support.KeyHolder;
 
-import java.sql.Connection;
 import java.sql.PreparedStatement;
-import java.sql.ResultSet;
-import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
-import java.util.logging.Level;
-import java.util.logging.Logger;
-import org.springframework.jdbc.support.GeneratedKeyHolder;
-import org.springframework.jdbc.support.KeyHolder;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
+import java.util.stream.Collectors;
 
 /**
  *
@@ -39,17 +38,19 @@ import org.springframework.jdbc.support.KeyHolder;
  */
 public class TagQueries {
 
-    private static final Logger LOGGER = Logger.getLogger(TagQueries.class.getName());
-
     public static com.juick.Tag getTag(JdbcTemplate sql, int tid) {
-        return sql.queryForObject("SELECT synonym_id,name FROM tags WHERE tag_id=?",
-                new Object[]{tid}, (rs, num) -> {
-                    Tag ret = new Tag();
-                    ret.TID = tid;
-                    ret.SynonymID = rs.getInt(1);
-                    ret.Name = rs.getString(2);
-                    return ret;
-                });
+        try {
+            return sql.queryForObject("SELECT synonym_id,name FROM tags WHERE tag_id=?",
+                    new Object[]{tid}, (rs, num) -> {
+                        Tag ret = new Tag();
+                        ret.TID = tid;
+                        ret.SynonymID = rs.getInt(1);
+                        ret.Name = rs.getString(2);
+                        return ret;
+                    });
+        } catch (EmptyResultDataAccessException e) {
+            return null;
+        }
     }
 
     public static com.juick.Tag getTag(JdbcTemplate sql, String tag, boolean autoCreate) {
@@ -132,4 +133,34 @@ public class TagQueries {
     public static List<String> getPopularTags(JdbcTemplate sql) {
         return sql.queryForList("SELECT name FROM tags WHERE top=1 ORDER BY name ASC", String.class);
     }
+    public static List<Tag> updateTags(JdbcTemplate sql, int mid, List<Tag> newTags) {
+        List<Tag> currentTags = MessagesQueries.getMessageTags(sql, mid);
+        newTags.stream().filter(currentTags::contains)
+                .forEach(t -> sql.update("DELETE FROM messages_tags WHERE message_id=? AND tag_id=?", mid, t.TID));
+        newTags.stream().filter(t -> !currentTags.contains(t))
+                .forEach(t -> sql.update("INSERT INTO messages_tags(message_id,tag_id) VALUES (?,?)", mid, t.TID));
+        return MessagesQueries.getMessageTags(sql, mid);
+    }
+
+    public static List<Tag> fromString(JdbcTemplate sql, String txt, boolean tagsOnly) {
+        String patternString = tagsOnly ? "^(?:(?:\\*[^ \\r\\n\\t]+)|\\s)+$" : "^\\*([^ \\r\\n\\t]+)\\s+([\\s\\S]+)";
+        Pattern tagsPattern = Pattern.compile(patternString);
+        if (tagsPattern.matcher(txt).matches()) {
+            Pattern tagPattern = Pattern.compile("\\*([^ \\r\\n\\t]+)");
+            Matcher tagMatcher = tagPattern.matcher(txt);
+            List<Tag> tags = new ArrayList<>();
+            // TODO: process readonly, private, friends, public
+            while (tagMatcher.find()) {
+                for (int i = 1; i <= tagMatcher.groupCount(); i++) {
+                    tags.add(getTag(sql, tagMatcher.group(i), true));
+                }
+            }
+            return tags;
+        }
+        return Collections.emptyList();
+    }
+    public static String toString(List<Tag> tags) {
+        return tags.stream().map(t -> " *" + t.Name)
+                .collect(Collectors.joining());
+    }
 }
-- 
cgit v1.2.3


From 67d53fe24081ade138b5d9756fd835575eb4ae79 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Fri, 1 Jul 2016 14:00:31 +0300
Subject: fix getSubscribedUsers

---
 src/main/java/com/juick/server/SubscriptionsQueries.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'src/main/java/com/juick/server/SubscriptionsQueries.java')

diff --git a/src/main/java/com/juick/server/SubscriptionsQueries.java b/src/main/java/com/juick/server/SubscriptionsQueries.java
index 9f0372ff..0cf3f169 100644
--- a/src/main/java/com/juick/server/SubscriptionsQueries.java
+++ b/src/main/java/com/juick/server/SubscriptionsQueries.java
@@ -14,6 +14,7 @@ import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.logging.Logger;
+import java.util.stream.Collectors;
 
 /**
  *
@@ -36,9 +37,9 @@ public class SubscriptionsQueries {
     }
 
     public static List<User> getSubscribedUsers(JdbcTemplate sql, int uid, int mid) {
-        List<Integer> userids = UserQueries.getUserRead(sql, uid);
+        List<User> userids = UserQueries.getUserReaders(sql, uid);
         Set<Integer> set = new HashSet<>();
-        set.addAll(userids);
+        set.addAll(userids.stream().map(User::getUID).collect(Collectors.toList()));
         List<Integer> tags = MessagesQueries.getMessageTagsIDs(sql, mid);
         if (tags.size() > 0) {
             List<Integer> tagUsers = sql.queryForList("SELECT suser_id FROM subscr_tags " +
-- 
cgit v1.2.3