Fix possible NPE when moving to snapshots

See gh-43045
This commit is contained in:
Andy Wilkinson 2025-02-08 10:09:38 +00:00
parent e40a7686cd
commit b7499222b8
1 changed files with 12 additions and 10 deletions

View File

@ -92,19 +92,21 @@ public abstract class MoveToSnapshots extends UpgradeDependencies {
VersionOption versionOption = resolver.apply(library, dependencyVersion); VersionOption versionOption = resolver.apply(library, dependencyVersion);
if (versionOption != null) { if (versionOption != null) {
List<Release> releases = scheduledReleases.get(library.getCalendarName()); List<Release> releases = scheduledReleases.get(library.getCalendarName());
List<Release> matches = releases.stream() if (releases != null) {
.filter((release) -> dependencyVersion.isSnapshotFor(release.getVersion())) List<Release> matches = releases.stream()
.toList(); .filter((release) -> dependencyVersion.isSnapshotFor(release.getVersion()))
if (matches.isEmpty()) { .toList();
if (logger.isInfoEnabled()) { if (!matches.isEmpty()) {
logger.info("Ignoring {}. No release of {} scheduled before {}", dependencyVersion, return new VersionOption.SnapshotVersionOption(versionOption.getVersion(),
library.getName(), milestone.getDueOn()); matches.get(0).getVersion());
} }
return null;
} }
return new VersionOption.SnapshotVersionOption(versionOption.getVersion(), matches.get(0).getVersion()); if (logger.isInfoEnabled()) {
logger.info("Ignoring {}. No release of {} scheduled before {}", dependencyVersion,
library.getName(), milestone.getDueOn());
}
} }
return versionOption; return null;
}; };
} }