diff options
author | Simon Brandhof <simon.brandhof@sonarsource.com> | 2014-07-01 22:13:28 +0200 |
---|---|---|
committer | Simon Brandhof <simon.brandhof@sonarsource.com> | 2014-07-01 22:33:45 +0200 |
commit | 09b33b23a4d08ba107572c8b2e8ca30302e41db1 (patch) | |
tree | eaa43ddb8fee352f94db279d2e48202ff2d114e4 /sonar-core | |
parent | 8576bc7a4c715298a585f72d06c29617b17a8991 (diff) | |
download | sonarqube-09b33b23a4d08ba107572c8b2e8ca30302e41db1.tar.gz sonarqube-09b33b23a4d08ba107572c8b2e8ca30302e41db1.zip |
Fix some quality flaws
Diffstat (limited to 'sonar-core')
-rw-r--r-- | sonar-core/src/main/java/org/sonar/core/plugins/PluginClassloaders.java | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/sonar-core/src/main/java/org/sonar/core/plugins/PluginClassloaders.java b/sonar-core/src/main/java/org/sonar/core/plugins/PluginClassloaders.java index 976505dc3c4..d00f8242c95 100644 --- a/sonar-core/src/main/java/org/sonar/core/plugins/PluginClassloaders.java +++ b/sonar-core/src/main/java/org/sonar/core/plugins/PluginClassloaders.java @@ -131,10 +131,11 @@ public class PluginClassloaders { } return realm; } catch (UnsupportedClassVersionError e) { - throw new SonarException("The plugin " + plugin.getKey() + " is not supported with Java " + SystemUtils.JAVA_VERSION_TRIMMED, e); + throw new SonarException(String.format("The plugin %s is not supported with Java %s", plugin.getKey(), + SystemUtils.JAVA_VERSION_TRIMMED), e); } catch (Exception e) { - throw new SonarException("Fail to build the classloader of " + plugin.getKey(), e); + throw new SonarException(String.format("Fail to build the classloader of %s", plugin.getKey()), e); } } @@ -146,7 +147,8 @@ public class PluginClassloaders { ClassRealm base = world.getRealm(plugin.getBasePlugin()); if (base == null) { // Ignored, because base plugin is not installed - LOG.warn("Plugin " + plugin.getKey() + " is ignored because base plugin is not installed: " + plugin.getBasePlugin()); + LOG.warn(String.format("Plugin %s is ignored because base plugin is not installed: %s", + plugin.getKey(), plugin.getBasePlugin())); return false; } // we create new realm to be able to return it by key without conversion to baseKey @@ -156,10 +158,12 @@ public class PluginClassloaders { } return true; } catch (UnsupportedClassVersionError e) { - throw new SonarException("The plugin " + plugin.getKey() + " is not supported with Java " + SystemUtils.JAVA_VERSION_TRIMMED, e); + throw new SonarException(String.format("The plugin %s is not supported with Java %s", + plugin.getKey(), SystemUtils.JAVA_VERSION_TRIMMED), e); } catch (Exception e) { - throw new SonarException("Fail to extend the plugin " + plugin.getBasePlugin() + " for " + plugin.getKey(), e); + throw new SonarException(String.format("Fail to extend the plugin %s for %s", + plugin.getBasePlugin(), plugin.getKey()), e); } } @@ -223,10 +227,11 @@ public class PluginClassloaders { return (Plugin) clazz.newInstance(); } catch (UnsupportedClassVersionError e) { - throw new SonarException("The plugin " + plugin.getKey() + " is not supported with Java " + SystemUtils.JAVA_VERSION_TRIMMED, e); + throw new SonarException(String.format("The plugin %s is not supported with Java %s", + plugin.getKey(), SystemUtils.JAVA_VERSION_TRIMMED), e); } catch (Exception e) { - throw new SonarException("Fail to load plugin " + plugin.getKey(), e); + throw new SonarException(String.format("Fail to load plugin %s", plugin.getKey()), e); } } |