Hotfix 1.0.4, enchance issue location to be more fault tolerant

Author: adrian-palanques-cloudappi

its/pom.xml

@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
         <artifactId>dosonarapi</artifactId>
-        <version>1.0.3</version>
+        <version>1.0.4</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.3</version>
+    <version>1.0.4</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.3</version>
+    <version>1.0.4</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.3</version>
+    <version>1.0.4</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 

pom.xml

@@ -9,7 +9,7 @@
 
   <groupId>org.apiaddicts.apitools.dosonarapi</groupId>
   <artifactId>dosonarapi</artifactId>
-  <version>1.0.3</version>
+  <version>1.0.4</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.3</version>
+    <version>1.0.4</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 

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

@@ -87,7 +87,15 @@ private static NewIssueLocation newLocation(InputFile inputFile, NewIssue issue,
       if (location.startLineOffset() == IssueLocation.UNDEFINED_OFFSET) {
         range = inputFile.selectLine(location.startLine());
       } else {
-        range = inputFile.newRange(location.startLine(), location.startLineOffset(), location.endLine(), location.endLineOffset());
+        try {
+          range = inputFile.newRange(location.startLine(), location.startLineOffset(), location.endLine(), location.endLineOffset());
+        } catch (IllegalArgumentException e) {
+          try {
+            range = inputFile.selectLine(location.startLine());
+          } catch (IllegalArgumentException e2) {
+            range = inputFile.selectLine(1);
+          }
+        }
       }
       newLocation.at(range);
     }
@@ -133,13 +141,14 @@ private void scanFile(InputFile inputFile) {
 
     } catch (RecognitionException e) {
       visitorContext = new OpenApiVisitorContext(openApiFile, e);
-      LOG.error("Unable to parse file: " + inputFile.filename() + "\"\n" + e.getMessage());
+      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);
-      LOG.error("Unable to parse file: " + inputFile.filename() + "\"\n" + ex.getMessage());
+      LOG.error("Unable to parse file in i/o: " + inputFile.filename() + "\"\n" + ex.getMessage());
+      dumpException(re, inputFile);
     }
 
     for (OpenApiCheck check : checks.all()) {
@@ -172,11 +181,11 @@ private String getContent(InputFile inputFile) throws IOException {
 
   private void dumpException(RecognitionException e, InputFile inputFile) {
     int line = e.getLine();
-    if (line == 0) {
+    if (line == 0 || line > inputFile.lines()) {
       line = 1;
     }
     int column = 0;
-    if (e instanceof ValidationException) {
+    if (line != 1 && (e instanceof ValidationException)) {
       column = ((ValidationException) e).getNode().getToken().getColumn();
       for (ValidationException cause : ((ValidationException) e).getCauses()) {
         dumpException(cause, inputFile);