diff --git a/src/main/java/org/jenkinsci/plugins/rolestrategy/AmbiguousSidsAdminMonitor.java b/src/main/java/org/jenkinsci/plugins/rolestrategy/AmbiguousSidsAdminMonitor.java index 2f487458..73635d29 100644 --- a/src/main/java/org/jenkinsci/plugins/rolestrategy/AmbiguousSidsAdminMonitor.java +++ b/src/main/java/org/jenkinsci/plugins/rolestrategy/AmbiguousSidsAdminMonitor.java @@ -3,6 +3,7 @@ import com.michelin.cio.hudson.plugins.rolestrategy.AuthorizationType; import com.michelin.cio.hudson.plugins.rolestrategy.Messages; import com.michelin.cio.hudson.plugins.rolestrategy.PermissionEntry; +import edu.umd.cs.findbugs.annotations.NonNull; import hudson.Extension; import hudson.ExtensionList; import hudson.model.AdministrativeMonitor; @@ -10,7 +11,6 @@ import java.util.Collection; import java.util.Collections; import java.util.List; -import javax.annotation.Nonnull; import org.kohsuke.accmod.Restricted; import org.kohsuke.accmod.restrictions.NoExternalUse; @@ -23,9 +23,9 @@ @Restricted(NoExternalUse.class) public class AmbiguousSidsAdminMonitor extends AdministrativeMonitor { - private @Nonnull List ambiguousEntries = Collections.emptyList(); + private @NonNull List ambiguousEntries = Collections.emptyList(); - public static @Nonnull AmbiguousSidsAdminMonitor get() { + public static @NonNull AmbiguousSidsAdminMonitor get() { return ExtensionList.lookupSingleton(AmbiguousSidsAdminMonitor.class); } @@ -34,7 +34,7 @@ public class AmbiguousSidsAdminMonitor extends AdministrativeMonitor { * * @param entries All entries in the system. */ - public void updateEntries(@Nonnull Collection entries) { + public void updateEntries(@NonNull Collection entries) { List ambiguous = new ArrayList<>(); for (PermissionEntry entry : entries) { try { @@ -48,7 +48,7 @@ public void updateEntries(@Nonnull Collection entries) { ambiguousEntries = ambiguous; } - public @Nonnull List getAmbiguousEntries() { + public @NonNull List getAmbiguousEntries() { return ambiguousEntries; }