From dd1bdf6a2a93bcb6d74521d3305565abfe68174b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Knud=20=C3=85ge=20Hansen?= <kaah@statsbiblioteket.dk>
Date: Fri, 29 Sep 2017 10:21:29 +0200
Subject: [PATCH] NAS-2642: Refactored a bit again

---
 .../webinterface/servlet/JobResource.java     | 55 +++++++++----------
 1 file changed, 27 insertions(+), 28 deletions(-)

diff --git a/harvester/harvester-core/src/main/java/dk/netarkivet/harvester/webinterface/servlet/JobResource.java b/harvester/harvester-core/src/main/java/dk/netarkivet/harvester/webinterface/servlet/JobResource.java
index b9cbea6eae..6bbf0a82ae 100644
--- a/harvester/harvester-core/src/main/java/dk/netarkivet/harvester/webinterface/servlet/JobResource.java
+++ b/harvester/harvester-core/src/main/java/dk/netarkivet/harvester/webinterface/servlet/JobResource.java
@@ -604,11 +604,6 @@ public void job(HttpServletRequest req, HttpServletResponse resp, List<Integer>
     }
 
     public void crawllog_list(HttpServletRequest req, HttpServletResponse resp, List<Integer> numerics) throws IOException {
-
-//        page = getPage(req, page);
-//        linesPerPage = getLinesPerPage(req, linesPerPage);
-//        pageString = getParameterPage(req, pageString);
-
         long lines;
         long linesPerPage = 100;
         long page = 1;
@@ -622,31 +617,35 @@ public void crawllog_list(HttpServletRequest req, HttpServletResponse resp, List
         TemplateBuilderFactory<MasterTemplateBuilder> tplBuilder = TemplateBuilderFactory.getInstance(environment.templateMaster, "master.tpl", "UTF-8", MasterTemplateBuilder.class);
         MasterTemplateBuilder masterTplBuilder = tplBuilder.getTemplateBuilder();
 
-        String tmpStr;
-        tmpStr = req.getParameter("page");
-        if (tmpStr != null && tmpStr.length() > 0) {
-            try {
-                page = Long.parseLong(tmpStr);
-            } catch (NumberFormatException e) {
-            }
-        }
-        tmpStr = req.getParameter("itemsperpage");
-        if (tmpStr != null && tmpStr.length() > 0) {
-            try {
-                linesPerPage = Long.parseLong(tmpStr);
-            } catch (NumberFormatException e) {
-            }
-        }
-
-        if (linesPerPage < 25) {
-            linesPerPage = 25;
-        }
-        if (linesPerPage > 1000) {
-            linesPerPage = 1000;
-        }
+        page = getPage(req, page);
+//        String tmpStr;
+//        tmpStr = req.getParameter("page");
+//        if (tmpStr != null && tmpStr.length() > 0) {
+//            try {
+//                page = Long.parseLong(tmpStr);
+//            } catch (NumberFormatException e) {
+//            }
+//        }
+
+        linesPerPage = getLinesPerPage(req, linesPerPage);
+//        tmpStr = req.getParameter("itemsperpage");
+//        if (tmpStr != null && tmpStr.length() > 0) {
+//            try {
+//                linesPerPage = Long.parseLong(tmpStr);
+//            } catch (NumberFormatException e) {
+//            }
+//        }
+//
+//        if (linesPerPage < 25) {
+//            linesPerPage = 25;
+//        }
+//        if (linesPerPage > 1000) {
+//            linesPerPage = 1000;
+//        }
 
 
-        tmpStr = req.getParameter("q");
+//        pageString = getParameterPage(req, pageString);
+        String tmpStr = req.getParameter("q");
         if (tmpStr != null && tmpStr.length() > 0 && !tmpStr.equalsIgnoreCase(".*")) {
             pageString = tmpStr;
         }