[JENKINS-37041] Address Code review comments

(cherry picked from commit d92fd4cc63)
This commit is contained in:
Stephen Connolly authored and Oliver Gondža committed 2016-08-17 20:00:05 +02:00
1 parent 6ea7883fed
commit 28ee03724d
2 files changed
+9 -13

No files matched your search

+3 -3
View File
@@ -719,11 +719,11 @@ public abstract class PluginManager extends AbstractModelObject implements OnMas
final Set<ClassicPluginStrategy.DetachedPlugin> forceUpgrade = new HashSet<>();
for (ClassicPluginStrategy.DetachedPlugin p : ClassicPluginStrategy.getDetachedPlugins()) {
VersionNumber installedVersion = getPluginVersion(rootDir, p.getShortName());
VersionNumber requireVersion = p.getRequireVersion();
if (installedVersion != null && installedVersion.isOlderThan(requireVersion)) {
VersionNumber requiredVersion = p.getRequiredVersion();
if (installedVersion != null && installedVersion.isOlderThan(requiredVersion)) {
LOGGER.log(Level.WARNING,
"Detached plugin {0} found at version {1}, required minimum version is {2}",
new Object[]{p.getShortName(), installedVersion, requireVersion});
new Object[]{p.getShortName(), installedVersion, requiredVersion});
forceUpgrade.add(p);
}
}