* Add Handlers for Ritual of Crushing
Fixes the crushing ritual's corrosive will augment.
* Simplify Handler Class
Use Primitives, ensure return value is non-null
* Make the Handler Handle Draining Per Use
* Revert "Make the Handler Handle Draining Per Use"
This reverts commit bacaa610febc5a609a7a891ceed41b0e0fb2f05d.
* Reinstated Compression sigil.
- does not compress wooden planks into crafting tables
- searches for reversible 3x3/2x2 recipes with a single material type
- probably has a lot of redundant stuff and looks silly
- uses try/catch (might want to find a different approach, some people scoff at this)
- should probably have spent the night sleeping, I'm taking exams tomorrow and will probably sleep the whole day through.
* Learned how to properly handle the "NullPointerException"-situation.
* Update BaseCompressionHandler.java
* Update CompressionRegistry.java
* Removed (almost) all code comments (only a one-liner remains that serves as pointer to a completly commented-out class (StorageBlockCraftingRecipeAssimilator)).
Made methods and variables for long function calls to be more efficient.
Fixed an oversight that resulted in a NullPointerException after removing redundant checks that were made to prevent exactly that.
Rearranged and reformatted code.
* corrected something that could be considered a typo but was an oversight
* This should be it. An Array should be more efficient but you can correct me if I'm wrong. In either case it does what it is supposed to do.
* Fixed. Needed a seperate inventory for the reversible check (or clear the previously used one, but then I'd had to repopulate again and that would just have been messy)
* Forgot one of my lines.
* Fix and cleanup.
Could definitely clean more but that should suffice for now.
* Modified sigils of lava, water, and void to interact with tanks correctly.
* Fixed up some formatting errors
* Fixed?
* Reimplemented necessary fluid code as extensible functions in ItemSigilFluidBase and made the fluid sigils draw functions from there rather than FluidUtil
* Fixed up formatting and used an actual IDE for once.
* Added a new DamageSource for the Well of Suffering
Added an event handler for death through Well of Suffering
Added a config option to enable (true) or disable (false) Well of Suffering Mob drops.
* Moved the DamageSource creation to RitualManager
Renamed the new DamageSource to "RITUAL_DAMAGE"
DamageSource "RITUAL_DAMAGE" is now used by RitualForsakenSoul and RitualWellOfSuffering
Added death message string for "RITUAL_DAMAGE" for en_US and de_DE
* Cleanup, removed the ability to damage entities in creative mode (creative mode should be considered as having infinite health, the rituals only damage mobs anyways).
* Update GenericHandler.java
* Teleportation now works similar to the Nether Portal:
if teleportation is attempted before the cooldown (10 ticks) is over, the cooldown gets reset.
This way teleportation loops are prevented.
You can now:
- Stand on a loop of permanently activated (well, constantly refreshing through a high-speed redstone clock) Teleposers, only getting teleposed once per stepping on it
- AFK in the Gate of the Fold without getting teleported constantly
- and other things, I guess.
* Bound Pickaxe AoE ability now destroys blocks properly.
closes#1001
* Streamlined bound tool harvest code.
Cleanup, duplicate code for bound tools moved as method to ItemBoundTool.sharedHarvest()
Tested aoe harvest with Stone, Dirt/Grass and Oak trees. Same result as before.
* silkTouch and fortuneLvl are now passed instead of recalculated on every block
* Rework. Desyncs for no apparent reason.
Apart from that, it works.
Desyncing started after I implemented the "diagTemplate" methods.
Can definitely be optimised, some advice for that would be good (lots of duplicated code for the "templates").
* Code cleanup.
Reverted stuff that I didn't want to commit.
* Some fine tuning for speed.
playerVel is not very accurate still but this should work out for most stuff.
Might need a lot of testing.
Might still not work properly on servers.
Diagonal templates are inconsistent, need variables switched.
Will deal with that later (maybe).
In any case, this works just fine unless you're speeding diagonally.
* The occasional stuff that should not be pushed.
Meaningless whitespaces or code reformatting.
* The occasional stuff that should not be pushed.
Meaningless whitespaces or code reformatting.
Take 2.
* MERGE ME I'M AWESOME
* removed unused import
* Refactoring, adding braces, renaming constants etc etc.
done.
* take 2
* take 2
* Removed one-liners
they grew organically because I initially created every "bridge" with a unique for loop before I removed duplicate code by passing variables/ranges
they were helpful until I found good names for the variables, now they've outlived their usefulness
* tiny bit of fine tuning
* various fluff / small fixes
more fine tuning, code reformatting, meaningful variables, for loop fix (variable mismatch), shrinking a redundant method to a call to an advanced method
- Checks if the arrow is the vanilla arrow, if it is then it continues with normal use. But, if arrow isn't the vanilla arrow it uses their custom arrow entity.
* Added Soul Fray check.
Sacrificial dagger now glows even more if you're fully prepared!
Added NBT field for maximum incense altar bonus from the last incense altar the player has encountered.
(There is a case in which the dagger glows even if the player is not at maximum incense bonus:
The player must have been at maximum incense bonus and then gone into the vincinity of a stronger incense altar.
The maximum incense bonus data field only updates once the maximum bonus has been reached for efficiency.)
* Multiplayer fixed.
* Fixed weirdness that occurred during a phase I don't remember.
* AWAITING ALLOWANCE FROM BRANDON3055 (PERMISSION TO USE CODE SNIPPET FOR INTERDIMENSIONAL/CROSSDIMENSIONAL TELEPORT) COPYRIGHT ALL RIGHTS RESERVED FOR BRANDON3055 (CODE SNIPPET)
AWAITING ALLOWANCE FROM TEHNUT, WAYOFTIME TO USE FOREIGN CODE (AS IT NEEDS ALLOWANCE & POSSIBLE CREDITS)
DO NOT USE.
* Forgot something.
Requires permission from Brandon3055 (copyright on code snippet)
Requires permission from WayofTime, TehNut (because it uses foreign code that might need attribution)
Do not use.
* License permits usage.
Sorry for bothering everyone involved.
Don't merge. Teleposer broken. Teleposition sigil works though.
* IT WORKS!!!!
- re-enabled cross dimensional telepositioning
- works even if target teleposer is unloaded at the time of activation (force loads the chunk so the teleposer can be found, releases ticket when player arrives)
- entity teleposition works properly
- I'm tired and slightly insane
- nobody asked for this.
closes#973 (rewrite might still be needed though)
might be able to close the following issue:
- #1198 (improved cross dimensional teleportation code thanks to brandon3055)
* Zombies stop holding onto tickets now.
* Oversight.
* Fixed downgrade level inconsistency
* Squashed commit of the following:
commit 7263e6fbc2
Author: PiscesdanAT <40119554+PiscesdanAT@users.noreply.github.com>
Date: Sat Jun 9 18:01:30 2018 +0200
Update de_DE.lang (#1337)
Someone mixed up dusk and dawn
Updated de_DE.lang
Added some missing stuff in en_US.lang
* Update en_US.lang
Fixed the typo I inserted with previous commit.
* bloodmagic/.../de_DE.lang
* more fitting translations
bloodmagicguide/.../de_DE.lang
* 1/3 finished
.. en_US.lang
Whitespaces
Signed-off-by: Iorce <tgremeyer@icloud.com>
* bloodmagic/.../de_DE.lang:
- improved translations - again
bloodmagicguide/.../en_US:
- mudpack -> modpack (someone doesn't like them, it seems)
- wares off -> wears off (Typo)
bloodmagicguide/.../de_DE:
- another batch of translations
Signed-off-by: Iorce <tgremeyer@icloud.com>
* bloodmagic/.../de_DE.lang:
- improved translations - again
bloodmagicguide/.../en_US:
- mudpack -> modpack (someone doesn't like them, it seems)
- wares off -> wears off (Typo)
bloodmagicguide/.../de_DE:
- another batch of translations
- finished the "Ritual Master"
Signed-off-by: Iorce <tgremeyer@icloud.com>
* bloodmagic/.../de_DE.lang:
- improved translations - a bit more literal for some while keeping variety (Suppression and Displacement can have the same translation)
bloodmagicguide/.../de_DE:
- translated architect entry texts / titles
* bloodmagicguide/.../en_US.lang:
consistency (was Sacrificial Knife, every occurrence of that item is now called "Sacrifical Dagger"
bloodmagicguide/.../de_DE.lang:
some more....
* bloodmagicguide/.../en_US.lang:
corrected required Blood Altar tier from "greater than" to "greater than or equal to"
bloodmagicguide/.../de_DE.lang:
some more....
* bloodmagicguide/.../en_US.lang:
added clarification for Incense Altar block range
bloodmagicguide/.../de_DE.lang:
changed "Wahrheitssiegel" to "Siegel der Weissagung"
changed "Weihrauchaltar" to "Räucheraltar"
some more....
* bloodmagicguide/.../en_US.lang:
clarification for Sigil of the Fast Miner entry ("and" -> "at")
bloodmagicguide/.../de_DE.lang:
some more....
* bloodmagicguide/.../de_DE.lang:
some more....
* Changed "Blutorb" to "Blutkugel" in translation. Might have missed some cases of severe german grammar.
bloodmagicguide/.../en_US.lang:
Removed a "my", if someone misses it, it myght be readded.
bloodmagicguide/.../de_DE.lang:
some more....
* long overdue: closes#1320 (the disorient thing that nobody cares about)
bloodmagicguide/.../en_US.lang:
people are now less warry and more wary about sentient mimics.
bloodmagicguide/.../de_DE.lang:
some more.... (just the mimic stuff, since I commit after every typo I fix :o )
* bloodmagicguide/.../de_DE.lang:
some more....
Completed "The Architect"
* bloodmagic/.../de_DE.lang:
Leere Tafel -> Leere Schiefertafel (material clarification, remaining names unchanged to keep the names short)
Rudimentäre Falle -> [...] Schlinge (item icon looks like the latter, word has multiple meanings)
Ansammlung Kristalle [...] -> Kirstallansammlung (easier on the tongue, better text flow)
bloodmagicguide/.../en_US.lang:
fixed mistake in textflow
bloodmagicguide/.../de_DE.lang:
some more....
* bloodmagic/.../de_DE.lang:
clarification
Haupt- -> Meister
bloodmagicguide/.../de_DE.lang:
some more....
bloodmagicguide/.../en_US.lang:
Updated entry to reflect the new mechanic for creating pure will types
* bloodmagic/.../de_DE.lang:
Ansammlung -> Gruppe
bloodmagicguide/.../de_DE.lang:
finished!
bloodmagicguide/.../en_US.lang:
fixed ritual name
* Finalized de_DE.lang
Ätzend -> Korrosiv
Contains: -> Enthält:
* Finalized de_DE.lang
ätzend -> korrosiv
* Update de_DE.lang
* Update de_DE.lang
Drained -> Verbraucht
Item -> Gegenstand
* Update de_DE.lang
Knoten-Router -> Netzknoten-Router
* Update de_DE.lang
Changed the base names for demonic Will to "Willpower"
-> clarification for the user that the residue is actualy willpower and the quality of said will is a measure of the potential
-> not changing other occurrences or notions in the book as no sane person would repeat "willpower" at every occasion.
* Applied quick fixes to de_DE.lang
* Merge me!
reverted a whitespace change in gradle.properties (no idea where that came from) and moved the disorient thingy to another PR)
* Take No.2
at fixing whitespaces.
* TAKE NO.3
(HOPEFULLY FINAL TAKE)
* Reimplemented a lot of mimic logic and did a first run of changing how mimics store their states.
* Finished removing all metadata calls for blocks replaced by mimics.
* Update EntityMimic.java
* Update ItemBlockMimic.java
* Update TileMimic.java
* How did I even replace a semicolon with a slash.
* Changed all tabs to 4 spaces. Changed Serializer to StateUtil
* Fixed spacing again, hopefully for the last time
* Players in creative mode can now add crystals to crystal blocks by right clicking them with an item demon crystal.
Crystal block texture now updates on the next tick. The render update is only called when a new crystal has been added (by natural or unnatural means).
Adding a crystal imitates a positive result of checkAndGrowCrystal()
closes#1151
* fixed conditional to allow harvesting crystals if not in creative and holding crystal
* Should close#1121
Not tested (yet) (because I haven't figured out how to launch the server with a fake player entity... yet).
* will close#1121
Tested.
Specters only attack players if they attack the owner or the specter.
Spectres now autoattack every instance of EntityMob in range.
//TODO: (fluff) make Bow Specter change angle based on target position (it currently always looks like it shoots into the ground)
* Fix the Blood Tank BB
* Add modid to command localizations to prevent conflicts
* Fixed the items not being drawn on the right Y-level for the Sigil of Holding HUD
Corrected localizations of other lang files
* SoulTicket internal implementation
* do what TehNut says
* implement hashCode()
* Fix toggleable sigils draining on r-click when it shouldn't
Also moved the ItemSigil and ItemSigilToggleable to the sigil package (why wasn't it there???)
* Add modid to command localizations to prevent conflicts
* Corrected localizations of other lang files
* Fixed the items not being drawn on the right Y-level for the Sigil of Holding HUD
This is solving the issue I raised in https://github.com/WayofTime/BloodMagic/issues/1368
Added a single check to line 123 of src/main/java/WayofTime/bloodmagic/alchemyArray/AlchemyArrayEffectMobSacrifice.java.
The check makes sure the effective health of the currently targeted entity is above 0. So bosses and players are no longer set to -1 health when the array finishes crafting.
