Syncing SetActive over network

I am trying to implement switching between two weapons. My method is currently:

1) Player prefab has both weapons as children. The first one is active while the second one is inactive to begin with.

2) When a player presses the switch weapon button ('R'), the second one becomes active and the first one become inactive (and vice versa).

When the host presses 'R', it calls RpcToggleWeapon() to toggle its weapon for all clients. When a client presses 'R', it calls CmdToggleWeapon() to make the server call RpcToggleWeapon(), to toggle its weapon for all clients (host included).

My problem:

If a client connects after a host/client already switched weapons, it will not register on the new client.

e.g.

Host starts game.

Client1 connects.

Client1 presses 'R'. Weapon is changed correctly on Host and Client1.

Client2 connects, but sees the wrong weapon for Client1.

Question 1)

Is my method of client2client communication correct? (call cmd to server then have it call rpc to all other clients) I feel there is a better way to sync SetActive...

Question 2)

If my method is the best way, how can I fix the issue described above?

I've attached some code for reference,

 void Update () {
     if (!isLocalPlayer) return;
     if (Input.GetKeyDown(KeyCode.R)){
         if (!isServer) { // Client
             CmdToggleWeapon();
         } else { // Host
             RpcToggleWeapon();
         }
     }
 }

 [Command]
 void CmdToggleWeapon(){
     RpcToggleWeapon();
 }

 [ClientRpc]
 void RpcToggleWeapon(){
     for (int i = 0; i < weapons.Length; i++) {
         weapons[i].gameObject.SetActive(!weapons[i].gameObject.activeSelf);
     }
 }

Thanks!