Move Scripts and dependent classes to web module

Author: ahornace

opengrok-indexer/src/main/java/org/opengrok/indexer/search/SearchEngine.java

@@ -34,13 +34,11 @@
 import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.Set;
 import java.util.SortedSet;
 import java.util.TreeSet;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 import java.util.zip.GZIPInputStream;
-import javax.servlet.http.HttpServletRequest;
 import org.apache.lucene.analysis.charfilter.HTMLStripCharFilter;
 import org.apache.lucene.document.Document;
 import org.apache.lucene.index.DirectoryReader;
@@ -68,9 +66,7 @@
 import org.opengrok.indexer.search.context.Context;
 import org.opengrok.indexer.search.context.HistoryContext;
 import org.opengrok.indexer.util.TandemPath;
-import org.opengrok.indexer.web.PageConfig;
 import org.opengrok.indexer.web.Prefix;
-import org.opengrok.indexer.web.ProjectHelper;
 
 /**
  * This is an encapsulation of the details on how to search in the index database.
@@ -276,49 +272,11 @@ public IndexSearcher getSearcher() {
      * Call to search() must be eventually followed by call to destroy()
      * so that IndexSearcher objects are properly freed.
      *
-     * @param req request
-     * @param projectNames names of the projects
+     * @param projects projects to search
      * @return The number of hits
-     * @see ProjectHelper#getAllProjects()
-     */
-    public int search(HttpServletRequest req, String... projectNames) {
-        ProjectHelper pHelper = PageConfig.get(req).getProjectHelper();
-        Set<Project> allProjects = pHelper.getAllProjects();
-        List<Project> filteredProjects = new ArrayList<>();
-        for (Project project: allProjects) {
-            for (String name : projectNames) {
-                if (project.getName().equalsIgnoreCase(name)) {
-                    filteredProjects.add(project);
-                }
-            }
-        }
-        return search(
-                filteredProjects,
-                new File(RuntimeEnvironment.getInstance().getDataRootFile(), IndexDatabase.INDEX_DIR));
-    }
-
-    /**
-     * Execute a search aware of current request.
-     *
-     * This filters out all projects which are not allowed for the current request.
-     *
-     * Before calling this function,
-     * you must set the appropriate search criteria with the set-functions. Note
-     * that this search will return the first cachePages of hitsPerPage, for
-     * more you need to call more.
-     *
-     * Call to search() must be eventually followed by call to destroy()
-     * so that IndexSearcher objects are properly freed.
-     *
-     * @param req request
-     * @return The number of hits
-     * @see ProjectHelper#getAllProjects()
      */
