diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/TemplatePaperPlugin.java b/src/main/java/org/esoteric/minecraft/plugins/template/TemplatePaperPlugin.java index 296c5777..92923156 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/TemplatePaperPlugin.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/TemplatePaperPlugin.java @@ -2,9 +2,6 @@ import org.bukkit.NamespacedKey; import org.bukkit.plugin.java.JavaPlugin; -import org.checkerframework.checker.nullness.qual.NonNull; -import org.checkerframework.checker.nullness.qual.Nullable; -import org.checkerframework.framework.qual.DefaultQualifier; import dev.jorel.commandapi.CommandAPI; import dev.jorel.commandapi.CommandAPIBukkitConfig; @@ -25,65 +22,48 @@ import org.esoteric.minecraft.plugins.template.language.Message; import org.esoteric.minecraft.plugins.template.resourcepack.ResourcePackManager; -@DefaultQualifier(NonNull.class) public final class TemplatePaperPlugin extends JavaPlugin { - @Nullable private FileManager fileManager; - @Nullable private PlayerDataManager playerDataManager; - @Nullable private LanguageManager languageManager; - @Nullable private ResourcePackManager resourcePackManager; - @Nullable private HttpServerManager httpServerManager; - @Nullable private CustomItemManager customItemManager; - @Nullable private CustomMultientityManager customMultientityManager; - @Nullable private CustomMultiblockManager customMultiblockManager; private final NamespacedKey customItemIdKey = new NamespacedKey(this, "custom_item_id"); private final NamespacedKey customEntityIdKey = new NamespacedKey(this, "custom_entity_id"); - @Nullable public FileManager getFileManager() { return fileManager; } - @Nullable public PlayerDataManager getPlayerDataManager() { return playerDataManager; } - @Nullable public LanguageManager getLanguageManager() { return languageManager; } - @Nullable public ResourcePackManager getResourcePackManager() { return resourcePackManager; } - @Nullable public HttpServerManager getHttpServerManager() { return httpServerManager; } - @Nullable public CustomItemManager getCustomItemManager() { return customItemManager; } - @Nullable public CustomMultientityManager getCustomMultientityManager() { return customMultientityManager; } - @Nullable public CustomMultiblockManager getCustomMultiblockManager() { return customMultiblockManager; } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/commands/SetLanguageCommand.java b/src/main/java/org/esoteric/minecraft/plugins/template/commands/SetLanguageCommand.java index 970a57e2..56ae2cb7 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/commands/SetLanguageCommand.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/commands/SetLanguageCommand.java @@ -13,10 +13,11 @@ import org.esoteric.minecraft.plugins.template.language.LanguageManager; import org.esoteric.minecraft.plugins.template.language.Message; import net.kyori.adventure.text.Component; +import org.jetbrains.annotations.NotNull; public class SetLanguageCommand extends CommandAPICommand { - public SetLanguageCommand(TemplatePaperPlugin plugin) { + public SetLanguageCommand(@NotNull TemplatePaperPlugin plugin) { super("set-language"); LanguageManager languageManager = plugin.getLanguageManager(); diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/custom/items/AbstractCustomItem.java b/src/main/java/org/esoteric/minecraft/plugins/template/custom/items/AbstractCustomItem.java index d4db73f3..dc4205a2 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/custom/items/AbstractCustomItem.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/custom/items/AbstractCustomItem.java @@ -7,8 +7,8 @@ import org.bukkit.inventory.ItemStack; import org.bukkit.persistence.PersistentDataType; -import org.checkerframework.checker.nullness.qual.NonNull; import org.esoteric.minecraft.plugins.template.TemplatePaperPlugin; +import org.jetbrains.annotations.NotNull; public abstract class AbstractCustomItem implements Listener { @@ -17,7 +17,7 @@ public abstract class AbstractCustomItem implements Listener { private final CustomItem itemId; private final Material material; - public AbstractCustomItem(CustomItemManager customItemManager, CustomItem itemId, Material material) { + public AbstractCustomItem(@NotNull CustomItemManager customItemManager, CustomItem itemId, Material material) { this.plugin = customItemManager.getPlugin(); this.itemId = itemId; @@ -57,7 +57,7 @@ public boolean isItem(ItemStack itemStack) { } } - public void give(@NonNull Player player) { + public void give(@NotNull Player player) { player.getInventory().addItem(getCustomItem(player)); } } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/AbstractCustomMultiblock.java b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/AbstractCustomMultiblock.java index 3df62b43..7de103a0 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/AbstractCustomMultiblock.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/AbstractCustomMultiblock.java @@ -10,6 +10,7 @@ import org.esoteric.minecraft.plugins.template.TemplatePaperPlugin; import org.esoteric.minecraft.plugins.template.util.types.BlockLocation; +import org.jetbrains.annotations.NotNull; public abstract class AbstractCustomMultiblock implements Listener { @@ -25,7 +26,7 @@ public void addMultiblocks(List> addedMultiblocks) { multiblocks.addAll(addedMultiblocks); } - public AbstractCustomMultiblock(CustomMultiblockManager customMultiblockManager, CustomMultiblock multiblockId) { + public AbstractCustomMultiblock(@NotNull CustomMultiblockManager customMultiblockManager, CustomMultiblock multiblockId) { this.plugin = customMultiblockManager.getPlugin(); Bukkit.getPluginManager().registerEvents(this, plugin); diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/CustomMultiblockManager.java b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/CustomMultiblockManager.java index ed5abbf1..b3631aa0 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/CustomMultiblockManager.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/CustomMultiblockManager.java @@ -17,7 +17,7 @@ import com.google.gson.Gson; import org.esoteric.minecraft.plugins.template.TemplatePaperPlugin; -import org.checkerframework.checker.nullness.qual.NonNull; +import org.jetbrains.annotations.NotNull; public class CustomMultiblockManager { @@ -34,7 +34,7 @@ public TemplatePaperPlugin getPlugin() { return plugin; } - public CustomMultiblockManager(TemplatePaperPlugin plugin) { + public CustomMultiblockManager(@NotNull TemplatePaperPlugin plugin) { this.plugin = plugin; multiblocksFilePath = plugin.getDataFolder() + File.separator + multiblocksResourceFilePath; @@ -114,7 +114,7 @@ public void save() { } } - public void loadStoredCustomMultiblocks(@NonNull StoredCustomMultiblocks multiblocks) { + public void loadStoredCustomMultiblocks(@NotNull StoredCustomMultiblocks multiblocks) { for (StoredCustomMultiblock multiblock : multiblocks.getStoredCustomMultiblocks()) { customMultiblockMap.get(multiblock.getMultiblockId()).addMultiblocks(multiblock.getBlockLocations()); } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/StoredCustomMultiblock.java b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/StoredCustomMultiblock.java index 1f56a08c..493cd39e 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/StoredCustomMultiblock.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multiblocks/StoredCustomMultiblock.java @@ -4,7 +4,7 @@ import java.util.Map.Entry; import org.esoteric.minecraft.plugins.template.util.types.BlockLocation; -import org.checkerframework.checker.nullness.qual.NonNull; +import org.jetbrains.annotations.NotNull; public class StoredCustomMultiblock { private final List> blockLocations; @@ -23,7 +23,7 @@ public StoredCustomMultiblock(List> blockLocations, CustomMu this.multiblockId = multiblockId; } - public StoredCustomMultiblock(@NonNull Entry multiblockInfo) { + public StoredCustomMultiblock(@NotNull Entry multiblockInfo) { this(multiblockInfo.getValue().getMultiblocks(), multiblockInfo.getKey()); } } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multientities/AbstractCustomMultientity.java b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multientities/AbstractCustomMultientity.java index 9b455b7f..b1b72765 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/custom/multientities/AbstractCustomMultientity.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/custom/multientities/AbstractCustomMultientity.java @@ -9,6 +9,7 @@ import org.bukkit.persistence.PersistentDataType; import org.esoteric.minecraft.plugins.template.TemplatePaperPlugin; +import org.jetbrains.annotations.NotNull; public abstract class AbstractCustomMultientity implements Listener { @@ -16,7 +17,7 @@ public abstract class AbstractCustomMultientity implements Lis private final CustomMultientity entityId; - protected AbstractCustomMultientity(CustomMultientityManager customMultientityManager, CustomMultientity entityId) { + protected AbstractCustomMultientity(@NotNull CustomMultientityManager customMultientityManager, CustomMultientity entityId) { this.plugin = customMultientityManager.getPlugin(); this.entityId = entityId; diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/data/player/PlayerDataManager.java b/src/main/java/org/esoteric/minecraft/plugins/template/data/player/PlayerDataManager.java index cec88253..7565cd13 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/data/player/PlayerDataManager.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/data/player/PlayerDataManager.java @@ -14,7 +14,7 @@ import com.google.gson.Gson; import org.esoteric.minecraft.plugins.template.TemplatePaperPlugin; -import org.checkerframework.checker.nullness.qual.NonNull; +import org.jetbrains.annotations.NotNull; public class PlayerDataManager { @@ -28,7 +28,7 @@ public class PlayerDataManager { private final Map playerData = new HashMap<>(); - public PlayerDataManager(TemplatePaperPlugin plugin) { + public PlayerDataManager(@NotNull TemplatePaperPlugin plugin) { this.plugin = plugin; playerDataFolderPath = plugin.getDataPath() + File.separator + playerDataFolderName; @@ -121,7 +121,7 @@ public PlayerProfile getPlayerProfile(UUID uuid) { return getPlayerProfile(uuid, true); } - public PlayerProfile getPlayerProfile(@NonNull Player player, boolean createNewProfile) { + public PlayerProfile getPlayerProfile(@NotNull Player player, boolean createNewProfile) { return getPlayerProfile(player.getUniqueId(), createNewProfile); } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/file/FileUtil.java b/src/main/java/org/esoteric/minecraft/plugins/template/file/FileUtil.java index daeed791..87a95419 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/file/FileUtil.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/file/FileUtil.java @@ -14,8 +14,8 @@ import java.security.NoSuchAlgorithmException; import net.lingala.zip4j.ZipFile; -import org.checkerframework.checker.nullness.qual.NonNull; -import org.checkerframework.checker.nullness.qual.Nullable; +import org.jetbrains.annotations.NotNull; +import org.jetbrains.annotations.Nullable; public class FileUtil { @@ -53,7 +53,7 @@ public static List getResourceFileFolderResourceFilePaths(String resourc } } - public static @NonNull List getResourceFileFolderResourceFilePathsRecursively(String resourceFileFolderPath) throws IOException { + public static @NotNull List getResourceFileFolderResourceFilePathsRecursively(String resourceFileFolderPath) throws IOException { List paths = new ArrayList<>(); for (String resourceFilePath : getResourceFileFolderResourceFilePaths(resourceFileFolderPath)) { @@ -68,7 +68,7 @@ public static List getResourceFileFolderResourceFilePaths(String resourc return paths; } - public static void zipFolder(@NonNull File sourceFolder, File zipFile) throws IOException { + public static void zipFolder(@NotNull File sourceFolder, File zipFile) throws IOException { try (ZipFile zipFileInstance = new ZipFile(zipFile)) { for (File file : sourceFolder.listFiles()) { if (file.isDirectory()) { @@ -80,7 +80,7 @@ public static void zipFolder(@NonNull File sourceFolder, File zipFile) throws IO } } - public static boolean isDirectoryRecursivelyEmpty(@NonNull File directory) { + public static boolean isDirectoryRecursivelyEmpty(@NotNull File directory) { if (!directory.isDirectory()) { throw new IllegalArgumentException("The specified path is not a directory"); } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/http/server/HttpServerManager.java b/src/main/java/org/esoteric/minecraft/plugins/template/http/server/HttpServerManager.java index d32bfb68..93bf8f35 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/http/server/HttpServerManager.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/http/server/HttpServerManager.java @@ -17,6 +17,7 @@ import java.net.InetSocketAddress; import org.bukkit.Bukkit; +import org.jetbrains.annotations.NotNull; public class HttpServerManager { @@ -42,7 +43,7 @@ public String getSocketAddress() { return getHostName() + ":" + getPort(); } - public HttpServerManager(TemplatePaperPlugin plugin) { + public HttpServerManager(@NotNull TemplatePaperPlugin plugin) { if (plugin.getResourcePackManager().getResourcePackZipFile() == null) { return; } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/http/server/event/listeners/PlayerJoinListener.java b/src/main/java/org/esoteric/minecraft/plugins/template/http/server/event/listeners/PlayerJoinListener.java index df26b776..1e21787a 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/http/server/event/listeners/PlayerJoinListener.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/http/server/event/listeners/PlayerJoinListener.java @@ -12,19 +12,19 @@ import org.esoteric.minecraft.plugins.template.http.server.HttpServerManager; import net.kyori.adventure.resource.ResourcePackInfo; import net.kyori.adventure.resource.ResourcePackRequest; -import org.checkerframework.checker.nullness.qual.NonNull; +import org.jetbrains.annotations.NotNull; public class PlayerJoinListener implements Listener { private ResourcePackInfo resourcePackInfo; - public PlayerJoinListener(TemplatePaperPlugin plugin, HttpServerManager httpServerManager) { + public PlayerJoinListener(@NotNull TemplatePaperPlugin plugin, @NotNull HttpServerManager httpServerManager) { resourcePackInfo = ResourcePackInfo.resourcePackInfo().hash(FileUtil.getSha1HexString(plugin.getResourcePackManager().getResourcePackZipFile())) .uri(URI.create("http://" + httpServerManager.getSocketAddress() + "/")).build(); } @EventHandler - public void onPlayerJoin(@NonNull PlayerJoinEvent event) { + public void onPlayerJoin(@NotNull PlayerJoinEvent event) { Player player = event.getPlayer(); player.sendResourcePacks(ResourcePackRequest.resourcePackRequest().packs(resourcePackInfo).required(true).build()); } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/language/LanguageManager.java b/src/main/java/org/esoteric/minecraft/plugins/template/language/LanguageManager.java index 54206992..8df63303 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/language/LanguageManager.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/language/LanguageManager.java @@ -18,8 +18,8 @@ import org.bukkit.command.CommandSender; import org.bukkit.configuration.file.YamlConfiguration; import org.bukkit.entity.Player; -import org.checkerframework.checker.nullness.qual.NonNull; -import org.checkerframework.checker.nullness.qual.Nullable; +import org.jetbrains.annotations.NotNull; +import org.jetbrains.annotations.Nullable; public class LanguageManager { @@ -45,7 +45,7 @@ public Set getLanguages() { return languages.keySet(); } - public LanguageManager(TemplatePaperPlugin plugin) { + public LanguageManager(@NotNull TemplatePaperPlugin plugin) { this.plugin = plugin; languagesFolderPath = plugin.getDataPath() + File.separator + languagesFolderName; @@ -104,11 +104,11 @@ public String getLanguage(UUID uuid) { return language; } - public String getLanguage(@NonNull PlayerProfile profile) { + public String getLanguage(@NotNull PlayerProfile profile) { return getLanguage(profile.getUuid()); } - public void setLanguage(@NonNull PlayerProfile profile, String language) { + public void setLanguage(@NotNull PlayerProfile profile, String language) { profile.setLanguage(language); } @@ -117,7 +117,7 @@ public void setLanguage(UUID uuid, String language) { setLanguage(plugin.getPlayerDataManager().getPlayerProfile(uuid), language); } - public void setLanguage(@NonNull Player player, String language) { + public void setLanguage(@NotNull Player player, String language) { setLanguage(player.getUniqueId(), language); } @@ -141,7 +141,7 @@ private String getLocale(UUID uuid) { return getLocale(player); } - private String getLocale(@NonNull PlayerProfile profile) { + private String getLocale(@NotNull PlayerProfile profile) { return getLocale(profile.getUuid()); } @@ -182,7 +182,7 @@ private String getRawMessageString(Message message, String language) { return getRawMessageString(message, language, true); } - private Component getMessage(Message message, String language, boolean fallbackOnDefaultLanguage, Component @NonNull ... arguments) { + private Component getMessage(Message message, String language, boolean fallbackOnDefaultLanguage, Component @NotNull ... arguments) { String miniMessageString = getRawMessageString(message, language, fallbackOnDefaultLanguage); assert miniMessageString != null; @@ -257,19 +257,19 @@ public Component getMessage(Message message, PlayerProfile playerProfile, Object return getMessage(message, playerProfile, true, arguments); } - public void sendMessage(Message message, @NonNull CommandSender commandSender, boolean fallbackOnDefaultLanguage, Component... arguments) { + public void sendMessage(Message message, @NotNull CommandSender commandSender, boolean fallbackOnDefaultLanguage, Component... arguments) { commandSender.sendMessage(getMessage(message, getLanguage(commandSender), fallbackOnDefaultLanguage, arguments)); } - public void sendMessage(Message message, @NonNull CommandSender commandSender, Component... arguments) { + public void sendMessage(Message message, @NotNull CommandSender commandSender, Component... arguments) { commandSender.sendMessage(getMessage(message, commandSender, true, arguments)); } - public void sendMessage(Message message, @NonNull CommandSender commandSender, boolean fallbackOnDefaultLanguage, Object... arguments) { + public void sendMessage(Message message, @NotNull CommandSender commandSender, boolean fallbackOnDefaultLanguage, Object... arguments) { commandSender.sendMessage(getMessage(message, commandSender, fallbackOnDefaultLanguage, toComponents(arguments))); } - public void sendMessage(Message message, @NonNull CommandSender commandSender, Object... arguments) { + public void sendMessage(Message message, @NotNull CommandSender commandSender, Object... arguments) { commandSender.sendMessage(getMessage(message, commandSender, true, arguments)); } @@ -289,23 +289,23 @@ public void sendMessage(Message message, UUID uuid, Object... arguments) { Bukkit.getPlayer(uuid).sendMessage(getMessage(message, uuid, true, arguments)); } - public void sendMessage(Message message, @NonNull PlayerProfile playerProfile, boolean fallbackOnDefaultLanguage, Component... arguments) { + public void sendMessage(Message message, @NotNull PlayerProfile playerProfile, boolean fallbackOnDefaultLanguage, Component... arguments) { Bukkit.getPlayer(playerProfile.getUuid()).sendMessage(getMessage(message, getLanguage(playerProfile), fallbackOnDefaultLanguage, arguments)); } - public void sendMessage(Message message, @NonNull PlayerProfile playerProfile, Component... arguments) { + public void sendMessage(Message message, @NotNull PlayerProfile playerProfile, Component... arguments) { Bukkit.getPlayer(playerProfile.getUuid()).sendMessage(getMessage(message, playerProfile, true, arguments)); } - public void sendMessage(Message message, @NonNull PlayerProfile playerProfile, boolean fallbackOnDefaultLanguage, Object... arguments) { + public void sendMessage(Message message, @NotNull PlayerProfile playerProfile, boolean fallbackOnDefaultLanguage, Object... arguments) { Bukkit.getPlayer(playerProfile.getUuid()).sendMessage(getMessage(message, playerProfile, fallbackOnDefaultLanguage, toComponents(arguments))); } - public void sendMessage(Message message, @NonNull PlayerProfile playerProfile, Object... arguments) { + public void sendMessage(Message message, @NotNull PlayerProfile playerProfile, Object... arguments) { Bukkit.getPlayer(playerProfile.getUuid()).sendMessage(getMessage(message, playerProfile, true, arguments)); } - private Component @NonNull [] toComponents(Object... objects) { + private Component @NotNull [] toComponents(Object... objects) { return Stream.of(objects).map(this::toComponent).toArray(Component[]::new); } diff --git a/src/main/java/org/esoteric/minecraft/plugins/template/util/types/BlockLocation.java b/src/main/java/org/esoteric/minecraft/plugins/template/util/types/BlockLocation.java index 898bbe7b..90c0937c 100644 --- a/src/main/java/org/esoteric/minecraft/plugins/template/util/types/BlockLocation.java +++ b/src/main/java/org/esoteric/minecraft/plugins/template/util/types/BlockLocation.java @@ -2,7 +2,7 @@ import org.bukkit.Location; import org.bukkit.block.Block; -import org.checkerframework.checker.nullness.qual.NonNull; +import org.jetbrains.annotations.NotNull; public class BlockLocation { @@ -35,11 +35,11 @@ public BlockLocation(String worldName, int x, int y, int z) { this.z = z; } - public BlockLocation(@NonNull Location location) { + public BlockLocation(@NotNull Location location) { this(location.getWorld().getName(), (int) location.getX(), (int) location.getY(), (int) location.getZ()); } - public BlockLocation(@NonNull Block block) { + public BlockLocation(@NotNull Block block) { this(block.getLocation()); } }