Revert "Update comment regarding SCANNERAPI-167 and rename lib/batch by lib/scanner"

This reverts commit 0fa4d55b5cbc8a208907deedca21ad70f3fe1df6.

4 files changed, 9 insertions, 11 deletions

diff --git a/server/sonar-server/src/main/java/org/sonar/server/batch/BatchIndex.java b/server/sonar-server/src/main/java/org/sonar/server/batch/BatchIndex.java
index 9fbf1e9e0af..511c90baf40 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/batch/BatchIndex.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/batch/BatchIndex.java
@@ -34,8 +34,7 @@ import org.sonar.api.server.ServerSide;
 import org.sonar.server.platform.ServerFileSystem;
 
 /**
- * Scanner Engine JAR file(s) to be downloaded by sonar-scanner-api. There is currently only one JAR (see assembly.xml)
- * but let's keep possibility to pass several files for possible future evolutions.
+ * JAR files to be downloaded by sonar-runner.
  */
 @ServerSide
 public class BatchIndex implements Startable {
@@ -51,7 +50,7 @@ public class BatchIndex implements Startable {
   @Override
   public void start() {
     StringBuilder sb = new StringBuilder();
-    batchDir = new File(fs.getHomeDir(), "lib/scanner");
+    batchDir = new File(fs.getHomeDir(), "lib/batch");
     if (batchDir.exists()) {
       Collection<File> files = FileUtils.listFiles(batchDir, HiddenFileFilter.VISIBLE, FileFilterUtils.directoryFileFilter());
       for (File file : files) {
diff --git a/server/sonar-server/src/main/java/org/sonar/server/batch/BatchWs.java b/server/sonar-server/src/main/java/org/sonar/server/batch/BatchWs.java
index 8218c8e57fc..0b8b5bcb13b 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/batch/BatchWs.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/batch/BatchWs.java
@@ -55,7 +55,7 @@ public class BatchWs implements WebService {
     controller.createAction("index")
       .setInternal(true)
       .setSince("4.4")
-      .setDescription("List the JAR files to be downloaded by scanners")
+      .setDescription("List the JAR files to be downloaded by source analyzer")
       .setHandler((request, response) -> {
         try {
           response.stream().setMediaType("text/plain");
@@ -71,7 +71,7 @@ public class BatchWs implements WebService {
     NewAction action = controller.createAction("file")
       .setInternal(true)
       .setSince("4.4")
-      .setDescription("Download a JAR file listed in the index (see batch/index)")
+      .setDescription("Download a JAR file required by source analyzer")
       .setResponseExample(getClass().getResource("batch-file-example.txt"))
       .setHandler((request, response) -> {
         String filename = request.mandatoryParam("name");
diff --git a/server/sonar-server/src/main/java/org/sonar/server/platform/web/RoutesFilter.java b/server/sonar-server/src/main/java/org/sonar/server/platform/web/RoutesFilter.java
index 55338541c03..5de1bd2971b 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/platform/web/RoutesFilter.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/platform/web/RoutesFilter.java
@@ -19,6 +19,8 @@
  */
 package org.sonar.server.platform.web;
 
+import static java.lang.String.format;
+
 import java.io.IOException;
 import javax.servlet.Filter;
 import javax.servlet.FilterChain;
@@ -29,8 +31,6 @@ import javax.servlet.ServletResponse;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import static java.lang.String.format;
-
 public class RoutesFilter implements Filter {
 
   private static final String EMPTY = "";
@@ -43,10 +43,10 @@ public class RoutesFilter implements Filter {
     HttpServletResponse response = (HttpServletResponse) servletResponse;
     String path = request.getRequestURI().replaceFirst(request.getContextPath(), EMPTY);
     if (path.startsWith(BATCH_WS + "/") && path.endsWith(".jar")) {
-      // Old scanners were using /batch/file.jar url (see SCANNERAPI-167)
+      // Scanner is still using /batch/file.jar url
       response.sendRedirect(format("%s%s/file?name=%s", request.getContextPath(), BATCH_WS, path.replace(BATCH_WS + "/", EMPTY)));
     } else if ("/batch_bootstrap/index".equals(path)) {
-      // Old scanners were using /batch_bootstrap url (see SCANNERAPI-167)
+      // Scanner is still using /batch_bootstrap url
       response.sendRedirect(format("%s%s/index", request.getContextPath(), BATCH_WS));
     } else if (API_SOURCES_WS.equals(path)) {
       // SONAR-7852 /api/sources?resource url is still used
diff --git a/sonar-application/assembly.xml b/sonar-application/assembly.xml
index bdb5076457e..dc2e1dadf0b 100644
--- a/sonar-application/assembly.xml
+++ b/sonar-application/assembly.xml
@@ -80,9 +80,8 @@
       <scope>provided</scope>
     </dependencySet>
 
-    <!-- Scanner Engine that is returned by batch/index WS -->
     <dependencySet>
-      <outputDirectory>lib/scanner</outputDirectory>
+      <outputDirectory>lib/batch</outputDirectory>
       <useTransitiveDependencies>false</useTransitiveDependencies>
       <includes>
         <include>org.sonarsource.sonarqube:sonar-scanner-engine-shaded</include>