Undesirable call of Toggle's "On Value Changed" when "isOn" is changed through script

Question

I'm saving the user's value of "isOn" for various toggle states, and then setting those toggle states during start.

Each toggle has a method I want called when the user changes the value of the toggle, but not when the value is changed via script. However, it seems as though the "On Value Changed" event gets called when I change the value through script.

Here is a sample of the code that gets called in start to set the toggle

     private void Start()
     {  
         if (PlayerPrefs.HasKey("rememberAuth"))
         {
             rememberAuth = PlayerPrefsX.GetBool("rememberAuth");
             rememberToggle.isOn = rememberAuth;                
         }
         else
         {
             PlayerPrefsX.SetBool("rememberAuth", rememberAuth);
         }
     }

And here is a sample of the method that gets called by the "on value changed" event.

     public void RememberMeToggle()
     {
         if(rememberToggle.isOn)
         {
             SaveAuth();
         }
         else if (!rememberToggle.isOn)
         {
             DeleteAuth();
         }
         PlayerPrefsX.SetBool("rememberAuth", rememberAuth);
         PlayerPrefs.Save();
     }

Now, if the bool value of the toggle is different from the default value that was set in the editor, the RememberMeToggle() method gets called even though I don't want it to. T$$anonymous$$s can cause some issues.

I've found a solution for t$$anonymous$$s, and that is to remove the method call from the "On Value Changed" Event, in the inspector and then add an "Event Trigger" Component with a "Pointer Click" event and use that instead.

Is t$$anonymous$$s what I'm supposed to do? Is there a better option? Is the function of "On Value Changed" being trigger from the script desirable and intended?

I had a lot of trouble finding a solution through searches, so if I am doing t$$anonymous$$s right, even though I answered my own question, it might help someone else who's searc$$anonymous$$ng.

Answer 1

Best Answer

Answer by Hellium · Jun 19, 2018 at 05:05 PM

Here are two useful functions you can call to "turn off" and "turn on" the persistent listeners of a UnityEvent (such as toggle.onValueChanged)

 public void Mute( UnityEngine.Events.UnityEventBase ev )
 {
     int count = ev.GetPersistentEventCount();

     for ( int i = 0 ; i < count ; i++ )
     {
         ev.SetPersistentListenerState( i, UnityEngine.Events.UnityEventCallState.Off );
     }
 }
 
 public void Unmute( UnityEngine.Events.UnityEventBase ev )
 {
     int count = ev.GetPersistentEventCount();

     for ( int i = 0 ; i < count ; i++ )
     {
         ev.SetPersistentListenerState( i, UnityEngine.Events.UnityEventCallState.RuntimeOnly );
     }
 }

In your code:

   private void Start()
  {  
      if (PlayerPrefs.HasKey("rememberAuth"))
      {
          rememberAuth = PlayerPrefsX.GetBool("rememberAuth");
          Mute( rememberToggle.onValueChanged ) ;
          rememberToggle.isOn = rememberAuth;  
          Unmute( rememberToggle.onValueChanged ) ;              
      }
      else
      {
          PlayerPrefsX.SetBool("rememberAuth", rememberAuth);
      }
  }

2 Show 7 · Share

Fragmental · Jun 20, 2018 at 05:34 AM 0

Share

Thank you for your detailed response!

Fragmental · Jun 20, 2018 at 05:37 AM 0

Share

What are the for loops for in the mute/unmute methods?

Hellium Fragmental · Jun 20, 2018 at 06:06 AM 0

Share

The for loop retrieve each callback specified in the event, in the inspector (the persistent listener)

If the callbacks are attached using code, you will have to call toggle.onValueChanged.RemoveListener( function ) instead

Fragmental Hellium · Jun 20, 2018 at 06:49 AM 0

Share

Oh right. Of course. I forgot about adding multiple callbacks on the event. Thank you.

drebleezle · Apr 09, 2020 at 07:33 PM 0

Share

Thanks for the answer! Any other solutions since June 2018?

Hellium drebleezle · Apr 09, 2020 at 07:38 PM 0

Share

Doesn't the code works? I think the code is still valid today.

ramishaziev drebleezle · Apr 16, 2020 at 01:45 PM 1

Share

You can use Toggle.SetIsOnWithoutNotify if you want set isOn without invoking onValueChanged callback.

Answer 2

Answer by Speedomon · Apr 11, 2022 at 05:03 PM

Fortunately there is an easier way now: SetIsOnWithoutNotify

2 · Share

Undesirable call of Toggle's "On Value Changed" when "isOn" is changed through script

2 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions