]> WPIA git - gigi.git/commitdiff
upd: enforce pattern of making templates static and final.
authorFelix Dörre <felix@dogcraft.de>
Sun, 17 Jul 2016 14:09:38 +0000 (16:09 +0200)
committerFelix Dörre <felix@dogcraft.de>
Sun, 17 Jul 2016 23:08:54 +0000 (01:08 +0200)
Change-Id: I235adf31a8fd4919cbe3a73ed8a43f01b65c91c6

27 files changed:
src/org/cacert/gigi/Gigi.java
src/org/cacert/gigi/output/AssurancesDisplay.java
src/org/cacert/gigi/output/ClientCSRGenerate.java
src/org/cacert/gigi/pages/MainPage.java
src/org/cacert/gigi/pages/PasswordResetPage.java
src/org/cacert/gigi/pages/account/ChangeForm.java
src/org/cacert/gigi/pages/account/FindAgentAccess.java
src/org/cacert/gigi/pages/account/MyDetailsForm.java
src/org/cacert/gigi/pages/account/MyOrganisationsForm.java
src/org/cacert/gigi/pages/account/certs/Certificates.java
src/org/cacert/gigi/pages/account/domain/DomainPinglogForm.java
src/org/cacert/gigi/pages/account/domain/PingConfigForm.java
src/org/cacert/gigi/pages/account/mail/MailAddForm.java
src/org/cacert/gigi/pages/account/mail/MailManagementForm.java
src/org/cacert/gigi/pages/admin/TTPAdminForm.java
src/org/cacert/gigi/pages/admin/support/FindDomainForm.java
src/org/cacert/gigi/pages/admin/support/FindUserForm.java
src/org/cacert/gigi/pages/admin/support/SupportEnterTicketForm.java
src/org/cacert/gigi/pages/admin/support/SupportRevokeCertificatesForm.java
src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.java
src/org/cacert/gigi/pages/main/Signup.java
src/org/cacert/gigi/pages/orga/OrgDomainAddForm.java
src/org/cacert/gigi/pages/orga/ViewOrgPage.java
src/org/cacert/gigi/pages/wot/AssuranceForm.java
src/org/cacert/gigi/pages/wot/AssurePage.java
util-testing/org/cacert/gigi/DevelLauncher.java
util-testing/org/cacert/gigi/pages/Manager.java

index ab0e1b6fa7e2acefa4f15feea664d18648b183cc..e55f97a226e1d7692784b69102865368c0a109b2 100644 (file)
@@ -187,7 +187,6 @@ public final class Gigi extends HttpServlet {
             } catch (UnsupportedEncodingException e) {
                 throw new ServletException(e);
             }
-            baseTemplate = new Template(Gigi.class.getResource("Gigi.templ"));
             rootMenu = new MenuCollector();
 
             Menu languages = new Menu("Language");
