refactor: replace commons-lang Validate with Guava Preconditions (#3586)

Migrated all usages of org.apache.commons.lang.Validate to com.google.common.base.Preconditions as discussed in issue #3586. This removes the dependency on commons-lang for validation checks.

Author: ayethor

src/main/java/io/github/thebusybiscuit/slimefun4/api/MinecraftVersion.java

@@ -2,7 +2,7 @@
 
 import javax.annotation.Nonnull;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.Server;
 
 import io.github.thebusybiscuit.slimefun4.implementation.Slimefun;
@@ -233,7 +233,7 @@ public boolean isMinecraftVersion(int minecraftVersion, int patchVersion) {
      * @return Whether this {@link MinecraftVersion} is newer or equal to the given {@link MinecraftVersion}
      */
     public boolean isAtLeast(@Nonnull MinecraftVersion version) {
-        Validate.notNull(version, "A Minecraft version cannot be null!");
+        Preconditions.checkNotNull(version, "A Minecraft version cannot be null!");
 
         if (this == UNKNOWN) {
             return false;
@@ -268,7 +268,7 @@ public boolean isAtLeast(@Nonnull MinecraftVersion version) {
      * @return Whether this {@link MinecraftVersion} is older than the given one
      */
     public boolean isBefore(@Nonnull MinecraftVersion version) {
-        Validate.notNull(version, "A Minecraft version cannot be null!");
+        Preconditions.checkNotNull(version, "A Minecraft version cannot be null!");
 
         if (this == UNKNOWN) {
             return true;

src/main/java/io/github/thebusybiscuit/slimefun4/api/SlimefunAddon.java

@@ -5,7 +5,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.java.JavaPlugin;
@@ -86,7 +86,7 @@ public interface SlimefunAddon {
      * @return Whether this {@link SlimefunAddon} depends on the given {@link Plugin}
      */
     default boolean hasDependency(@Nonnull String dependency) {
-        Validate.notNull(dependency, "The dependency cannot be null");
+        Preconditions.checkNotNull(dependency, "The dependency cannot be null");
 
         // Well... it cannot depend on itself but you get the idea.
         if (getJavaPlugin().getName().equalsIgnoreCase(dependency)) {

src/main/java/io/github/thebusybiscuit/slimefun4/api/SlimefunBranch.java

@@ -2,7 +2,7 @@
 
 import javax.annotation.Nonnull;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 
 import io.github.bakedlibs.dough.common.CommonPatterns;
 
@@ -45,7 +45,7 @@ public enum SlimefunBranch {
     private final boolean official;
 
     SlimefunBranch(@Nonnull String name, boolean official) {
-        Validate.notNull(name, "The branch name cannot be null");
+        Preconditions.checkNotNull(name, "The branch name cannot be null");
 
         this.name = name;
         this.official = official;

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/AsyncAutoEnchanterProcessEvent.java

@@ -2,7 +2,7 @@
 
 import javax.annotation.Nonnull;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
@@ -31,9 +31,9 @@ public class AsyncAutoEnchanterProcessEvent extends Event implements Cancellable
     public AsyncAutoEnchanterProcessEvent(@Nonnull ItemStack item, @Nonnull ItemStack enchantedBook, @Nonnull BlockMenu menu) {
         super(true);
 
-        Validate.notNull(item, "The item to enchant cannot be null!");
-        Validate.notNull(enchantedBook, "The enchanted book to enchant cannot be null!");
-        Validate.notNull(menu, "The menu of auto-enchanter cannot be null!");
+        Preconditions.checkNotNull(item, "The item to enchant cannot be null!");
+        Preconditions.checkNotNull(enchantedBook, "The enchanted book to enchant cannot be null!");
+        Preconditions.checkNotNull(menu, "The menu of auto-enchanter cannot be null!");
 
         this.item = item;
         this.enchantedBook = enchantedBook;

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/AsyncProfileLoadEvent.java

@@ -4,7 +4,7 @@
 
 import javax.annotation.Nonnull;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
@@ -32,7 +32,7 @@ public class AsyncProfileLoadEvent extends Event {
     public AsyncProfileLoadEvent(@Nonnull PlayerProfile profile) {
         super(true);
 
-        Validate.notNull(profile, "The Profile cannot be null");
+        Preconditions.checkNotNull(profile, "The Profile cannot be null");
 
         this.uniqueId = profile.getUUID();
         this.profile = profile;
@@ -56,8 +56,8 @@ public PlayerProfile getProfile() {
      *            The {@link PlayerProfile}
      */
     public void setProfile(@Nonnull PlayerProfile profile) {
-        Validate.notNull(profile, "The PlayerProfile cannot be null!");
-        Validate.isTrue(profile.getUUID().equals(uniqueId), "Cannot inject a PlayerProfile with a different UUID");
+        Preconditions.checkNotNull(profile, "The PlayerProfile cannot be null!");
+        Preconditions.checkArgument(profile.getUUID().equals(uniqueId), "Cannot inject a PlayerProfile with a different UUID");
 
         this.profile = profile;
     }

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/BlockPlacerPlaceEvent.java

@@ -3,7 +3,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.ParametersAreNonnullByDefault;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.block.Block;
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
@@ -75,7 +75,7 @@ public ItemStack getItemStack() {
      *            The {@link ItemStack} to be placed
      */
     public void setItemStack(@Nonnull ItemStack item) {
-        Validate.notNull(item, "The ItemStack must not be null!");
+        Preconditions.checkNotNull(item, "The ItemStack must not be null!");
 
         if (!locked) {
             this.placedItem = item;

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/ClimbingPickLaunchEvent.java

@@ -3,7 +3,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.ParametersAreNonnullByDefault;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.block.Block;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Cancellable;
@@ -63,7 +63,7 @@ public Vector getVelocity() {
      *            The {@link Vector} velocity to apply
      */
     public void setVelocity(@Nonnull Vector velocity) {
-        Validate.notNull(velocity);
+        Preconditions.checkNotNull(velocity);
         this.velocity = velocity;
     }
 

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/CoolerFeedPlayerEvent.java

@@ -3,7 +3,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.ParametersAreNonnullByDefault;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
@@ -84,8 +84,8 @@ public ItemStack getConsumedItem() {
      *            The new {@link ItemStack}
      */
     public void setConsumedItem(@Nonnull ItemStack item) {
-        Validate.notNull(item, "The consumed Item cannot be null!");
-        Validate.isTrue(item.getItemMeta() instanceof PotionMeta, "The item must be a potion!");
+        Preconditions.checkNotNull(item, "The consumed Item cannot be null!");
+        Preconditions.checkArgument(item.getItemMeta() instanceof PotionMeta, "The item must be a potion!");
 
         this.consumedItem = item;
     }

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/ExplosiveToolBreakBlocksEvent.java

@@ -5,7 +5,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.ParametersAreNonnullByDefault;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.block.Block;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Cancellable;
@@ -38,10 +38,10 @@ public class ExplosiveToolBreakBlocksEvent extends PlayerEvent implements Cancel
     public ExplosiveToolBreakBlocksEvent(Player player, Block block, List<Block> blocks, ItemStack item, ExplosiveTool explosiveTool) {
         super(player);
 
-        Validate.notNull(block, "The center block cannot be null!");
-        Validate.notNull(blocks, "Blocks cannot be null");
-        Validate.notNull(item, "Item cannot be null");
-        Validate.notNull(explosiveTool, "ExplosiveTool cannot be null");
+        Preconditions.checkNotNull(block, "The center block cannot be null!");
+        Preconditions.checkNotNull(blocks, "Blocks cannot be null");
+        Preconditions.checkNotNull(item, "Item cannot be null");
+        Preconditions.checkNotNull(explosiveTool, "ExplosiveTool cannot be null");
 
         this.mainBlock = block;
         this.additionalBlocks = blocks;

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/PlayerPreResearchEvent.java

@@ -3,7 +3,7 @@
 import javax.annotation.Nonnull;
 import javax.annotation.ParametersAreNonnullByDefault;
 
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
@@ -35,9 +35,9 @@ public class PlayerPreResearchEvent extends Event implements Cancellable {
 
     @ParametersAreNonnullByDefault
     public PlayerPreResearchEvent(Player p, Research research, SlimefunItem slimefunItem) {
-        Validate.notNull(p, "The Player cannot be null");
-        Validate.notNull(research, "Research cannot be null");
-        Validate.notNull(slimefunItem, "SlimefunItem cannot be null");
+        Preconditions.checkNotNull(p, "The Player cannot be null");
+        Preconditions.checkNotNull(research, "Research cannot be null");
+        Preconditions.checkNotNull(slimefunItem, "SlimefunItem cannot be null");
 
         this.player = p;
         this.research = research;