mirror of
https://github.com/PaperMC/Paper.git
synced 2024-11-22 10:35:38 +01:00
dd941cc40a
* Re-add patches for can-place/can-destroy API This API is all deprecated for removal and has to be replaced as the structure of it has too radically changed * update patches * add link to MaterialRerouting * Deprecation --------- Co-authored-by: Bjarne Koll <lynxplay101@gmail.com>
135 lines
7.8 KiB
Diff
135 lines
7.8 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: Bjarne Koll <lynxplay101@gmail.com>
|
|
Date: Thu, 13 Jun 2024 11:02:36 +0200
|
|
Subject: [PATCH] Adopt MaterialRerouting
|
|
|
|
Adopts the paper-api to the material rerouting infrastructure introduced
|
|
by upstream.
|
|
|
|
diff --git a/src/main/java/org/bukkit/craftbukkit/legacy/MaterialRerouting.java b/src/main/java/org/bukkit/craftbukkit/legacy/MaterialRerouting.java
|
|
index 3ff0f0e34356cee4c510fdd60af723b1c5df156a..6cc9d7a9e6d4bfdc27e52fc581b2bb832616f121 100644
|
|
--- a/src/main/java/org/bukkit/craftbukkit/legacy/MaterialRerouting.java
|
|
+++ b/src/main/java/org/bukkit/craftbukkit/legacy/MaterialRerouting.java
|
|
@@ -600,4 +600,82 @@ public class MaterialRerouting {
|
|
public static void setBlocks(ToolComponent.ToolRule toolRule, Collection<Material> blocks) {
|
|
toolRule.setBlocks(blocks.stream().map(MaterialRerouting::transformToBlockType).toList());
|
|
}
|
|
+
|
|
+ // Paper start - register paper API specific material consumers in rerouting
|
|
+ // A lot of these methods do *not* run through MaterialRerouting to avoid the overhead of a system that
|
|
+ // currently is an effective noop.
|
|
+ // The only downside is that upstream moved the handling of legacy materials into transformFromXType methods.
|
|
+ // As such, methods introduced prior to 1.13 need to run through the transformation to make sure legacy material
|
|
+ // constants still work.
|
|
+
|
|
+ // Utility method for constructing a set from an existing one after mapping each element.
|
|
+ private static <I, O> Set<O> mapSet(final Set<I> input, final java.util.function.Function<I,O> mapper) {
|
|
+ final Set<O> output = new it.unimi.dsi.fastutil.objects.ObjectOpenHashSet<>(input.size());
|
|
+ for (final I i : input) {
|
|
+ output.add(mapper.apply(i));
|
|
+ }
|
|
+ return output;
|
|
+ }
|
|
+
|
|
+ // Method added post-1.13, noop (https://github.com/PaperMC/Paper/pull/4965)
|
|
+ public static org.bukkit.Material getMinecartMaterial(org.bukkit.entity.Minecart minecart, @InjectPluginVersion ApiVersion version) {
|
|
+ return minecart.getMinecartMaterial();
|
|
+ }
|
|
+
|
|
+ // Method added post-1.13, noop (https://github.com/PaperMC/Paper/pull/4965)
|
|
+ public static Material getBoatMaterial(Boat boat, @InjectPluginVersion ApiVersion version) {
|
|
+ return boat.getBoatMaterial();
|
|
+ }
|
|
+
|
|
+ // Method added post-1.13, noop (https://github.com/PaperMC/Paper/pull/3807)
|
|
+ public static Material getType(io.papermc.paper.event.player.PlayerItemCooldownEvent event, @InjectPluginVersion ApiVersion version) {
|
|
+ return event.getType();
|
|
+ }
|
|
+
|
|
+ // Method added post-1.13, noop (https://github.com/PaperMC/Paper/pull/3850)
|
|
+ public static Collection<Material> getInfiniburn(World world, @InjectPluginVersion ApiVersion version) {
|
|
+ return world.getInfiniburn();
|
|
+ }
|
|
+
|
|
+ // Method added pre-1.13, needs legacy rerouting (https://github.com/PaperMC/Paper/commit/3438e96192)
|
|
+ public static Set<Material> getTypes(
|
|
+ final com.destroystokyo.paper.event.player.PlayerArmorChangeEvent.SlotType slotType,
|
|
+ @InjectPluginVersion final ApiVersion apiVersion
|
|
+ ) {
|
|
+ if (apiVersion.isNewerThanOrSameAs(ApiVersion.FLATTENING)) return slotType.getTypes();
|
|
+ else return mapSet(slotType.getTypes(), MaterialRerouting::transformToItemType); // Needed as pre-flattening is hanled by transformToItemType
|
|
+ }
|
|
+
|
|
+ // Method added pre-1.13, needs legacy rerouting (https://github.com/PaperMC/Paper/commit/3438e96192)
|
|
+ @RerouteStatic("com/destroystokyo/paper/event/player/PlayerArmorChangeEvent$SlotType")
|
|
+ public static com.destroystokyo.paper.event.player.PlayerArmorChangeEvent.SlotType getByMaterial(
|
|
+ final Material material
|
|
+ ) {
|
|
+ return com.destroystokyo.paper.event.player.PlayerArmorChangeEvent.SlotType.getByMaterial(MaterialRerouting.transformToItemType(material));
|
|
+ }
|
|
+
|
|
+ // Method added pre-1.13, needs legacy rerouting (https://github.com/PaperMC/Paper/commit/3438e96192)
|
|
+ @RerouteStatic("com/destroystokyo/paper/event/player/PlayerArmorChangeEvent$SlotType")
|
|
+ public static boolean isEquipable(final Material material) {
|
|
+ return com.destroystokyo.paper.event.player.PlayerArmorChangeEvent.SlotType.isEquipable(MaterialRerouting.transformToItemType(material));
|
|
+ }
|
|
+
|
|
+ // Method added post 1.13, no-op (https://github.com/PaperMC/Paper/pull/1244)
|
|
+ public static Material getMaterial(final com.destroystokyo.paper.event.block.AnvilDamagedEvent.DamageState damageState) {
|
|
+ return damageState.getMaterial();
|
|
+ }
|
|
+
|
|
+ // Method added post 1.13, no-op (https://github.com/PaperMC/Paper/pull/1244)
|
|
+ @RerouteStatic("com/destroystokyo/paper/event/block/AnvilDamagedEvent$DamageState")
|
|
+ public static com.destroystokyo.paper.event.block.AnvilDamagedEvent.DamageState getState(
|
|
+ final Material material
|
|
+ ) {
|
|
+ return com.destroystokyo.paper.event.block.AnvilDamagedEvent.DamageState.getState(material);
|
|
+ }
|
|
+
|
|
+ // Method added post 1.13, no-op (https://github.com/PaperMC/Paper/pull/10290)
|
|
+ public static ItemStack withType(final ItemStack itemStack, final Material material) {
|
|
+ return itemStack.withType(material);
|
|
+ }
|
|
+ // Paper end - register paper API specific material consumers in rerouting
|
|
}
|
|
diff --git a/src/test/java/org/bukkit/craftbukkit/legacy/MaterialReroutingTest.java b/src/test/java/org/bukkit/craftbukkit/legacy/MaterialReroutingTest.java
|
|
index a5f3d562afec242912589bfc053ff91ede77347e..0fac826b9367a821c6801190997592219cb47f73 100644
|
|
--- a/src/test/java/org/bukkit/craftbukkit/legacy/MaterialReroutingTest.java
|
|
+++ b/src/test/java/org/bukkit/craftbukkit/legacy/MaterialReroutingTest.java
|
|
@@ -56,6 +56,9 @@ public class MaterialReroutingTest extends AbstractTestingBase {
|
|
.filter(entry -> !entry.getName().endsWith("ItemType.class"))
|
|
.filter(entry -> !entry.getName().endsWith("Registry.class"))
|
|
.filter(entry -> !entry.getName().startsWith("org/bukkit/material"))
|
|
+ // Paper start - types that cannot be translated to ItemType/BlockType
|
|
+ .filter(entry -> !entry.getName().equals("com/destroystokyo/paper/MaterialSetTag.class"))
|
|
+ // Paper end - types that cannot be translated to ItemType/BlockType
|
|
.map(entry -> {
|
|
try {
|
|
return MaterialReroutingTest.jarFile.getInputStream(entry);
|
|
@@ -93,6 +96,10 @@ public class MaterialReroutingTest extends AbstractTestingBase {
|
|
continue;
|
|
}
|
|
}
|
|
+ // Paper start - filter out more methods from rerouting test
|
|
+ if (methodNode.name.startsWith("lambda$")) continue;
|
|
+ if (isInternal(methodNode.invisibleAnnotations)) continue;
|
|
+ // Paper end - filter out more methods from rerouting test
|
|
|
|
if (!Commodore.rerouteMethods(Collections.emptySet(), ApiVersion.CURRENT, MaterialReroutingTest.MATERIAL_METHOD_REROUTE, (methodNode.access & Opcodes.ACC_STATIC) != 0, classNode.name, methodNode.name, methodNode.desc, a -> { })) {
|
|
missingReroute.add(methodNode.name + " " + methodNode.desc + " " + methodNode.signature);
|
|
@@ -109,6 +116,13 @@ public class MaterialReroutingTest extends AbstractTestingBase {
|
|
}
|
|
}
|
|
|
|
+ // Paper start - filter out more methods from rerouting test
|
|
+ private static boolean isInternal(final List<org.objectweb.asm.tree.AnnotationNode> annotationNodes) {
|
|
+ return annotationNodes != null
|
|
+ && annotationNodes.stream().anyMatch(a -> a.desc.equals("Lorg/jetbrains/annotations/ApiStatus$Internal;"));
|
|
+ }
|
|
+ // Paper end - filter out more methods from rerouting test
|
|
+
|
|
@AfterAll
|
|
public static void clear() throws IOException {
|
|
if (MaterialReroutingTest.jarFile != null) {
|