From 100861484a20394ef1d93ec3734ec6772e62890b Mon Sep 17 00:00:00 2001
From: dordsor21 <dordsor21@gmail.com>
Date: Sat, 18 Apr 2020 17:22:51 +0100
Subject: [PATCH] Fix plot claim.

---
 .../java/com/plotsquared/core/command/Claim.java     |  2 +-
 .../main/java/com/plotsquared/core/plot/Plot.java    | 12 +++++-------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/Core/src/main/java/com/plotsquared/core/command/Claim.java b/Core/src/main/java/com/plotsquared/core/command/Claim.java
index f6208bb38..b061e1fb6 100644
--- a/Core/src/main/java/com/plotsquared/core/command/Claim.java
+++ b/Core/src/main/java/com/plotsquared/core/command/Claim.java
@@ -132,7 +132,7 @@ public class Claim extends SubCommand {
         final String finalSchematic = schematic;
         DBFunc.createPlotSafe(plot, () -> TaskManager.IMP.sync(new RunnableVal<Object>() {
             @Override public void run(Object value) {
-                if (!plot.claim(player, true, finalSchematic)) {
+                if (!plot.claim(player, true, finalSchematic, false)) {
                     PlotSquared.get().getLogger().log(Captions.PREFIX.getTranslated() +
                         String.format("Failed to claim plot %s", plot.getId().toCommaSeparatedString()));
                     sendMessage(player, Captions.PLOT_NOT_CLAIMED);
diff --git a/Core/src/main/java/com/plotsquared/core/plot/Plot.java b/Core/src/main/java/com/plotsquared/core/plot/Plot.java
index fb50c6bbb..a79b6fb7d 100644
--- a/Core/src/main/java/com/plotsquared/core/plot/Plot.java
+++ b/Core/src/main/java/com/plotsquared/core/plot/Plot.java
@@ -1752,7 +1752,7 @@ public class Plot {
         return this.create(this.owner, true);
     }
 
-    public boolean claim(final PlotPlayer player, boolean teleport, String schematic) {
+    public boolean claim(@NotNull final PlotPlayer player, boolean teleport, String schematic) {
         if (!canClaim(player)) {
             PlotSquared.debug(Captions.PREFIX.getTranslated() + String
                 .format("Player %s attempted to claim plot %s, but was not allowed",
@@ -1762,7 +1762,7 @@ public class Plot {
         return claim(player, teleport, schematic, true);
     }
 
-    public boolean claim(final PlotPlayer player, boolean teleport, String schematic,
+    public boolean claim(@NotNull final PlotPlayer player, boolean teleport, String schematic,
         boolean updateDB) {
 
         if (updateDB) {
@@ -2390,9 +2390,9 @@ public class Plot {
      * @param player the claiming player
      * @return
      */
-    public boolean canClaim(@Nullable PlotPlayer player) {
+    public boolean canClaim(@NotNull PlotPlayer player) {
         PlotCluster cluster = this.getCluster();
-        if (cluster != null && player != null) {
+        if (cluster != null) {
             if (!cluster.isAdded(player.getUUID()) && !Permissions
                 .hasPermission(player, "plots.admin.command.claim")) {
                 return false;
@@ -2400,9 +2400,7 @@ public class Plot {
         }
         final UUID owner = this.guessOwner();
         if (owner != null) {
-            if (player == null || !player.getUUID().equals(owner)) {
-                return false;
-            }
+            return false;
         }
         return !isMerged();
     }