Updated to latest version of ImmersiveEngineering - #13
Open
Enginecrafter77 wants to merge 12 commits into
Open
Conversation
Some weird variable name indentation using tabs was fixed. Tabs are cool, but variable type should be separated using SINGLE space from variable's name.
Line 70 and 71 of ALternatingFlux.java were using legacy string key-based registration. As this method is (was?) deprecated, I replaced it with the standard ResourceLocation-based registration. It was done in background anyways, but this eliminates the need to do additional checks.
Well, not exactly the latest, but the latest usable forge version. Newer versions require Forge Gradle 3, which works only for MC 1.13+.
Also, general optimizations were done to the codebase.
The ItemIEBase was absolutely unsuitable to inherit from, since it does nothing besides promoting bad coding practice of tying registry names together with translation keys. Minecraft Forge officially states that this kind of relationship is considered a bad coding practice. As such, it was removed, and replaced with much cleaner implementation.
The code was moved over to BlockAFTileProvider
As stated in the last commit, the registry names have nothing to do with translation keys. This commit is a final sentence to that statement. It completely removes the leftover dependance in the main mod class, and also increases flexibility a little bit (See BlockUnregisteredIETileProvider).
The oridinal integer seemed like a remant from copy of the original IE class ItemWireCoil. As such, I removed it to only cover the needs of the AF system.
Moved the manual managing code to standalone class.
Also, JEI was added as runtime-only dependency, so the recipes are easier to navigate.
Fixed 2 switch statements without default case and 1 empty if block.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR aims to provide better compatibility with the latest version of Immersive Engineering for 1.12.2.
Additional features include:
The mod was tested on both singleplayer and dedicated server, all seems well. It would be good if you made a new version out of this.