Skip to content

Commit

Permalink
Disable entity activation range
Browse files Browse the repository at this point in the history
This patch is broken as is and Mojang has since introduced
simulation distance which more or less makes this deprecated.

The code also had logic bug where it tried to only tick
entities in unloaded chunks, contrary to what we wanted,
which would only rarely surface.
  • Loading branch information
aromaa committed Nov 1, 2024
1 parent 08a7c2c commit 2f56c5f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@ public static boolean checkEntityImmunities(final Entity entity) {
*/
public static boolean checkIfActive(final Entity entity) {
// Never safe to skip fireworks or entities not yet added to chunk
if (entity instanceof Player || entity.level().isClientSide() || !entity.touchingUnloadedChunk() || entity instanceof FireworkRocketEntity) {
if (entity instanceof Player || entity.level().isClientSide() || entity.touchingUnloadedChunk() || entity instanceof FireworkRocketEntity) {
return true;
}
final LevelChunkBridge activeChunk = ((ActiveChunkReferantBridge) entity).bridge$getActiveChunk();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ public class EntityActivationRangePlugin extends AbstractMixinConfigPlugin {

@Override
public boolean shouldApplyMixin(final String targetClassName, final String mixinClassName) {
return SpongeConfigs.getCommon().get().modules.entityActivationRange;
return false && SpongeConfigs.getCommon().get().modules.entityActivationRange;
}

@Override
Expand Down

0 comments on commit 2f56c5f

Please sign in to comment.