From: Benny Baumann Date: Sat, 28 Feb 2015 21:35:12 +0000 (+0100) Subject: Empty: Whitespace, serialVersionUID, final classes X-Git-Url: https://code.wpia.club/?p=gigi.git;a=commitdiff_plain;h=87c6ea794b343732e32514cba17d047c6fc196a4 Empty: Whitespace, serialVersionUID, final classes --- diff --git a/src/org/cacert/gigi/DevelLauncher.java b/src/org/cacert/gigi/DevelLauncher.java index 431b922d..e66b9157 100644 --- a/src/org/cacert/gigi/DevelLauncher.java +++ b/src/org/cacert/gigi/DevelLauncher.java @@ -92,7 +92,9 @@ public class DevelLauncher { public boolean needsLogin() { return false; } + }); + pages.put("/kill", new Page("Kill") { @Override @@ -105,6 +107,7 @@ public class DevelLauncher { return false; } }); + pageF.set(gigi, Collections.unmodifiableMap(pages)); } catch (ReflectiveOperationException e) { e.printStackTrace(); diff --git a/src/org/cacert/gigi/GigiApiException.java b/src/org/cacert/gigi/GigiApiException.java index b858e7d9..944fe645 100644 --- a/src/org/cacert/gigi/GigiApiException.java +++ b/src/org/cacert/gigi/GigiApiException.java @@ -13,6 +13,8 @@ import org.cacert.gigi.output.template.TranslateCommand; public class GigiApiException extends Exception { + private static final long serialVersionUID = -164928670180852588L; + private SQLException e; private LinkedList messages = new LinkedList<>(); diff --git a/src/org/cacert/gigi/api/GigiAPI.java b/src/org/cacert/gigi/api/GigiAPI.java index ae8aee35..744432f4 100644 --- a/src/org/cacert/gigi/api/GigiAPI.java +++ b/src/org/cacert/gigi/api/GigiAPI.java @@ -22,6 +22,8 @@ import org.cacert.gigi.util.PEM; public class GigiAPI extends HttpServlet { + private static final long serialVersionUID = 659963677032635817L; + @Override protected void service(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { String pi = req.getPathInfo(); diff --git a/src/org/cacert/gigi/dbObjects/CertificateOwner.java b/src/org/cacert/gigi/dbObjects/CertificateOwner.java index 82143db5..97194398 100644 --- a/src/org/cacert/gigi/dbObjects/CertificateOwner.java +++ b/src/org/cacert/gigi/dbObjects/CertificateOwner.java @@ -6,6 +6,8 @@ import org.cacert.gigi.database.GigiResultSet; public abstract class CertificateOwner implements IdCachable { + private static ObjectCache myCache = new ObjectCache<>(); + private int id; public CertificateOwner(int id) { @@ -18,8 +20,6 @@ public abstract class CertificateOwner implements IdCachable { return id; } - private static ObjectCache myCache = new ObjectCache<>(); - public static synchronized CertificateOwner getById(int id) { CertificateOwner u = myCache.get(id); if (u == null) { @@ -122,6 +122,7 @@ public abstract class CertificateOwner implements IdCachable { return true; } } + return false; } @@ -131,6 +132,7 @@ public abstract class CertificateOwner implements IdCachable { return true; } } + return false; } @@ -140,4 +142,5 @@ public abstract class CertificateOwner implements IdCachable { ps.execute(); myCache.remove(this); } + } diff --git a/src/org/cacert/gigi/dbObjects/IdCachable.java b/src/org/cacert/gigi/dbObjects/IdCachable.java index cc083d25..af439bcf 100644 --- a/src/org/cacert/gigi/dbObjects/IdCachable.java +++ b/src/org/cacert/gigi/dbObjects/IdCachable.java @@ -3,4 +3,5 @@ package org.cacert.gigi.dbObjects; public interface IdCachable { public int getId(); + } diff --git a/src/org/cacert/gigi/dbObjects/User.java b/src/org/cacert/gigi/dbObjects/User.java index 9b2cf515..a370d61a 100644 --- a/src/org/cacert/gigi/dbObjects/User.java +++ b/src/org/cacert/gigi/dbObjects/User.java @@ -42,12 +42,14 @@ public class User extends CertificateOwner { name = new Name(rs.getString("fname"), rs.getString("lname"), rs.getString("mname"), rs.getString("suffix")); dob = rs.getDate("dob"); email = rs.getString("email"); + String localeStr = rs.getString("language"); if (localeStr == null || localeStr.equals("")) { locale = Locale.getDefault(); } else { locale = Language.getLocaleFromString(localeStr); } + GigiPreparedStatement psg = DatabaseConnection.getInstance().prepare("SELECT permission FROM user_groups WHERE user=? AND deleted is NULL"); psg.setInt(1, rs.getInt("id")); GigiResultSet rs2 = psg.executeQuery(); @@ -224,6 +226,7 @@ public class User extends CertificateOwner { if (exp >= 50) { points += 5; } + return points; } @@ -421,6 +424,7 @@ public class User extends CertificateOwner { if (co instanceof User) { return (User) co; } + return null; } diff --git a/src/org/cacert/gigi/output/template/Form.java b/src/org/cacert/gigi/output/template/Form.java index f330fe86..ed1a8668 100644 --- a/src/org/cacert/gigi/output/template/Form.java +++ b/src/org/cacert/gigi/output/template/Form.java @@ -109,5 +109,7 @@ public abstract class Form implements Outputable { public static class CSRFException extends IOException { + private static final long serialVersionUID = 59708247477988362L; + } } diff --git a/tests/org/cacert/gigi/testUtils/ManagedTest.java b/tests/org/cacert/gigi/testUtils/ManagedTest.java index 4ac5d910..6564a5fd 100644 --- a/tests/org/cacert/gigi/testUtils/ManagedTest.java +++ b/tests/org/cacert/gigi/testUtils/ManagedTest.java @@ -286,12 +286,14 @@ public class ManagedTest extends ConfiguredTest { TestMail tm = ter.recieve(); Assert.assertNotNull(tm); tm.verify(); + GigiPreparedStatement ps = DatabaseConnection.getInstance().prepare("SELECT id FROM users where email=?"); ps.setString(1, email); GigiResultSet rs = ps.executeQuery(); if (rs.next()) { return rs.getInt(1); } + throw new Error(); } catch (InterruptedException e) { throw new Error(e); diff --git a/tests/org/cacert/gigi/testUtils/TestEmailReciever.java b/tests/org/cacert/gigi/testUtils/TestEmailReciever.java index b47133b7..dc90acde 100644 --- a/tests/org/cacert/gigi/testUtils/TestEmailReciever.java +++ b/tests/org/cacert/gigi/testUtils/TestEmailReciever.java @@ -14,7 +14,7 @@ import java.util.regex.Pattern; import org.cacert.gigi.email.EmailProvider; -public class TestEmailReciever extends EmailProvider implements Runnable { +public final class TestEmailReciever extends EmailProvider implements Runnable { public class TestMail {