Tasharen Entertainment Forum

Support => NGUI 3 Support => Topic started by: Chaosgod_Espér on December 25, 2014, 12:14:11 PM

Title: "Destroying GameObjects immediately is not permitted" - But i don´t destroy!
Post by: Chaosgod_Espér on December 25, 2014, 12:14:11 PM
Hi there..

I want to SetActive a UI GameObject at Runtime. The Lines for that action are:
  1.         void OnTriggerEnter(){
  2.                 if(InteractSymbol != null){
  3.                         NGUITools.SetActive(InteractSymbol, true);
  4.                         InteractSymbol.transform.FindChild("Label").GetComponent<UILabel>().text = cInput.GetText("Interact");
  5.                 }
  6.         }
  7.         void OnTriggerExit(){
  8.                 if(InteractSymbol != null){
  9.                         NGUITools.SetActive(InteractSymbol, false);
  10.                 }
  11.         }

Now on a Testplay, Unity gives me the following Error:
[spoiler]
  1. Destroying GameObjects immediately is not permitted during physics trigger/contact, animation event callbacks or OnValidate. You must use Destroy instead.
  2. UnityEngine.Object:DestroyImmediate(Object)
  3. NGUITools:DestroyImmediate(Object) (at Assets/NGUI/Scripts/Internal/NGUITools.cs:1085)
  4. UIDrawCall:ReleaseInactive() (at Assets/NGUI/Scripts/Internal/UIDrawCall.cs:839)
  5. UIDrawCall:ReleaseAll() (at Assets/NGUI/Scripts/Internal/UIDrawCall.cs:827)
  6. UIPanel:OnDisable() (at Assets/NGUI/Scripts/UI/UIPanel.cs:1014)
  7. UnityEngine.GameObject:SetActive(Boolean)
  8. NGUITools:SetActiveSelf(GameObject, Boolean) (at Assets/NGUI/Scripts/Internal/NGUITools.cs:1272)
[/spoiler]

The problem, as you can see at the Lines, is: I don´t destroy any object. I instantiated and Destroyed the Object, before i changed to SetActive - yes.. But i Changed it, saved it several times now.. But NGUI still thinks i´m destroying the Objects ._.

What´s wrong here?
Title: Re: "Destroying GameObjects immediately is not permitted" - But i don´t destroy!
Post by: pretender on December 25, 2014, 12:19:42 PM
Don't destroy/disable objects in OnTriggerEnter/OnCollisionEnter, make a coroutine to destroy/disable them instead...I believe simple method will work too.
Title: Re: "Destroying GameObjects immediately is not permitted" - But i don´t destroy!
Post by: Chaosgod_Espér on December 25, 2014, 12:21:52 PM
As said - i DON´T Destroy.. i simply change SetActive
Title: Re: "Destroying GameObjects immediately is not permitted" - But i don´t destroy!
Post by: Nicki on December 26, 2014, 04:49:59 PM
SetActive on panels makes the panel destroy the draw call. You still need to delay it until outside the physics call.

Try to do this:

  1. void OnTriggerEnter()
  2. {
  3.         if(InteractSymbol != null)
  4.         {
  5.                 StartCoroutine(DoDelayed(() => {
  6.                         NGUITools.SetActive(InteractSymbol, true);
  7.                         InteractSymbol.transform.FindChild("Label").GetComponent<UILabel>().text = cInput.GetText("Interact");                 
  8.                 }));
  9.         }
  10. }
  11. void OnTriggerExit()
  12. {
  13.         if(InteractSymbol != null)
  14.         {
  15.                 StartCoroutine(DoDelayed(() => {
  16.                         NGUITools.SetActive(InteractSymbol, false);
  17.                 }));
  18.         }
  19. }      
  20. IEnumerator DoDelayed(System.Action action)
  21. {
  22.         yield return null;
  23.         if (action != null) action();
  24. }
  25.  
Title: Re: "Destroying GameObjects immediately is not permitted" - But i don´t destroy!
Post by: Disastercake on December 27, 2014, 02:25:17 AM
Does NGUITools.SetActive have much added functionality now that Unity automatically disables objects under the parent?

It might be cool for NGUITools.SetActive to get some added functionality by doing this sort of IEnumerator yield for you, maybe by passing in an extra bool value to clarify you want it to wait a frame.
Title: Re: "Destroying GameObjects immediately is not permitted" - But i don´t destroy!
Post by: ArenMook on December 27, 2014, 09:28:32 AM
NGUITools.SetActive is there because it was added long before Unity 4, so it's there for legacy purposes more than anything else.