devbeni/Nitrox
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor EntityMetadataUpdateProcessor to simplify metadata update logic and remove container-related checks

Browse Source
This commit is contained in:
B3ni
2025-07-06 00:25:54 +02:00
parent 38f50c8819
commit 1b3da7bece
1 changed files with 3 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
NitroxServer/Communication/Packets/Processors/EntityMetadataUpdateProcessor.cs
View File

@@ -38,34 +38,20 @@ public class EntityMetadataUpdateProcessor : AuthenticatedPacketProcessor<Entity
foreach (Player player in playerManager.GetConnectedPlayers())
{
bool updateVisibleToPlayer = player.CanSee(entity);
// Always sync container/storage metadata to all visible players
bool isContainerMetadata = IsContainerRelatedMetadata(packet.NewValue);
if (player != sendingPlayer && (updateVisibleToPlayer || isContainerMetadata))
if (player != sendingPlayer && updateVisibleToPlayer)
{
player.SendPacket(packet);
}
}
}
private bool IsContainerRelatedMetadata(EntityMetadata metadata)
{
// Check if metadata is related to containers/storage
return metadata.GetType().Name.Contains("Container") ||
metadata.GetType().Name.Contains("Storage") ||
metadata.GetType().Name.Contains("Inventory");
}
private bool TryProcessMetadata(Player sendingPlayer, Entity entity, EntityMetadata metadata)
{
return metadata switch
{
PlayerMetadata playerMetadata => ProcessPlayerMetadata(sendingPlayer, entity, playerMetadata),
// Always allow container/storage metadata updates for proper sync
_ when IsContainerRelatedMetadata(metadata) => true,
// Allow metadata updates from any player by default
_ => true
};
@@ -87,4 +73,4 @@ public class EntityMetadataUpdateProcessor : AuthenticatedPacketProcessor<Entity
Log.WarnOnce($"Player {sendingPlayer.Name} tried updating metadata of another player's entity {entity.Id}");
return false;
}
}
}