]> source.dussan.org Git - sonarqube.git/commitdiff
Fix code quality issues
authorDuarte Meneses <duarte.meneses@sonarsource.com>
Wed, 10 Jul 2019 15:41:04 +0000 (10:41 -0500)
committerSonarTech <sonartech@sonarsource.com>
Fri, 12 Jul 2019 18:21:15 +0000 (20:21 +0200)
sonar-plugin-api-impl/src/main/java/org/sonar/api/batch/fs/internal/DefaultIndexedFile.java
sonar-plugin-api-impl/src/main/java/org/sonar/api/batch/fs/internal/DefaultTextPointer.java
sonar-plugin-api-impl/src/main/java/org/sonar/api/batch/fs/internal/DefaultTextRange.java
sonar-plugin-api-impl/src/main/java/org/sonar/api/batch/fs/internal/TestInputFileBuilder.java
sonar-plugin-api-impl/src/main/java/org/sonar/api/batch/sensor/internal/DefaultStorable.java
sonar-plugin-api-impl/src/main/java/org/sonar/api/impl/ws/ValidatingRequest.java
sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java

index 229280fca5b2518df0907c19f8925abc0f579c90..59b9c06fff232f31178c2e0050f1611456ed55a9 100644 (file)
@@ -126,7 +126,7 @@ public class DefaultIndexedFile extends DefaultInputComponent implements Indexed
       return true;
     }
 
-    if (!(o instanceof DefaultIndexedFile)) {
+    if (o == null || o.getClass() != this.getClass()) {
       return false;
     }
 
index c10efd01de7f03a258f258d21dc4e18a2ca82b7b..200bda8427b20b0799b893f01bbc211fe0ec33a7 100644 (file)
@@ -51,7 +51,7 @@ public class DefaultTextPointer implements TextPointer {
 
   @Override
   public boolean equals(Object obj) {
-    if (!(obj instanceof DefaultTextPointer)) {
+    if (obj == null || obj.getClass() != this.getClass()) {
       return false;
     }
     DefaultTextPointer other = (DefaultTextPointer) obj;
index efef79cf3678d8886e2c1058747c6c20d129d887..c46deeffe14c0922b2bbfe5b4d6e1940ae20f514 100644 (file)
@@ -59,7 +59,7 @@ public class DefaultTextRange implements TextRange {
 
   @Override
   public boolean equals(Object obj) {
-    if (!(obj instanceof DefaultTextRange)) {
+    if (obj == null || obj.getClass() != this.getClass()) {
       return false;
     }
     DefaultTextRange other = (DefaultTextRange) obj;
index 00f9319c2d466e6bba7b48c3c2b972d6d89ef3c5..bcbb27428d77127dedafb668eba009ca42603c8a 100644 (file)
@@ -86,10 +86,10 @@ public class TestInputFileBuilder {
    * filePath must point to a file that is within the module base directory.
    */
   public TestInputFileBuilder(String projectKey, File moduleBaseDir, File filePath) {
-    String relativePath = moduleBaseDir.toPath().relativize(filePath.toPath()).toString();
+    String relativePathStr = moduleBaseDir.toPath().relativize(filePath.toPath()).toString();
     this.projectKey = projectKey;
     setModuleBaseDir(moduleBaseDir.toPath());
-    this.relativePath = PathUtils.sanitize(relativePath);
+    this.relativePath = PathUtils.sanitize(relativePathStr);
     this.id = batchId++;
   }
 
index 822ffbc2a400a8a86b09d70ab9357f8db8c51fbf..d0ff1b59e60a5db4006273554912da0a55a30f52 100644 (file)
@@ -28,8 +28,8 @@ import static org.sonar.api.utils.Preconditions.checkState;
 
 public abstract class DefaultStorable {
 
-  protected final transient SensorStorage storage;
-  private transient boolean saved = false;
+  protected final SensorStorage storage;
+  private boolean saved = false;
 
   public DefaultStorable() {
     this.storage = null;
index 6e7d90b6a0634f7559c240e6cb4b2eb3d4b180f7..dda4cbd337cfac7a271b87b1ef6c53dc0a66725c 100644 (file)
@@ -85,6 +85,9 @@ public abstract class ValidatingRequest extends Request {
   @Override
   public List<String> multiParam(String key) {
     WebService.Param definition = action.param(key);
+    if (definition == null) {
+      throw new IllegalArgumentException("Parameter '" + key + "' not found for action '" + action.key() + "'");
+    }
     List<String> values = readMultiParamOrDefaultValue(key, definition);
     return validateValues(values, definition);
   }
@@ -224,7 +227,7 @@ public abstract class ValidatingRequest extends Request {
     checkArgument(valueAsInt <= maximumValue, "'%s' value (%s) must be less than %s", key, valueAsInt, maximumValue);
   }
 
-  private static void validateRequiredValue(String key, WebService.Param definition, String value) {
+  private static void validateRequiredValue(String key, WebService.Param definition, @Nullable String value) {
     boolean required = definition.isRequired();
     if (required) {
       checkArgument(value != null, format(MSG_PARAMETER_MISSING, key));
index ed0622fb956647946f09c36ea0c5c781ec00cd08..c91f9863745c777f5c00a953effc58e90b2296f5 100644 (file)
@@ -591,7 +591,7 @@ public class RequestTest {
   @Test
   public void fail_if_multi_param_is_not_defined() {
     expectedException.expect(IllegalArgumentException.class);
-    expectedException.expectMessage("BUG - parameter 'unknown' is undefined for action 'my_action'");
+    expectedException.expectMessage("Parameter 'unknown' not found for action 'my_action'");
 
     underTest.multiParam("unknown");
   }