@@ -223,7 +222,7 @@ public final class Gigi extends HttpServlet {
 
     private static Gigi instance;
 
-    private Template baseTemplate;
+    private static final Template baseTemplate = new Template(Gigi.class.getResource("Gigi.templ"));;
 
     private PingerDaemon pinger;
 
index 639197cd279024ec417da073a5c3362cc07aad86..9cd7558c1e54d5bc6dd65d7b2159a685c8a27238 100644 (file)
@@ -11,16 +11,12 @@ import org.cacert.gigi.output.template.Template;
 
 public class AssurancesDisplay implements Outputable {
 
-    private static Template template;
+    private static final Template template = new Template(AssurancesDisplay.class.getResource("AssurancesDisplay.templ"));
 
     private boolean assurer;
 
     public String assuranceArray;
 
-    static {
-        template = new Template(AssurancesDisplay.class.getResource("AssurancesDisplay.templ"));
-    }
-
     public AssurancesDisplay(String assuranceArray, boolean assurer) {
         this.assuranceArray = assuranceArray;
         this.assurer = assurer;
index 440d2b0c3087dbb3d06f6f6d9b376e5297ab2b45..49be42590b1adf35d8257bc54d81776f56264a16 100644 (file)
@@ -12,11 +12,7 @@ import org.cacert.gigi.util.ServerConstants;
 
 public class ClientCSRGenerate {
 
-    private static Template normal;
-
-    static {
-        normal = new Template(ClientCSRGenerate.class.getResource("ClientCSRGenerate.templ"));
-    }
+    private static final Template normal = new Template(ClientCSRGenerate.class.getResource("ClientCSRGenerate.templ"));
 
     public static void output(HttpServletRequest req, HttpServletResponse resp) {
         HashMap<String, Object> vars = new HashMap<String, Object>();
index a24188a5ad6d381edac7da6a20bd1f48cb0ef964..0cccbfb535ec223c452f0b132f0ad55a6d8d95ed 100644 (file)
@@ -10,7 +10,7 @@ import org.cacert.gigi.output.template.Template;
 
 public class MainPage extends Page {
 
-    Template notLog = new Template(MainPage.class.getResource("MainPageNotLogin.templ"));
+    private static final Template notLog = new Template(MainPage.class.getResource("MainPageNotLogin.templ"));
 
     public MainPage() {
         super("Home");
index 7ba5d73c15d9599d4d3b390af2e96bf7d317519c..1be5643c30d08e1ee16cd5e5e62f6df388c6f103 100644 (file)
@@ -34,7 +34,7 @@ public class PasswordResetPage extends Page {
 
     public static class PasswordResetForm extends Form {
 
-        private static Template t = new Template(PasswordResetForm.class.getResource("PasswordResetForm.templ"));
+        private static final Template t = new Template(PasswordResetForm.class.getResource("PasswordResetForm.templ"));
 
         private User u;
 
index 8e73d3f27f83c16a70f67c1d36471b217a6c8dc3..667dc751fbc84a9e28637e410ba728e4e922a4b8 100644 (file)
@@ -21,10 +21,7 @@ public class ChangeForm extends Form {
         this.target = target;
     }
 
-    private static Template t;
-    static {
-        t = new Template(ChangePasswordPage.class.getResource("ChangePasswordForm.templ"));
-    }
+    private static final Template t = new Template(ChangePasswordPage.class.getResource("ChangePasswordForm.templ"));
 
     @Override
     public void outputContent(PrintWriter out, Language l, Map<String, Object> vars) {
index 2f325ebbdca39c41b74d99ab88c3ba6604ca7b5b..98ee3ae35d0a6f156da8bc8577ac340c585e88c5 100644 (file)
@@ -24,10 +24,7 @@ public class FindAgentAccess extends Form {
 
     public static final String PATH = "/account/find-agent";
 
-    private static Template t;
-    static {
-        t = new Template(ChangePasswordPage.class.getResource("FindAgentAccess.templ"));
-    }
+    private static final Template t = new Template(ChangePasswordPage.class.getResource("FindAgentAccess.templ"));
 
     @Override
     public boolean submit(PrintWriter out, HttpServletRequest req) throws GigiApiException {
index 9ea66b359965f8ba0142b1ca4cbcced21f252335..d35ba45e84bea8ec27872807196abf4de2be6632 100644 (file)
@@ -17,12 +17,9 @@ import org.cacert.gigi.util.HTMLEncoder;
 
 public class MyDetailsForm extends Form {
 
-    private static Template assured = new Template(MyDetails.class.getResource("MyDetailsFormAssured.templ"));
+    private static final Template assured = new Template(MyDetails.class.getResource("MyDetailsFormAssured.templ"));
 
-    private static Template templ;
-    static {
-        templ = new Template(MyDetailsForm.class.getResource("MyDetailsForm.templ"));
-    }
+    private static final Template templ = new Template(MyDetailsForm.class.getResource("MyDetailsForm.templ"));
 
     private User target;
 
index 443a9efeca153dbd1b0c223a18ea7d488be584b0..706e95976d0b4222e4541cbef277d23228891af2 100644 (file)
@@ -26,11 +26,7 @@ public class MyOrganisationsForm extends Form {
         target = LoginPage.getAuthorizationContext(hsr);
     }
 
-    private static Template template;
-
-    static {
-        template = new Template(MyOrganisationsForm.class.getResource("MyOrganisationsForm.templ"));
-    }
+    private static final Template template = new Template(MyOrganisationsForm.class.getResource("MyOrganisationsForm.templ"));
 
     @Override
     public boolean submit(PrintWriter out, HttpServletRequest req) {
index d40bbaccb9981853eaefdf76a00c1e4caf8d551f..9d05db4fcb3bba6011f17d629af7d25d8750d7a1 100644 (file)
@@ -25,7 +25,7 @@ import org.cacert.gigi.util.PEM;
 
 public class Certificates extends Page implements HandlesMixedRequest {
 
-    private Template certDisplay = new Template(Certificates.class.getResource("CertificateDisplay.templ"));
+    private static final Template certDisplay = new Template(Certificates.class.getResource("CertificateDisplay.templ"));
 
     public static final String PATH = "/account/certs";
 
index 954c57369c483035dac7451d314964e481f33cb4..525cd125c0ac5eef7c26d6a9f50b854fc20ac695 100644 (file)
@@ -18,7 +18,7 @@ import org.cacert.gigi.pages.LoginPage;
 
 public class DomainPinglogForm extends Form {
 
-    static Template t = new Template(DomainPinglogForm.class.getResource("DomainPinglogForm.templ"));
+    private static final Template t = new Template(DomainPinglogForm.class.getResource("DomainPinglogForm.templ"));
 
     Domain target;
 
index 1810b7ae0543b0249e161116840058f40f449f9c..5a13482f6221e87123745cc44534f48b6cdfed21 100644 (file)
@@ -50,7 +50,7 @@ public class PingConfigForm extends Form {
 
     private SSLType[] sslTypes = new SSLType[MAX_SSL_TESTS];
 
-    private final Template t = new Template(PingConfigForm.class.getResource("PingConfigForm.templ"));
+    private static final Template t = new Template(PingConfigForm.class.getResource("PingConfigForm.templ"));
 
     public PingConfigForm(HttpServletRequest hsr, Domain target) throws GigiApiException {
         super(hsr);
index eaf35f4004975f6e840279f4ab53334209dbbac1..1a67f8e23214cf137a906d571bc0dbe6ebec61d7 100644 (file)
@@ -16,12 +16,9 @@ import org.cacert.gigi.pages.Page;
 
 public class MailAddForm extends Form {
 
-    private static Template t;
+    private static final Template t = new Template(MailAddForm.class.getResource("MailAddForm.templ"));;
 
     private String mail;
-    static {
-        t = new Template(MailAddForm.class.getResource("MailAddForm.templ"));
-    }
 
     private User target;
 
index cbf143976ac6a7bdecf14ad8de9116fb30252ea3..9a399884a94a502a70e0a0ec533bd83df55e58b1 100644 (file)
@@ -17,12 +17,9 @@ import org.cacert.gigi.pages.Page;
 
 public class MailManagementForm extends Form {
 
-    private static Template t;
+    private static final Template t = new Template(MailAddForm.class.getResource("MailManagementForm.templ"));
 
     private User target;
-    static {
-        t = new Template(MailAddForm.class.getResource("MailManagementForm.templ"));
-    }
 
     public MailManagementForm(HttpServletRequest hsr, User target) {
         super(hsr);
index a52afed348bbce0e5612d7deff8fe18ee047be84..2c8b5719b3d30d570fcab43979785f3d1411306e 100644 (file)
@@ -14,7 +14,7 @@ import org.cacert.gigi.pages.LoginPage;
 
 public class TTPAdminForm extends Form {
 
-    private static Template t = new Template(TTPAdminForm.class.getResource("TTPAdminForm.templ"));
+    private static final Template t = new Template(TTPAdminForm.class.getResource("TTPAdminForm.templ"));
 
     User u;
 
index 1096f3d7ab204915643ebb1d80bc532dc57e1af7..93c4b0c158bfffe0eb00f14b2afaac09bac1f0a8 100644 (file)
@@ -17,10 +17,7 @@ public class FindDomainForm extends Form {
 
     private CertificateOwner res = null;
 
-    private static Template t;
-    static {
-        t = new Template(FindDomainForm.class.getResource("FindDomainForm.templ"));
-    }
+    private static final Template t = new Template(FindDomainForm.class.getResource("FindDomainForm.templ"));
 
     public FindDomainForm(HttpServletRequest hsr) {
         super(hsr);
index 21276dba2e8bc1743001a5f70002e4b7acfbd46f..3a7bf07c1e979191005d378c6aa07d111d54359d 100644 (file)
@@ -16,10 +16,7 @@ public class FindUserForm extends Form {
 
     private User users[];
 
-    private static Template t;
-    static {
-        t = new Template(FindDomainForm.class.getResource("FindUserForm.templ"));
-    }
+    private static final Template t = new Template(FindDomainForm.class.getResource("FindUserForm.templ"));
 
     public FindUserForm(HttpServletRequest hsr) {
         super(hsr);
index 5b57a1c7330aeb76710593dab98c15eb96ff0e66..18afc0c662279ccf76ca0fd503e57693220c4272 100644 (file)
@@ -15,11 +15,7 @@ import org.cacert.gigi.util.AuthorizationContext;
 
 public class SupportEnterTicketForm extends Form {
 
-    private static Template t;
-
-    static {
-        t = new Template(SupportEnterTicketForm.class.getResource("SupportEnterTicketForm.templ"));
-    }
+    private static final Template t = new Template(SupportEnterTicketForm.class.getResource("SupportEnterTicketForm.templ"));
 
     public SupportEnterTicketForm(HttpServletRequest hsr) {
         super(hsr);
index 32f52254247f16711615c6ccd1d5ba5c2536ebaf..9c1f3f5be3ae726cd98cab710ac1be4dcd41695c 100644 (file)
@@ -20,12 +20,9 @@ import org.cacert.gigi.output.template.Template;
 
 public class SupportRevokeCertificatesForm extends Form {
 
-    private static Template t;
+    private static final Template t = new Template(SupportRevokeCertificatesForm.class.getResource("SupportRevokeCertificatesForm.templ"));
 
     private SupportedUser user;
-    static {
-        t = new Template(SupportRevokeCertificatesForm.class.getResource("SupportRevokeCertificatesForm.templ"));
-    }
 
     public SupportRevokeCertificatesForm(HttpServletRequest hsr, SupportedUser user) {
         super(hsr);
index 60251e7e4f789161336796b129a6059aff33cddf..15ed2818ba6b76cc2b113718e725af4d532f5525 100644 (file)
@@ -22,7 +22,7 @@ import org.cacert.gigi.pages.PasswordResetPage;
 
 public class SupportUserDetailsForm extends Form {
 
-    private static Template t;
+    private static final Template t = new Template(FindDomainForm.class.getResource("SupportUserDetailsForm.templ"));
 
     private SupportedUser user;
 
@@ -30,10 +30,6 @@ public class SupportUserDetailsForm extends Form {
 
     private GroupSelector value = new GroupSelector("groupToModify");
 
-    static {
-        t = new Template(FindDomainForm.class.getResource("SupportUserDetailsForm.templ"));
-    }
-
     public SupportUserDetailsForm(HttpServletRequest hsr, SupportedUser user) {
         super(hsr);
         this.user = user;
index 4df3ba4903026ea32013dda55adcf83545fac9ec..0fc04d8e9f9144e3202b956ef009bc74a3cab31b 100644 (file)
@@ -32,13 +32,13 @@ public class Signup extends Form {
 
     String email = "";
 
-    private Template t;
+    private static final Template t = new Template(Signup.class.getResource("Signup.templ"));
 
     boolean general = true, country = true, regional = true, radius = true;
 
     public Signup(HttpServletRequest hsr) {
         super(hsr);
-        t = new Template(Signup.class.getResource("Signup.templ"));
+
     }
 
     DateSelector myDoB = new DateSelector("day", "month", "year");
index b3df26eb4867eeda19a937d47e6909452fcd26c8..c18cf8f11494cd7399accb31019596923c348dc7 100644 (file)
@@ -15,7 +15,7 @@ import org.cacert.gigi.pages.LoginPage;
 
 public class OrgDomainAddForm extends Form {
 
-    public static Template t = new Template(OrgDomainAddForm.class.getResource("OrgDomainAddForm.templ"));
+    private static final Template t = new Template(OrgDomainAddForm.class.getResource("OrgDomainAddForm.templ"));
 
     Organisation target;
 
index 49833e572b55a00baf600d3fa8cd130128905f47..8dcc06d25e173c550d87ef2945dca561bfe712d6 100644 (file)
@@ -23,9 +23,9 @@ import org.cacert.gigi.util.AuthorizationContext;
 
 public class ViewOrgPage extends Page {
 
-    private final Template orgas = new Template(ViewOrgPage.class.getResource("ViewOrgs.templ"));
+    private static final Template orgas = new Template(ViewOrgPage.class.getResource("ViewOrgs.templ"));
 
-    private final Template mainTempl = new Template(ViewOrgPage.class.getResource("EditOrg.templ"));
+    private static final Template mainTempl = new Template(ViewOrgPage.class.getResource("EditOrg.templ"));
 
     public static final String DEFAULT_PATH = "/orga";
 
index 9188013de885f34653737b491fa89130bef9db87..07a0fdc22d11746e72741867e1afbbab1739cd4e 100644 (file)
@@ -40,10 +40,7 @@ public class AssuranceForm extends Form {
 
     private AssuranceType type = AssuranceType.FACE_TO_FACE;
 
-    private static final Template templ;
-    static {
-        templ = new Template(AssuranceForm.class.getResource("AssuranceForm.templ"));
-    }
+    private static final Template templ = new Template(AssuranceForm.class.getResource("AssuranceForm.templ"));
 
     public AssuranceForm(HttpServletRequest hsr, User assuree) {
         super(hsr);
index dfd72b7b6966a85536a8c2f5cf7b3dd97fe1c61e..4603e8436263e12634f697dbeed91acda975d834 100644 (file)
@@ -25,7 +25,7 @@ public class AssurePage extends Page {
 
     DateSelector ds = new DateSelector("day", "month", "year");
 
-    Template t;
+    private final Template t;
 
     public AssurePage() {
         super("Assure someone");
index f84d728ba487acb282c9c6185c565780ea95e6b4..6639635b0cbb551bef7ba9caf1e1cc06262956b0 100644 (file)
@@ -181,7 +181,7 @@ public class DevelLauncher {
     private static void addTicketPage(HashMap<String, Page> pages) {
         pages.put("/ticketWait", new Page("ticket") {
 
-            Template t = new Template(DevelLauncher.class.getResource("DevelTicketWait.templ"));
+            private final Template t = new Template(DevelLauncher.class.getResource("DevelTicketWait.templ"));
 
             @Override
             public boolean needsLogin() {
index dfa4b5a4ee85a7902264e98c0d7170862fa641a9..6a4a521a7b730289ea1b4a8b338c63316b354914 100644 (file)
@@ -125,7 +125,7 @@ public class Manager extends Page {
 
     private static Manager instance;
 
-    Template t = new Template(Manager.class.getResource("ManagerMails.templ"));
+    private static final Template t = new Template(Manager.class.getResource("ManagerMails.templ"));
 
     HashMap<String, LinkedList<String>> emails = new HashMap<>();
 
@@ -367,7 +367,7 @@ public class Manager extends Page {
         }
     }
 
-    private Template form = new Template(Manager.class.getResource("Manager.templ"));
+    private static final Template form = new Template(Manager.class.getResource("Manager.templ"));
 
     @Override
     public boolean needsLogin() {