upd: add exception handling

related to change #797

Change-Id: I7ab53fa7b71f019d9ba14b321998ebf011834ad2

diff --git a/util-testing/club/wpia/gigi/pages/Manager.java b/util-testing/club/wpia/gigi/pages/Manager.java
index 1c06d3c0736b321d79eeabd4563176eebb94fbaf..2f866b2d22db8a3dc2ae3cbeb53211852467f095 100644 (file)
--- a/util-testing/club/wpia/gigi/pages/Manager.java
+++ b/util-testing/club/wpia/gigi/pages/Manager.java
@@ -372,7 +372,13 @@ public class Manager extends Page {
                 resp.getWriter().println("No test given.");
                 return;
             }
                 resp.getWriter().println("No test given.");
                 return;
             }

-            CATSType test = CATSType.values()[Integer.parseInt(catsTypeId)];
+            CATSType test = null;
+            try {
+                test = CATSType.values()[Integer.parseInt(catsTypeId)];
+            } catch (NumberFormatException e) {
+                resp.getWriter().println("No valid integer given.");
+                return;
+            }

             passCATS(byEmail, test);
             resp.getWriter().println("Test '" + test.getDisplayName() + "' was added to user account.");
         } else if (req.getParameter("catsexpire") != null) {
             passCATS(byEmail, test);
             resp.getWriter().println("Test '" + test.getDisplayName() + "' was added to user account.");
         } else if (req.getParameter("catsexpire") != null) {


@@ -387,7 +393,13 @@ public class Manager extends Page {
                 resp.getWriter().println("No test given.");
                 return;
             }
                 resp.getWriter().println("No test given.");
                 return;
             }

-            CATSType test = CATSType.values()[Integer.parseInt(catsTypeId)];
+            CATSType test = null;
+            try {
+                test = CATSType.values()[Integer.parseInt(catsTypeId)];
+            } catch (NumberFormatException e) {
+                resp.getWriter().println("No valid integer given.");
+                return;
+            }

             expireCATS(userByEmail, test);
             resp.getWriter().println("Test '" + test.getDisplayName() + "' is set expired for user account.");
         } else if (req.getParameter("verify") != null) {
             expireCATS(userByEmail, test);
             resp.getWriter().println("Test '" + test.getDisplayName() + "' is set expired for user account.");
         } else if (req.getParameter("verify") != null) {