Hotfix 1.0.5, avoid issue sharing between files that use the same parser

Author: adrian-palanques-cloudappi

its/pom.xml

@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
         <artifactId>dosonarapi</artifactId>
-        <version>1.0.4</version>
+        <version>1.0.5</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>

openapi-checks/pom.xml

@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
     <artifactId>dosonarapi</artifactId>
-    <version>1.0.4</version>
+    <version>1.0.5</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 

openapi-front-end/pom.xml

@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
     <artifactId>dosonarapi</artifactId>
-    <version>1.0.4</version>
+    <version>1.0.5</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 

openapi-test-tools/pom.xml

@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
     <artifactId>dosonarapi</artifactId>
-    <version>1.0.4</version>
+    <version>1.0.5</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 

pom.xml

@@ -9,7 +9,7 @@
 
   <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
   <artifactId>dosonarapi</artifactId>
-  <version>1.0.4</version>
+  <version>1.0.5</version>
   <packaging>pom</packaging>
 
   <name>SonarOpenAPI</name>

sonar-openapi-plugin/pom.xml

@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
     <artifactId>dosonarapi</artifactId>
-    <version>1.0.4</version>
+    <version>1.0.5</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 

sonar-openapi-plugin/src/main/java/org/apiaddicts/apitools/dosonarapi/plugin/OpenApiAnalyzer.java

@@ -60,11 +60,9 @@ public class OpenApiAnalyzer {
   private final SensorContext context;
   private final List<InputFile> inputFiles;
   private final OpenApiChecks checks;
-  private final YamlParser v2parser;
-  private final YamlParser v3parser;
-  private final YamlParser genericParser;
   private final NoSonarFilter noSonarFilter;
   private final OpenApiCpdAnalyzer cpdAnalyzer;
+  private final OpenApiConfiguration configuration;
   private FileLinesContextFactory fileLinesContextFactory;
 
   public OpenApiAnalyzer(SensorContext context, OpenApiChecks checks, FileLinesContextFactory fileLinesContextFactory, NoSonarFilter noSonarFilter, List<InputFile> inputFiles/*, boolean isv2*/) {
@@ -74,10 +72,7 @@ public OpenApiAnalyzer(SensorContext context, OpenApiChecks checks, FileLinesCon
     this.noSonarFilter = noSonarFilter;
     this.cpdAnalyzer = new OpenApiCpdAnalyzer(context);
     this.inputFiles = inputFiles;
-    OpenApiConfiguration configuration = new OpenApiConfiguration(context.fileSystem().encoding(), true);
-    this.v2parser = OpenApiParser.createV2(configuration);
-    this.v3parser = OpenApiParser.createV3(configuration);
-    this.genericParser = OpenApiParser.createGeneric(configuration);
+    this.configuration = new OpenApiConfiguration(context.fileSystem().encoding(), true);
   }
 
   private static NewIssueLocation newLocation(InputFile inputFile, NewIssue issue, IssueLocation location) {
@@ -122,12 +117,12 @@ private void scanFile(InputFile inputFile) {
     try {
       String content = getContent(inputFile);
       if (!content.contains("swagger") && !content.contains("openapi")) return;
-      JsonNode rootNode = genericParser.parse(content);
+      JsonNode rootNode = OpenApiParser.createGeneric(configuration).parse(content);
       boolean isV2 = !rootNode.at("/swagger").isMissing();
       boolean isV3 = !rootNode.at("/openapi").isMissing();
       YamlParser targetParser = null;
-      if (isV2) targetParser = v2parser;
-      if (isV3) targetParser = v3parser;
+      if (isV2) targetParser = OpenApiParser.createV2(configuration);
+      if (isV3) targetParser = OpenApiParser.createV3(configuration);
       if (targetParser == null) return;
 
       visitorContext = new OpenApiVisitorContext(targetParser.parse(content), targetParser.getIssues(), openApiFile);
@@ -138,12 +133,10 @@ private void scanFile(InputFile inputFile) {
       for (ValidationException cause : e.getCauses()) {
         dumpException(cause, inputFile);
       }
-
     } catch (RecognitionException e) {
       visitorContext = new OpenApiVisitorContext(openApiFile, e);
       LOG.error("Unable to parse file in recognition: " + inputFile.filename() + "\"\n" + e.getMessage());
       dumpException(e, inputFile);
-
     } catch (IOException ex) {
       RecognitionException re = new RecognitionException(0, ex.getMessage());
       visitorContext = new OpenApiVisitorContext(openApiFile, re);