Is this Multiplayer movement logic correct?

Most have issues with replication because they don’t understand the client-server model. In most cases all you need to do is tell the server and it will handle telling everyone else.