lauchmelder/BloodMagic
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed a bug in the Ritual setting system.

Browse source 
Apparently `null` was being passed for the Will Config and the Will Holder. Seriously, who designed this system!?

... Oh.
This commit is contained in:
WayofTime 2020-11-09 17:12:35 -05:00
parent f9e78fcb40
commit bac2af8857
5 changed files with 26 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

9
src/main/java/wayoftime/bloodmagic/common/data/GeneratorLanguage.java
View file

@ -111,6 +111,15 @@ public class GeneratorLanguage extends LanguageProvider
add("tooltip.bloodmagic.holdShiftForInfo", "Press shift for extra info");
add("ritual.bloodmagic.blockRange.tooBig", "The block range given is too big! Needs to be at most %s blocks.");
add("ritual.bloodmagic.blockRange.tooFar", "The block range given is too far! Needs to be within a vertical range of %s blocks and a horizontal range of %s blocks.");
add("ritual.bloodmagic.blockRange.inactive", "The ritual stone is currently inactive, and cannot have its range modified.");
add("ritual.bloodmagic.blockRange.noRange", "The range was not properly chosen.");
add("ritual.bloodmagic.blockRange.firstBlock", "First block for new range stored.");
add("ritual.bloodmagic.blockRange.success", "New range successfully set!");
add("ritual.bloodmagic.willConfig.set", "The ritual will use these Demon Will types: %s");
add("ritual.bloodmagic.willConfig.void", "The ritual no longer uses Demon Will");
add("ritual.bloodmagic.testRitual", "Test Ritual");
add("ritual.bloodmagic.waterRitual", "Ritual of the Full Spring");
add("ritual.bloodmagic.lavaRitual", "Serenade of the Nether");

10
src/main/java/wayoftime/bloodmagic/common/item/ItemRitualReader.java
View file

@ -26,6 +26,7 @@ import net.minecraft.world.World;
import net.minecraftforge.api.distmarker.Dist;
import net.minecraftforge.api.distmarker.OnlyIn;
import wayoftime.bloodmagic.BloodMagic;
import wayoftime.bloodmagic.demonaura.WorldDemonWillHandler;
import wayoftime.bloodmagic.ritual.AreaDescriptor;
import wayoftime.bloodmagic.ritual.EnumRitualReaderState;
import wayoftime.bloodmagic.ritual.IMasterRitualStone;
@ -34,6 +35,7 @@ import wayoftime.bloodmagic.util.ChatUtil;
import wayoftime.bloodmagic.util.Constants;
import wayoftime.bloodmagic.util.helper.NBTHelper;
import wayoftime.bloodmagic.util.helper.TextHelper;
import wayoftime.bloodmagic.will.DemonWillHolder;
import wayoftime.bloodmagic.will.EnumDemonWillType;
import wayoftime.bloodmagic.will.IDiscreteDemonWill;
@ -202,10 +204,12 @@ public class ItemRitualReader extends Item
this.setCurrentBlockRange(stack, newRange);
}
Ritual ritual = master.getCurrentRitual();
List<EnumDemonWillType> willConfig = master.getActiveWillConfig();
DemonWillHolder holder = WorldDemonWillHandler.getWillHolder(master.getWorldObj(), master.getBlockPos());
int maxHorizontalRange = ritual.getMaxHorizontalRadiusForRange(range, null, null);
int maxVerticalRange = ritual.getMaxVerticalRadiusForRange(range, null, null);
int maxVolume = ritual.getMaxVolumeForRange(range, null, null);
int maxHorizontalRange = ritual.getMaxHorizontalRadiusForRange(range, willConfig, holder);
int maxVerticalRange = ritual.getMaxVerticalRadiusForRange(range, willConfig, holder);
int maxVolume = ritual.getMaxVolumeForRange(range, willConfig, holder);
switch (master.setBlockRangeByBounds(player, range, containedPos, pos2))
{

3
src/main/java/wayoftime/bloodmagic/ritual/types/RitualLava.java
View file

@ -274,8 +274,7 @@ public class RitualLava extends Ritual
@Override
public ITextComponent[] provideInformationOfRitualToPlayer(PlayerEntity player)
{
return new ITextComponent[]
{ new TranslationTextComponent(this.getTranslationKey() + ".info"),
return new ITextComponent[] { new TranslationTextComponent(this.getTranslationKey() + ".info"),
new TranslationTextComponent(this.getTranslationKey() + ".default.info"),
new TranslationTextComponent(this.getTranslationKey() + ".corrosive.info"),
new TranslationTextComponent(this.getTranslationKey() + ".steadfast.info"),