Skip to content
This repository has been archived by the owner on Apr 28, 2020. It is now read-only.

1.13.1 Port #58

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Conversation

Chocohead
Copy link

Supplants #50, effectively applies #54. All access transformations have been updated, as have the MCP mappings and the test mod now compiles. Still need to update some overwrite Mixins that definitely don't work correctly anymore, but it loads into the world a basic things work.

modmuss50 and others added 10 commits October 2, 2018 14:35
No more compile errors, but there are definitely some Mixins that don't work (or make sense) with 1.13
Now it compiles too
Bundled the updating script designed to do MCP methods to Notch names too, fields and classes are comparatively easier (and faster)
Game can now load to the world
Don't think anything changed that was being used, but that's what I was using
Also updates the profile to extend from 1.13.1
@modmuss50 modmuss50 mentioned this pull request Nov 7, 2018
4 tasks
`ItemRenderer` will cause `ModelResourceLocation` to crash unhelpfully if an item is registered with a null resource location
This doesn't fix the fact it crashes, just allows for the item to be found
Now there is a proper dimension registry the old hacks can start to be removed
Breaks the test mod loading into the world from `MixinMinecraftServer` needing updating too
Removes ATing string registering methods when there are resource location versions instead
Mojang now has a `DimensionType` -> `World` map instead of the old `World[]` array so most of the old hacks can go
Most `ClientTickable`s will need it anyway, so might as well make it as convenient as `ServerTickable` is
Technically `func_212321_a` is the proper portal teleport, but it has a whole load of expectations that makes it break too easily
So we just jump on the TeleportCommand which is actually designed for what we're using it for
It's less annoying now, but still worth testing to make sure enum extending is fine
It's been replaced in favour of the `Message` system for a while now
@Chocohead Chocohead self-assigned this Dec 3, 2018
@Martin-H2
Copy link

Martin-H2 commented Apr 13, 2019

Guys why is there another fork of this ?
Why not simply use one repo ?
Accept this PullRqeuest ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants