tooltip from kubejs buged with ModerFix

Question

tooltip from kubejs buged with ModerFix

Kisuny opened this issue 7 months ago · 11 comments

Bug Description

When I add kubejs items tooltops with ModernFix all the tooltips added this way are displayed twice.

Disabling mixin.perf.resourcepacks removes this problem, but judging from the descriptions of this feature - it's a major fix, is there another way to fix this problem?

Screenshot:

kubejs code exmaple:

ItemEvents.tooltip(tooltip => {
let artifactsTooltip = (args) => {
            tooltip.addAdvanced(args.item, (item, advanced, text) => {
                if (!tooltip.ctrl) {
                    text.add(1, ctrlText)
                } else {
                    text.add(1, Text.gold('Способ получения:').bold(true))
                    text.add(2, Text.gray(args.text1))
                    text.add(3, Text.darkGray(args.text2))
                }
            })
        };

artifactsTooltip(
            {
                item: 'artifacts:cloud_in_a_bottle', 
                text1: 'some text for description after press ctrl',
                text2: 'second exmaple text'
        });
});

Reproduction Steps

Everything is described in detail above

Log File

latest.log

Mickeon · Answer 1 · 2024-05-16T20:11:40.000Z

I can confirm that we are experiencing a form of this bug, as well. I "fixed" it by storing whether the tooltips were already processed in a variable, but it's quite unideal and not guaranteed to work all the time.

Kisuny · Answer 2 · 2024-05-03T10:26:13.000Z

Wrong, mixin.perf.resourcepacks does not help...

Empirically I managed to find out that the problem is in these parameters:
mixin.perf.mojang_registry_size=false
mixin.perf.reduce_blockstate_cache_rebuilds=false

embeddedt · Answer 3 · 2024-05-03T16:05:34.000Z

Neither of those options should have an impact on tooltips.

Can you reproduce this with just ModernFix, KubeJS, and one other content mod? There are many mods installed that could be causing this.

I would also suggest ensuring you are using the latest version of KubeJS (6.1 or later), as older versions had concurrency bugs that could cause these types of problems randomly.

Kisuny · Answer 4 · 2024-05-03T18:05:20.000Z

Neither of those options should have an impact on tooltips.

Can you reproduce this with just ModernFix, KubeJS, and one other content mod? There are many mods installed that could be causing this.

I would also suggest ensuring you are using the latest version of KubeJS (6.1 or later), as older versions had concurrency bugs that could cause these types of problems randomly.

I did it again with fewer mods.

latest.log

mod list:
fabric-api-0.77.0+1.19.2.jar
rhino-fabric-1902.2.3-build.284.jar
architectury-6.6.92-fabric.jar
modernfix-fabric-5.17.0+mc1.19.2.jar
RoughlyEnoughItems-9.2.724-fabric.jar
artifacts-7.1.1+fabric.jar (added for test tooltips)
cloth-config-8.3.115-fabric.jar
trinkets-3.4.2.jar
kubejs-fabric-1902.6.2-build.63.jar (last version for 1.19.2)

Only one client side scritps:

