From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 From: Jake Potrebic Date: Tue, 4 Jul 2023 11:27:10 -0700 Subject: [PATCH] Fix BanList API diff --git a/src/main/java/org/bukkit/craftbukkit/CraftOfflinePlayer.java b/src/main/java/org/bukkit/craftbukkit/CraftOfflinePlayer.java index a0fcd11e6b0ca2a7055a4d1910124b20bd9c0b94..9daec0782774ab51ea8091cb8ed9d0a106e34cfa 100644 --- a/src/main/java/org/bukkit/craftbukkit/CraftOfflinePlayer.java +++ b/src/main/java/org/bukkit/craftbukkit/CraftOfflinePlayer.java @@ -114,17 +114,17 @@ public class CraftOfflinePlayer implements OfflinePlayer, ConfigurationSerializa } @Override - public BanEntry ban(String reason, Date expires, String source) { + public BanEntry ban(String reason, Date expires, String source) { // Paper - fix ban list API return ((ProfileBanList) this.server.getBanList(BanList.Type.PROFILE)).addBan(this.getPlayerProfile(), reason, expires, source); } @Override - public BanEntry ban(String reason, Instant expires, String source) { + public BanEntry ban(String reason, Instant expires, String source) { // Paper - fix ban list API return ((ProfileBanList) this.server.getBanList(BanList.Type.PROFILE)).addBan(this.getPlayerProfile(), reason, expires, source); } @Override - public BanEntry ban(String reason, Duration duration, String source) { + public BanEntry ban(String reason, Duration duration, String source) { // Paper - fix ban list API return ((ProfileBanList) this.server.getBanList(BanList.Type.PROFILE)).addBan(this.getPlayerProfile(), reason, duration, source); } diff --git a/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanEntry.java b/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanEntry.java index 13e5e44b069121e51b9486c445902937f1d6c6d8..4a37c8172b42b10472bb90c9310c7ae3eeaa3481 100644 --- a/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanEntry.java +++ b/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanEntry.java @@ -9,7 +9,7 @@ import org.bukkit.BanEntry; import org.bukkit.craftbukkit.profile.CraftPlayerProfile; import org.bukkit.profile.PlayerProfile; -public final class CraftProfileBanEntry implements BanEntry { +public final class CraftProfileBanEntry implements BanEntry { // Paper private static final Date minorDate = Date.from(Instant.parse("1899-12-31T04:00:00Z")); private final UserBanList list; private final GameProfile profile; @@ -33,8 +33,8 @@ public final class CraftProfileBanEntry implements BanEntry { } @Override - public PlayerProfile getBanTarget() { - return new CraftPlayerProfile(this.profile); + public com.destroystokyo.paper.profile.PlayerProfile getBanTarget() { // Paper + return new com.destroystokyo.paper.profile.CraftPlayerProfile(this.profile); // Paper } @Override diff --git a/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanList.java b/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanList.java index 172202accf4448a933fcf1ff820316c7910dd7f7..50ee7656580d386db473c054f5c5ec57bb2b1424 100644 --- a/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanList.java +++ b/src/main/java/org/bukkit/craftbukkit/ban/CraftProfileBanList.java @@ -24,42 +24,80 @@ public class CraftProfileBanList implements ProfileBanList { } @Override - public BanEntry getBanEntry(String target) { + public BanEntry getBanEntry(String target) { // Paper Preconditions.checkArgument(target != null, "Target cannot be null"); return this.getBanEntry(CraftProfileBanList.getProfile(target)); } @Override - public BanEntry getBanEntry(PlayerProfile target) { + public BanEntry getBanEntry(PlayerProfile target) { // Paper Preconditions.checkArgument(target != null, "Target cannot be null"); - return this.getBanEntry(((CraftPlayerProfile) target).buildGameProfile()); + return this.getBanEntry(((com.destroystokyo.paper.profile.SharedPlayerProfile) target).buildGameProfile()); // Paper + } + // Paper start - fix ban list API + @Override + public BanEntry getBanEntry(final com.destroystokyo.paper.profile.PlayerProfile target) { + Preconditions.checkArgument(target != null, "target cannot be null"); + + return this.getBanEntry(((com.destroystokyo.paper.profile.SharedPlayerProfile) target).buildGameProfile()); + } + + @Override + public BanEntry addBan(final com.destroystokyo.paper.profile.PlayerProfile target, final String reason, final Date expires, final String source) { + Preconditions.checkArgument(target != null, "PlayerProfile cannot be null"); + Preconditions.checkArgument(target.getId() != null, "The PlayerProfile UUID cannot be null"); + + return this.addBan(((com.destroystokyo.paper.profile.SharedPlayerProfile) target).buildGameProfile(), reason, expires, source); + } + + @Override + public boolean isBanned(final com.destroystokyo.paper.profile.PlayerProfile target) { + return this.isBanned((com.destroystokyo.paper.profile.SharedPlayerProfile) target); + } + + @Override + public void pardon(final com.destroystokyo.paper.profile.PlayerProfile target) { + this.pardon((com.destroystokyo.paper.profile.SharedPlayerProfile) target); } @Override - public BanEntry addBan(String target, String reason, Date expires, String source) { + public BanEntry addBan(final com.destroystokyo.paper.profile.PlayerProfile target, final String reason, final Instant expires, final String source) { + Date date = expires != null ? Date.from(expires) : null; + return this.addBan(target, reason, date, source); + } + + @Override + public BanEntry addBan(final com.destroystokyo.paper.profile.PlayerProfile target, final String reason, final Duration duration, final String source) { + Instant instant = duration != null ? Instant.now().plus(duration) : null; + return this.addBan(target, reason, instant, source); + } + // Paper end - fix ban list API + + @Override + public BanEntry addBan(String target, String reason, Date expires, String source) { // Paper - fix ban list API Preconditions.checkArgument(target != null, "Ban target cannot be null"); return this.addBan(CraftProfileBanList.getProfileByName(target), reason, expires, source); } @Override - public BanEntry addBan(PlayerProfile target, String reason, Date expires, String source) { + public BanEntry addBan(PlayerProfile target, String reason, Date expires, String source) { // Paper - fix ban list API Preconditions.checkArgument(target != null, "PlayerProfile cannot be null"); Preconditions.checkArgument(target.getUniqueId() != null, "The PlayerProfile UUID cannot be null"); - return this.addBan(((CraftPlayerProfile) target).buildGameProfile(), reason, expires, source); + return this.addBan(((com.destroystokyo.paper.profile.SharedPlayerProfile) target).buildGameProfile(), reason, expires, source); // Paper } @Override - public BanEntry addBan(PlayerProfile target, String reason, Instant expires, String source) { + public BanEntry addBan(PlayerProfile target, String reason, Instant expires, String source) { // Paper - fix ban list API Date date = expires != null ? Date.from(expires) : null; return this.addBan(target, reason, date, source); } @Override - public BanEntry addBan(PlayerProfile target, String reason, Duration duration, String source) { + public BanEntry addBan(PlayerProfile target, String reason, Duration duration, String source) { // Paper - fix ban list API Instant instant = duration != null ? Instant.now().plus(duration) : null; return this.addBan(target, reason, instant, source); } @@ -76,8 +114,8 @@ public class CraftProfileBanList implements ProfileBanList { } @Override - public Set> getEntries() { - ImmutableSet.Builder> builder = ImmutableSet.builder(); + public Set> getEntries() { // Paper + ImmutableSet.Builder> builder = ImmutableSet.builder(); // Paper for (UserBanListEntry entry : this.list.getEntries()) { GameProfile profile = entry.getUser(); builder.add(new CraftProfileBanEntry(profile, entry, this.list)); @@ -88,9 +126,14 @@ public class CraftProfileBanList implements ProfileBanList { @Override public boolean isBanned(PlayerProfile target) { + // Paper start + return this.isBanned((com.destroystokyo.paper.profile.SharedPlayerProfile) target); + } + private boolean isBanned(com.destroystokyo.paper.profile.SharedPlayerProfile target) { + // Paper end Preconditions.checkArgument(target != null, "Target cannot be null"); - return this.isBanned(((CraftPlayerProfile) target).buildGameProfile()); + return this.isBanned(target.buildGameProfile()); // Paper } @Override @@ -102,9 +145,14 @@ public class CraftProfileBanList implements ProfileBanList { @Override public void pardon(PlayerProfile target) { + // Paper start + this.pardon((com.destroystokyo.paper.profile.SharedPlayerProfile) target); + } + private void pardon(com.destroystokyo.paper.profile.SharedPlayerProfile target) { + // Paper end Preconditions.checkArgument(target != null, "Target cannot be null"); - this.pardon(((CraftPlayerProfile) target).buildGameProfile()); + this.pardon(target.buildGameProfile()); // Paper } @Override @@ -114,7 +162,7 @@ public class CraftProfileBanList implements ProfileBanList { this.pardon(CraftProfileBanList.getProfile(target)); } - public BanEntry getBanEntry(GameProfile profile) { + public BanEntry getBanEntry(GameProfile profile) { // Paper if (profile == null) { return null; } @@ -127,7 +175,7 @@ public class CraftProfileBanList implements ProfileBanList { return new CraftProfileBanEntry(profile, entry, this.list); } - public BanEntry addBan(GameProfile profile, String reason, Date expires, String source) { + public BanEntry addBan(GameProfile profile, String reason, Date expires, String source) { // Paper if (profile == null) { return null; } diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java index 2b20ddfbd599d157f3a3658957407040c992e68d..c1cc6f615e90a6dcb071b3e731cfaa76d2d2ae23 100644 --- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java +++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java @@ -1653,23 +1653,23 @@ public class CraftPlayer extends CraftHumanEntity implements Player { } @Override - public BanEntry ban(String reason, Date expires, String source) { + public BanEntry ban(String reason, Date expires, String source) { // Paper - fix ban list API return this.ban(reason, expires, source, true); } @Override - public BanEntry ban(String reason, Instant expires, String source) { + public BanEntry ban(String reason, Instant expires, String source) { // Paper - fix ban list API return this.ban(reason, expires != null ? Date.from(expires) : null, source); } @Override - public BanEntry ban(String reason, Duration duration, String source) { + public BanEntry ban(String reason, Duration duration, String source) { // Paper - fix ban list API return this.ban(reason, duration != null ? Instant.now().plus(duration) : null, source); } @Override - public BanEntry ban(String reason, Date expires, String source, boolean kickPlayer) { - BanEntry banEntry = ((ProfileBanList) this.server.getBanList(BanList.Type.PROFILE)).addBan(this.getPlayerProfile(), reason, expires, source); + public BanEntry ban(String reason, Date expires, String source, boolean kickPlayer) { // Paper - fix ban list API + BanEntry banEntry = ((ProfileBanList) this.server.getBanList(BanList.Type.PROFILE)).addBan(this.getPlayerProfile(), reason, expires, source); // Paper - fix ban list API if (kickPlayer) { this.kickPlayer(reason); } @@ -1677,12 +1677,12 @@ public class CraftPlayer extends CraftHumanEntity implements Player { } @Override - public BanEntry ban(String reason, Instant instant, String source, boolean kickPlayer) { + public BanEntry ban(String reason, Instant instant, String source, boolean kickPlayer) { // Paper - fix ban list API return this.ban(reason, instant != null ? Date.from(instant) : null, source, kickPlayer); } @Override - public BanEntry ban(String reason, Duration duration, String source, boolean kickPlayer) { + public BanEntry ban(String reason, Duration duration, String source, boolean kickPlayer) { // Paper - fix ban list API return this.ban(reason, duration != null ? Instant.now().plus(duration) : null, source, kickPlayer); } diff --git a/src/main/java/org/bukkit/craftbukkit/profile/CraftPlayerProfile.java b/src/main/java/org/bukkit/craftbukkit/profile/CraftPlayerProfile.java index 6422c58907ee289359a11054fec1e4de6f495ae3..bc6151cfeb7ace4755414614723ee830081094af 100644 --- a/src/main/java/org/bukkit/craftbukkit/profile/CraftPlayerProfile.java +++ b/src/main/java/org/bukkit/craftbukkit/profile/CraftPlayerProfile.java @@ -28,7 +28,7 @@ import org.bukkit.profile.PlayerProfile; import org.bukkit.profile.PlayerTextures; @SerializableAs("PlayerProfile") -public final class CraftPlayerProfile implements PlayerProfile, com.destroystokyo.paper.profile.SharedPlayerProfile { // Paper +public final class CraftPlayerProfile implements PlayerProfile, com.destroystokyo.paper.profile.SharedPlayerProfile, com.destroystokyo.paper.profile.PlayerProfile { // Paper @Nonnull public static GameProfile validateSkullProfile(@Nonnull GameProfile gameProfile) { @@ -123,7 +123,7 @@ public final class CraftPlayerProfile implements PlayerProfile, com.destroystoky } @Override - public CompletableFuture update() { + public CompletableFuture update() { // Paper - have to remove generic to avoid clashing between bukkit.PlayerProfile and paper.PlayerProfile return CompletableFuture.supplyAsync(this::getUpdatedProfile, Util.PROFILE_EXECUTOR); // Paper - not a good idea to use BLOCKING OPERATIONS on the worldgen executor } @@ -277,4 +277,71 @@ public final class CraftPlayerProfile implements PlayerProfile, com.destroystoky // Paper - diff on change return profile; } + + // Paper start - This must implement our PlayerProfile so generic casts succeed from cb.CraftPlayerProfile to paper.PlayerProfile + // The methods don't actually have to be implemented, because the profile should immediately be cast to SharedPlayerProfile + @Override + public String setName(final String name) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public UUID getId() { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public UUID setId(final UUID uuid) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public java.util.Set getProperties() { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public boolean hasProperty(final String property) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public void setProperty(final com.destroystokyo.paper.profile.ProfileProperty property) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public void setProperties(final java.util.Collection properties) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public void clearProperties() { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public boolean completeFromCache() { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public boolean completeFromCache(final boolean onlineMode) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public boolean completeFromCache(final boolean lookupUUID, final boolean onlineMode) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public boolean complete(final boolean textures) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } + + @Override + public boolean complete(final boolean textures, final boolean onlineMode) { + throw new UnsupportedOperationException("Do not cast to com.destroystokyo.paper.profile.PlayerProfile"); + } }