Wizardry icon indicating copy to clipboard operation
Wizardry copied to clipboard

[Bug] NON_ELEMENTAL_UPGRADE_BONUS doesn't work

Open touhouzun opened this issue 6 months ago • 0 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Observed behaviour

2024-08-18_16 02 43 2024-08-18_16 13 50 2024-08-18_16 19 23 Couldn't apply melee_upgrade or other upgrades to none-elemental wand any further like these screenshots.

Expected behaviour

None-elemental master wand can be applied with up to 12 upgrades, instead of 9.

		// Special upgrades
		Item specialUpgrade = upgrade.getItem();

		int maxUpgrades = this.tier.upgradeLimit;
		if(this.element == Element.MAGIC) maxUpgrades += Constants.NON_ELEMENTAL_UPGRADE_BONUS;

		if(WandHelper.getTotalUpgrades(wand) < maxUpgrades
				&& WandHelper.getUpgradeLevel(wand, specialUpgrade) < Constants.UPGRADE_STACK_LIMIT){

Steps to reproduce

  1. take a none-elemental wand and several upgrades from the inventory.
  2. upgrade the wand to maximum.

Crashlog

No response

Environment (Singleplayer/Server, etc.)

Singleplayer

Mod version

4.3.14

Minecraft version

1.12.2

Forge version

1.12.2 - 14.23.5.2860

Other mods

None in the second and third screenshots.

touhouzun avatar Aug 18 '24 08:08 touhouzun