ItemEvents.tooltip(tooltip => {
    
        const shiftText = [Text.of('Зажми ').gold(), Text.of('Shift ').yellow(), Text.of('для подробной информации.').gold()]
        const ctrlText = [Text.of('Зажми ').gray(), Text.of('CTRL ').gold(), Text.of('Чтобы узнать как получить.').gray()]
        tooltip.addAdvanced('artifacts:helium_flamingo', (item, advanced, text) => {
            // shift, alt and ctrl are all keys you can check!
            if (!tooltip.shift) {
                text.add(1, shiftText)
            } else {
                text.add(1, Text.gold('Способ получения:').bold(true))
                text.add(2, Text.gray('🎣 Выбить при рыбной ловле существ, из ночной фурии.'))
                text.add(3, Text.darkGray('Условия поимки ночной фурии: ночь, вода, обычный мир.'))
                text.add(4, Text.gray('🗡 Мешок с добычей плоти и крови.'))
            }
        })
    
    
    
        // тултипы для артефактов
        let artifactsTooltip = (args) => {
            tooltip.addAdvanced(args.item, (item, advanced, text) => {
                if (!tooltip.ctrl) {
                    text.add(1, ctrlText)
                } else {
                    text.add(1, Text.gold('Способ получения:').bold(true))
                    text.add(2, Text.gray(args.text1))
                    text.add(3, Text.darkGray(args.text2))
                }
            })
        };
        artifactsTooltip(
            {
                item: 'artifacts:cloud_in_a_bottle', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Призрака».',
                text2: 'Условия поимки призрака: ночь, вода, обычный мир.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:vampiric_glove', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Вампира».',
                text2: 'Условия поимки вампира: ночь, вода, обычный мир.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:crystal_heart', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Гаснущий Хризолит».',
                text2: 'Условия поимки «Гаснущий Хризолит»: Кристаллическая жидкость'
        });
        artifactsTooltip(
            {
                item: 'artifacts:superstitious_hat', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Гаснущий Хризолит».',
                text2: 'Условия поимки «Гаснущий Хризолит»: кристаллическая жидкость'
        });
        artifactsTooltip(
            {
                item: 'artifacts:flippers', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Кракен», «Мегалодон», «Морской император».',
                text2: 'Условия поимки «Кракен»: ночь, вода, обычный мир.\nУсловия поимки «Мегалодон»: вода, обычный мир.\nУсловия поимки «Морской император»: вода, обычный мир.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:snorkel', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Кракен», «Морской император».',
                text2: 'Условия поимки «Кракен»: ночь, вода, обычный мир.\nУсловия поимки «Морской император»: вода, обычный мир.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:night_vision_goggles', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Смотрящий».',
                text2: 'Условия поимки «Смотрящий»: вода, энд.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:antidote_vessel', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Багровая плоть», «Оживший корень».',
                text2: 'Условия поимки «Багровая плоть»: лава, ад.\nУсловия поимки «Оживший корень»: жидкая грязь'
        });
        artifactsTooltip(
            {
                item: 'artifacts:digging_claws', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Тёмный дракон», «Восставший из драконьей гнили»',
                text2: 'Условия поимки «Тёмный дракон»: драконья гниль.\nУсловия поимки «Восставший из драконьей гнили»: драконья гниль.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:feral_claws', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Тёмный дракон»',
                text2: 'Условия поимки «Тёмный дракон»: драконья гниль.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:lucky_scarf', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Восставший из драконьей гнили»',
                text2: 'Условия поимки «Восставший из драконьей гнили»: драконья гниль.'
        });
        artifactsTooltip(
            {
                item: 'artifacts:running_shoes', 
                text1: '🎣 Выбить при рыбной ловле существ, из «Зубастик»',
                text2: 'Условия поимки «Зубастик»: драконья гниль.'
        });
    });

embeddedt · Answer 5 · 2024-05-03T18:15:56.000Z

I tried using just these mods, but still don't see the bug.

Kisuny · Answer 6 · 2024-05-03T18:18:53.000Z

I tried using just these mods, but still don't see the bug.

Do you have an REI?

embeddedt · Answer 7 · 2024-05-03T18:19:32.000Z

Yes, you can see the search bar at the edge of the image, and the buttons next to it.

Kisuny · Answer 8 · 2024-05-03T18:20:25.000Z

Yes, you can see the search bar at the edge of the image, and the buttons next to it.

How is that even possible?

Inqie · Answer 9 · 2024-05-12T02:37:00.000Z

KubeJS loads the tooltips late enough that REI renders them twice. Try the below in client scripts, obviously with your own list of items, to load the tooltips earlier. If it doesn't work, check the support Discord server for KubeJS.

const $TooltipFlag$Default = Java.loadClass('net.minecraft.world.item.TooltipFlag$Default')
REIEvents.hide("item", e => {
  let items = ["minecraft:apple"];
  items.forEach((i) => {
    Item.of(i).getTooltipLines(
      Client.player,
      $TooltipFlag$Default.ADVANCED
    );
  });
});

embeddedt · Answer 10 · 2024-06-05T00:05:37.000Z

As of now I no longer support 1.19.2 (although I'll still take PRs if someone wants to backport a bugfix), and I'm not sure ModernFix is actually the cause of this bug.

Mickeon · Answer 11 · 2024-06-05T09:14:56.000Z

I am on 1.20.1 and still experiencing this on the occasion (it seems to be related to resources reloading), but whether it is ModernFix causing this is indeed questionable.

Share to

Bug Description

Reproduction Steps

Log File