Skip to content

Refactor networking#68

Open
ekulxam wants to merge 18 commits intodoctor4t:mainfrom
ekulxam:networking
Open

Refactor networking#68
ekulxam wants to merge 18 commits intodoctor4t:mainfrom
ekulxam:networking

Conversation

@ekulxam
Copy link
Contributor

@ekulxam ekulxam commented Nov 25, 2025

Originally from #22

  • Put all client receivers in the client, ensuring that they are never loaded on the server
  • Use UUID over String in ShootMuzzleS2CPayload
  • Extract PoisonOverlayS2CPayload from PoisonUtils
  • Unit payloads now only have one instance and cannot be instantiated
  • Added direction (s2c or c2s) in the payload class name

@doctor4t
Copy link
Owner

Resolve conflicts that ensued from the new rewrite please

# Conflicts:
#	src/main/java/dev/doctor4t/trainmurdermystery/cca/PlayerMoodComponent.java
#	src/main/java/dev/doctor4t/trainmurdermystery/game/GameFunctions.java
# Conflicts:
#	src/main/java/dev/doctor4t/trainmurdermystery/networking/GunShootC2SPayload.java
@doctor4t
Copy link
Owner

Looks clean, sorry there's more conflicts LMAO

# Conflicts:
#	src/main/java/dev/doctor4t/trainmurdermystery/game/MurderGameMode.java
#	src/main/java/dev/doctor4t/trainmurdermystery/item/DerringerItem.java
#	src/main/java/dev/doctor4t/trainmurdermystery/item/KnifeItem.java
#	src/main/java/dev/doctor4t/trainmurdermystery/item/RevolverItem.java
#	src/main/java/dev/doctor4t/trainmurdermystery/networking/GunShootC2SPayload.java
#	src/main/java/dev/doctor4t/trainmurdermystery/networking/KnifeStabC2SPayload.java
#	src/main/java/dev/doctor4t/trainmurdermystery/util/PoisonUtils.java
#	src/main/java/dev/doctor4t/trainmurdermystery/util/ShootMuzzleS2CPayload.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants