UNET problem with syncing the change of color.

I have a door wich changes its color to blue when i press the f key.
The code for the input and the network syncing of the door are both on the player object and are ase follows:

Code (CSharp):

1. private void DoorController()
2.     {
3.         /*--------RAY STUFF--------*/
4.         RaycastHit myHit;
5.         Ray ray = GameObject.FindGameObjectWithTag("FPSCamera").GetComponent<Camera>().ScreenPointToRay(new Vector3(Screen.width / 2, Screen.height / 2));
6.        
7.         CastRay (ray, out myHit, m_RaycastOptions.DoorRayDistance);//Cast a ray.
8.        
9.         if (myHit.transform && myHit.transform.GetComponent<TagSystem> ()) {
10.             m_RaycastOptions.Pointer.enabled = true;
11.             if (myHit.transform.GetComponent<TagSystem> ().Tag == "Door_Normal") {//Check if we are actually looking at a door.
12.                 //Execute door code in here.
13.                 m_RaycastOptions.Pointer.enabled = true;
14.                 DoorAnimator = myHit.transform.GetComponent<Animator> ();
15.                
16.                 bool Opened = DoorAnimator.GetBool ("Opened");
17.                
18.                 m_RaycastOptions.Pointer.enabled = true;
19.  
20.                 if (Input.GetKeyDown (KeyCode.F)) {
21.                     if (Opened) {
22.                         //Close Door.
23.                         UnlockSys.GetDoorState (false,DoorAnimator);
24.                         //Play closing sound.
25.                     } else if (!Opened) {
26.                         //Open Door.
27.                         UnlockSys.GetDoorState (true,DoorAnimator);
28.                         //Play opening sound.
29.                     }
30.                 }
31. }

And then i have the class wich syncs that color change :

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3. using UnityEngine.Networking;
4.  
5. /// <summary>
6. /// Class used for the networking of the door animator and door functions.
7. /// </summary>
8. public class NetworkUnlockSystem : NetworkBehaviour
9. {
10.     public Animator DoorAnimator;
11.     [SyncVar]public bool DoorOpened;
12.    
13.  
14.      /// <summary>
15.      /// Used for getting the information that we need for each door.
16.      /// </summary>
17.      /// <param name="Opened"></param>
18.      /// <param name="GotAnim"></param>
19.      public void GetDoorState(bool Opened,Animator GotAnim)
20.      {
21.         if(isLocalPlayer)
22.         {
23.             DoorAnimator = GotAnim;
24.  
25.             if (isServer)
26.             {
27.                 RpcSendDoorStateToAll(Opened, DoorAnimator.transform.name);
28.                 Debug.Log("Im server");
29.             }
30.             else
31.             {
32.                 CmdSendDoorStateToServer(Opened, DoorAnimator.transform.name);
33.                 Debug.Log("Im client");
34.             }
35.         }
36.     }
37.     [ClientRpc]
38.     private void RpcSendDoorStateToAll(bool DoorState,string DoorName)
39.     {
40.         //Updates the DoorOpened variable on all clients from the server.
41.         DoorOpened = DoorState;
42.  
43.         GameObject.Find(DoorName).GetComponent<MeshRenderer>().material.color = Color.blue;
44.         /*SINC THE STUFF IN THE CLIENTS*/
45.         //Works if host does the color change but not if a client does it.
46.     }
47.     [Command]
48.     private void CmdSendDoorStateToServer(bool DoorState,string DoorName)
49.     {
50.         //Update the server from a client.
51.         RpcSendDoorStateToAll(DoorState, DoorAnimator.transform.name);
52.     }
53.  
54. }
55.  

Now, this is only working on the host side and by that i mean if the host goes to the door and presses f the color changes to blue for all the clients and for the server but if a client goes to the door and presses f nothing happens, why is this happening?

 

Hi !

Try to call directly CmdSendDoorStateToServer, I guess isLocalPlayer is false when you are a remote client.

You should use the same function for a local client and remote client to have the same behaviour or you will have some issue like this one.

Cheers !

 

Hi, thanks for the answer!
But unfortunately that seems to not be the problem.
I tried removing it and the result is the same

 

And how would i look up the door without using GameObject.Find()? Since the most possible thing that could happen is for the players to have different doorAnimators assigned.

Anyway, also wanted to say that i edited the code and now looks like this:

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3. using UnityEngine.Networking;
4.  
5. /// <summary>
6. /// Class used for the networking of the door animator and door functions.
7. /// </summary>
8. public class NetworkUnlockSystem : NetworkBehaviour
9. {
10.     public Animator DoorAnimator;
11.     [SyncVar]public bool DoorOpened;
12.    
13.  
14.      /// <summary>
15.      /// Used for getting the information that we need for each door.
16.      /// </summary>
17.      /// <param name="Opened"></param>
18.      /// <param name="GotAnim"></param>
19.      public void GetDoorState(bool Opened,Animator GotAnim)
20.      {
21.         DoorAnimator = GotAnim;
22.  
23.         CmdSendDoorStateToServer(Opened, DoorAnimator.transform.name);
24.     }
25.  
26.     [ClientRpc]
27.     private void RpcSendDoorStateToAll(bool DoorState,string DoorName)
28.     {
29.         //Updates the DoorOpened variable on all clients from the server.
30.         DoorOpened = DoorState;
31.  
32.         GameObject.Find(DoorName).GetComponent<MeshRenderer>().material.color = Color.blue;
33.         /*SINC THE STUFF IN THE CLIENTS*/
34.         //Works if host does the color change but not if a client does it.
35.     }
36.     [Command]
37.     private void CmdSendDoorStateToServer(bool DoorState,string DoorName)
38.     {
39.         //Update the server from a client.
40.         RpcSendDoorStateToAll(DoorState, DoorAnimator.transform.name);
41.     }
42.  
43. }
44.  

but it just does exactly the same thing.

 

Can you please explain further?
I mean, i alredy added a networkIndentity but what do you mean by the clientRpc getting routed directly?

 

Actually.. No , it should be passing DoorName.
That still doesnt fix my main problem anyway

Actually, it kinda does...
The clients can now also color the door but the color doesnt get synced back to the server though

 

Thanks @seanr for all your help, i finally made it work.
I am posting here the final code with animator syncing also, the purpose of all the color stuff was to get the hang of how
networking works so now i have this 2 classes:

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3. using UnityEngine.Networking;
4.  
5. public class Door : NetworkBehaviour
6. {
7.     [SyncVar(hook = "OnOpen")]
8.     public bool openState;
9.     public Animator DoorAnimator;
10.  
11.     public override void OnStartClient()
12.     {
13.         // set color here
14.         DoorAnimator = GetComponent<Animator>();
15.         DoorAnimator.SetBool("Opened", openState);
16.     }
17.  
18.     void OnOpen(bool newOpenState)
19.     {
20.         // set color here
21.         openState = newOpenState;
22.         DoorAnimator.SetBool("Opened", openState);
23.     }
24.  
25. }
26.  

And:

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3. using UnityEngine.Networking;
4.  
5. /// <summary>
6. /// Class used for the networking of the door animator and door functions.
7. /// </summary>
8. public class NetworkUnlockSystem : NetworkBehaviour
9. {
10.     private Animator DoorAnimator;
11.     [SyncVar]private bool DoorOpened;
12.    
13.  
14.      /// <summary>
15.      /// Used for getting the information that we need for each door.
16.      /// </summary>
17.      /// <param name="Opened"></param>
18.      /// <param name="GotAnim"></param>
19.      public void GetDoorState(bool Opened,Animator GotAnim)
20.      {
21.         DoorAnimator = GotAnim;
22.         DoorOpened = Opened;
23.  
24.         CmdOpenDoor(DoorAnimator.GetComponent<NetworkIdentity>().netId,DoorOpened);
25.     }
26.  
27.     [Command]
28.     void CmdOpenDoor(NetworkInstanceId doorId,bool Opened)
29.     {
30.         GameObject doorObj = NetworkServer.FindLocalObject(doorId);
31.         if (doorObj != null)
32.         {
33.             Door door = doorObj.GetComponent<Door>();
34.             if (door != null)
35.             {
36.                 door.openState = Opened;
37.             }
38.         }
39.     }
40.  
41. }

Thanks for all the help provided!!