Move default canDisableShield impl to AxeItem

Author: Virtuoel

patchwork-extensions/src/main/java/com/patchworkmc/mixin/extension/MixinAxeItem.java

@@ -0,0 +1,35 @@
+/*
+ * Minecraft Forge, Patchwork Project
+ * Copyright (c) 2016-2020, 2019-2020
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation version 2.1
+ * of the License.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+package com.patchworkmc.mixin.extension;
+
+import org.spongepowered.asm.mixin.Mixin;
+import net.minecraftforge.common.extensions.IForgeItem;
+
+import net.minecraft.entity.LivingEntity;
+import net.minecraft.item.AxeItem;
+import net.minecraft.item.ItemStack;
+
+@Mixin(AxeItem.class)
+public abstract class MixinAxeItem implements IForgeItem {
+	@Override
+	public boolean canDisableShield(ItemStack stack, ItemStack shield, LivingEntity entity, LivingEntity attacker) {
+		return true;
+	}
+}

patchwork-extensions/src/main/java/net/minecraftforge/common/extensions/IForgeItem.java

@@ -43,7 +43,6 @@
 import net.minecraft.entity.attribute.EntityAttributeModifier;
 import net.minecraft.entity.mob.MobEntity;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.AxeItem;
 import net.minecraft.item.Item;
 import net.minecraft.item.ItemGroup;
 import net.minecraft.item.ItemPropertyGetter;
@@ -726,7 +725,7 @@ default String getCreatorModId(ItemStack itemStack) {
 	 * @return True if this ItemStack can disable the shield in question.
 	 */
 	default boolean canDisableShield(ItemStack stack, ItemStack shield, LivingEntity entity, LivingEntity attacker) {
-		return this instanceof AxeItem;
+		return false;
 	}
 
 	// TODO: Call locations: Patches: MobEntity, PlayerEntity, Forge classes: IForgeItemStack

patchwork-extensions/src/main/resources/patchwork-extensions.mixins.json

@@ -4,6 +4,7 @@
   "compatibilityLevel": "JAVA_8",
   "mixins": [
     "ItemTagsAccessor",
+    "MixinAxeItem",
     "MixinEnchantedBookItem",
     "MixinEntityType",
     "MixinEntityTypeBuilder",