search

pnill / T4MP

T4MP (Turok 4 Multiplayer) is an modification to bring online play to Turok 4 Evolution for PC it's currently not any where near being actually useable for competitive or serious gameplay but it's getting there.
GNU Affero General Public License v3.0
16 stars 2 forks source link

Weapon Modifiers are not Synced #25

Closed pnill closed 4 years ago

pnill commented 6 years ago

Weapon modifiers when right-clicking are not currently synced (Such as Spiked Mines, MiniGun, etc.)

pnill commented 4 years ago

https://github.com/PermaNulled/T4MP/commit/cc715d473127b5c3c667e512c850ae54aa9abfa8 introduced a hook that will allow the syncing of this, trying to determine how it will be implemented currently.

Thinking the best way to go about it is forcing the client to wait until the server tells them the switch happened on the server-end.

pnill commented 4 years ago

Partially fixed with https://github.com/PermaNulled/T4MP/commit/13ee7758622f2b744f61c22b792afcfcdd3e8347

Shotgun still having issues, and clients aren't syncing properly need to determine why.

pnill commented 4 years ago

As of https://github.com/PermaNulled/T4MP/commit/0e31717f9306ccc8e54c9575b26b494e1ee20a42 shotgun is still not working but some networking fixes were applied which made it so the other side was able to switch through the modifications.

Testing of this allowed for finding other bugs like the fact that the WeaponWheel would benefit from being entirely server controlled as currently there's a lot of issues with the client believing it has weapons or modifiers it does not, will probably look at that next as it will improve overall game-play.

pnill commented 4 years ago

The shot gun is calling the same function but for some reason does not apply the same result, I believe it may be another function which is being called on the release of the mouse click that causes the shotgun to load more shells, I haven't found any other weapons which are not controlled by this same function/hook.

pnill commented 4 years ago

As of https://github.com/PermaNulled/T4MP/commit/6b46a9ce1e8f0181c6daf2c357f723c16220d766 this appears to be completely resolved.