Conversation
notTamion
left a comment
There was a problem hiding this comment.
some grammar in the jd needs to be fixed
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
Lulu13022002
left a comment
There was a problem hiding this comment.
You should also handle unplaced block state properly
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
paper-server/src/main/java/org/bukkit/craftbukkit/block/CraftCreakingHeart.java
Outdated
Show resolved
Hide resolved
paper-server/src/main/java/org/bukkit/craftbukkit/block/CraftCreakingHeart.java
Outdated
Show resolved
Hide resolved
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
...r/patches/sources/net/minecraft/world/level/block/entity/CreakingHeartBlockEntity.java.patch
Outdated
Show resolved
Hide resolved
|
There is a lot of stuff depending on the despawn distance being 34. e.g. Creaking roaming distance and follow distance, redstone output, player spawning range. When setting the despawn distance to anything lower it will cause semi weird behaivour as the creaking will willingly move outside its despawn distance and etc. So i think we should either scale that stuff, Math.min it or add extra api for it. though leaving it half broken like this feels bad |
ok for that then im wait for some more feedback about what do...
|
- allow manage creaking attached - helper method for spawn creaking
…th for low values
Co-authored-by: Tamion <70228790+notTamion@users.noreply.github.com>
This reverts commit 0306df1.
9d00d07 to
179a9a0
Compare
|
A little bump of this... maybe can take the second option for add tags to the rest and make user handle all if wanna touch that (or remove the set and forget the problem) the only issue i has is where are the static fields being used.... many "32" and not sure what of all are for what :/ |
|
I remove the whole distance API based in the implications for support that, also update for the last changes in that block from NMS |
- Missing Nullable annotation for construct a instance with not location (like happen in clone) - Change how check the ServerLevel for avoid IDE warnings correctly
Add more methods to the Creaking Heart