无法附魔粉笔mod物品

[donmor]

不确定CF上的包是arch-1.20.1分支还是forge-1.20.1，但都有类似以下代码：

    @Override
    public boolean isAcceptableItem(ItemStack stack) {
        ModConfig config = ModConfig.get();
        switch (config.compatibilityMode) {
            case WHITELIST_ONLY -> {
                return hasMatchItemStack(config.whitelist, stack);
            }
            case BLACKLIST_ONLY -> {
                return !hasMatchItemStack(config.blacklist, stack);
            }
            case WHITELIST_AND_DEFAULT -> {
                if (!hasMatchItemStack(config.whitelist, stack)) {
                    return false;
                }
            }
            case BLACKLIST_AND_DEFAULT -> {
                if (hasMatchItemStack(config.blacklist, stack)) {
                    return false;
                }
            }
        }
        return stack.isDamageable() || !stack.isStackable() || stack.isOf(Items.BOOK) || super.isAcceptableItem(stack);
    }

显然case WHITELIST_AND_DEFAULT段实际排除了任何白名单以外且不满足默认条件的物品，应改为if (!hasMatchItemStack(config.whitelist, stack)) return true;；但在WHITELIST_ONLY模式下，又无法附魔任意物品（比如可堆叠物品），愿意不明

[iMoonDay]

WHITELIST_ONLY模式就是只判断白名单，其他条件一律忽略。
WHITELIST_AND_DEFAULT模式指的是如果物品在白名单里，则再进行二次判断是否符合基本的附魔条件。
hasMatchItemStack方法是判断物品是否在白名单中，怎么会在返回false时通过判断呢？

[donmor]

那没法在原有条件之外额外允许白名单物品吗，比如WHITELIST_OR_DEFAULT之类？

[donmor]

以及我在使用WHITELIST_ONLY模式时，即使指定了白名单，仍无法给可堆叠物品附魔

[iMoonDay]

因为附魔机制就是不兼容可堆叠物品呀，只能通过铁砧附魔上去

[donmor]

而最下面的一行默认直接让任何不可堆叠物品通过（甚至水瓶这种），但粉笔mod的粉笔盒却无法附魔

[iMoonDay]

你是选的哪种模式？如果是DEFAULT模式应该就可以正常附魔了

[iMoonDay]

那没法在原有条件之外额外允许白名单物品吗，比如WHITELIST_OR_DEFAULT之类？

暂时还没有人反应需要这个功能，我也就没加，因为DEFAULT模式已经基本够用了

[donmor]

DEFAULT也无法附魔粉笔盒

[iMoonDay]

那这可能是bug，发一下模组链接，我有空修复一下

[iMoonDay]

DEFAULT也无法附魔粉笔盒

我刚刚看了一下粉笔的源代码，是作者写死了的，不允许附魔https://github.com/mortuusars/Chalk/blob/1.20/src/main/java/io/github/mortuusars/chalk/items/ChalkBoxItem.java

    @Override
    public boolean isRepairable(@NotNull ItemStack stack) {
        return false;
    }
    @Override
    public boolean isEnchantable(@NotNull ItemStack stack) {
        return false;
    }
    @Override
    public boolean isBookEnchantable(ItemStack stack, ItemStack book) {
        return false;
    }
    @Override
    public boolean canApplyAtEnchantingTable(ItemStack stack, Enchantment enchantment) {
        return false;
    }

[donmor]

啊这，优先级比这边的高的吗？

[donmor]

有没有办法覆盖掉？

[iMoonDay]

目前不清楚是否有解决办法，等我有空的时候加一个强制附魔功能吧

[donmor]

袜，牛