-    public int search(HttpServletRequest req) {
-        ProjectHelper pHelper = PageConfig.get(req).getProjectHelper();
-        return search(
-                new ArrayList<Project>(pHelper.getAllProjects()),
-                new File(RuntimeEnvironment.getInstance().getDataRootFile(), IndexDatabase.INDEX_DIR));
+    public int search(List<Project> projects) {
+        return search(projects, new File(RuntimeEnvironment.getInstance().getDataRootFile(), IndexDatabase.INDEX_DIR));
     }
 
     /**

opengrok-web/src/main/java/org/opengrok/web/AuthorizationFilter.java

@@ -37,7 +37,6 @@
 
 import org.opengrok.indexer.configuration.Project;
 import org.opengrok.indexer.logger.LoggerFactory;
-import org.opengrok.indexer.web.PageConfig;
 import org.opengrok.indexer.web.Laundromat;
 import org.opengrok.web.api.v1.RestApp;
 

opengrok-web/src/main/java/org/opengrok/web/GetFile.java

@@ -33,7 +33,6 @@
 import java.io.OutputStream;
 
 import org.opengrok.indexer.history.HistoryGuru;
-import org.opengrok.indexer.web.PageConfig;
 import org.opengrok.indexer.web.Prefix;
 
 import javax.servlet.http.HttpServlet;

opengrok-indexer/src/main/java/org/opengrok/indexer/web/PageConfig.java renamed to opengrok-web/src/main/java/org/opengrok/web/PageConfig.java

@@ -22,7 +22,7 @@
  * Portions copyright (c) 2011 Jens Elkner.
  * Portions Copyright (c) 2017-2018, 2020, Chris Fraire <[email protected]>.
  */
-package org.opengrok.indexer.web;
+package org.opengrok.web;
 
 import static org.opengrok.indexer.index.Indexer.PATH_SEPARATOR;
 import static org.opengrok.indexer.index.Indexer.PATH_SEPARATOR_STRING;
@@ -81,6 +81,15 @@
 import org.opengrok.indexer.util.IOUtils;
 import org.opengrok.indexer.util.LineBreaker;
 import org.opengrok.indexer.util.TandemPath;
+import org.opengrok.indexer.web.DiffData;
+import org.opengrok.indexer.web.DiffType;
+import org.opengrok.indexer.web.EftarFileReader;
+import org.opengrok.indexer.web.Laundromat;
+import org.opengrok.indexer.web.Prefix;
+import org.opengrok.indexer.web.QueryParameters;
+import org.opengrok.indexer.web.SearchHelper;
+import org.opengrok.indexer.web.SortOrder;
+import org.opengrok.indexer.web.Util;
 import org.opengrok.indexer.web.messages.MessagesContainer.AcceptedMessage;
 import org.suigeneris.jrcs.diff.Diff;
 import org.suigeneris.jrcs.diff.DifferentiationFailedException;

opengrok-indexer/src/main/java/org/opengrok/indexer/web/ProjectHelper.java renamed to opengrok-web/src/main/java/org/opengrok/web/ProjectHelper.java

@@ -22,9 +22,7 @@
  * Portions Copyright (c) 2018, Chris Fraire <[email protected]>.
  * Portions Copyright (c) 2019, Krystof Tulinger <[email protected]>.
  */
-package org.opengrok.indexer.web;
-
-import static org.opengrok.indexer.web.PageConfig.OPEN_GROK_PROJECT;
+package org.opengrok.web;
 
 import java.util.ArrayList;
 import java.util.Comparator;
@@ -40,6 +38,8 @@
 import org.opengrok.indexer.configuration.Project;
 import org.opengrok.indexer.history.RepositoryInfo;
 
+import static org.opengrok.web.PageConfig.OPEN_GROK_PROJECT;
+
 /**
  * Preprocessing of projects, repositories and groups for the UI.
  *

opengrok-indexer/src/main/java/org/opengrok/indexer/web/Scripts.java renamed to opengrok-web/src/main/java/org/opengrok/web/Scripts.java

@@ -21,7 +21,7 @@
  * Copyright (c) 2017, 2020 Oracle and/or its affiliates. All rights reserved.
  * Portions Copyright (c) 2017, 2020, Chris Fraire <[email protected]>.
  */
-package org.opengrok.indexer.web;
+package org.opengrok.web;
 
 import java.util.Comparator;
 import java.util.Iterator;

opengrok-web/src/main/java/org/opengrok/web/StatisticsFilter.java

@@ -37,7 +37,6 @@
 import io.micrometer.core.instrument.DistributionSummary;
 import io.micrometer.core.instrument.Timer;
 import org.opengrok.indexer.Metrics;
-import org.opengrok.indexer.web.PageConfig;
 import org.opengrok.indexer.web.SearchHelper;
 
 public class StatisticsFilter implements Filter {

opengrok-web/src/main/java/org/opengrok/web/WebappListener.java

@@ -31,7 +31,6 @@
 import org.opengrok.indexer.configuration.CommandTimeoutType;
 import org.opengrok.indexer.configuration.RuntimeEnvironment;
 import org.opengrok.indexer.logger.LoggerFactory;
-import org.opengrok.indexer.web.PageConfig;
 import org.opengrok.indexer.web.SearchHelper;
 import org.opengrok.web.api.v1.suggester.provider.service.SuggesterServiceFactory;
 

opengrok-web/src/main/java/org/opengrok/web/api/v1/controller/SearchController.java

@@ -24,9 +24,11 @@
 package org.opengrok.web.api.v1.controller;
 
 import org.apache.lucene.search.Query;
+import org.opengrok.indexer.configuration.Project;
 import org.opengrok.indexer.search.Hit;
 import org.opengrok.indexer.search.SearchEngine;
 import org.opengrok.indexer.web.QueryParameters;
+import org.opengrok.web.PageConfig;
 import org.opengrok.web.api.v1.filter.CorsEnable;
 import org.opengrok.web.api.v1.suggester.provider.service.SuggesterService;
 
@@ -47,6 +49,7 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 import java.util.stream.Collectors;
 
 @Path(SearchController.PATH)
@@ -126,10 +129,13 @@ public List<Hit> search(
                 final int startDocIndex,
                 final int maxResults
         ) {
+            Set<Project> allProjects = PageConfig.get(req).getProjectHelper().getAllProjects();
             if (projects == null || projects.isEmpty()) {
-                numResults = engine.search(req);
+                numResults = engine.search(new ArrayList<>(allProjects));
             } else {
-                numResults = engine.search(req, projects.toArray(new String[0]));
+                numResults = engine.search(allProjects.stream()
+                        .filter(p -> projects.contains(p.getName()))
+                        .collect(Collectors.toList()));
             }
 
             if (startDocIndex > numResults) {

opengrok-web/src/main/webapp/WEB-INF/tags/repositories.tag

@@ -23,7 +23,7 @@ Portions Copyright (c) 2019, Krystof Tulinger <[email protected]>.
 <%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn" %>
 <%@ taglib prefix="opengrok" tagdir="/WEB-INF/tags" %>
 
-<%@ attribute name="pageConfig" required="true" type="org.opengrok.indexer.web.PageConfig" %>
+<%@ attribute name="pageConfig" required="true" type="org.opengrok.web.PageConfig" %>
 <%@ attribute name="repositories" required="true" type="java.util.Set<org.opengrok.indexer.configuration.Project>" %>
 
 <c:if test="${repositories.size() > 0}">