Comments

• ChrisIceBox

  September 2024

  Open up AC's StateHandler script and replace line 536:

  if (KickStarter.playerInteraction.InPreInteractionCutscene) return GameState.Cutscene;
  

  with:

  if (KickStarter.playerInteraction && KickStarter.playerInteraction.InPreInteractionCutscene) return GameState.Cutscene;
  

  Does that resolve it?
• stelabouras

  September 2024

  Unfortunately not, if a nullability check is added for playerInteraction then a NullReferenceException is produced because of the checks that follow right after that:

  if (KickStarter.actionListManager.IsGameplayBlocked ())

  and

  if (KickStarter.playerInput.IsInConversation (true))

  and even if I add nullability checks there for actionListManager and playerInput, then I get a NullReferenceException in AC_Trigger line 296:

  KickStarter.playerInteraction.DeselectHotspot (false);

  I am wondering whether it makes sense for the gameState getter to return the GameState.Normal value if a manager instance is not available, instead of GameState.Paused.

  I don't know how this change will affect other AC subsystems, but if the gameState getter had something like:

  if (KickStarter.playerInteraction == null) return GameState.Paused;

  then the AC_Trigger IsObjectCorrect() method would return false and wouldn't call its Interact() method.

  Here's my modified version of the gameState getter that doesn't produce any errors:

          /** The current state of the game (Normal, Cutscene, Paused, DialogOptions) */
          public GameState gameState
          {
              get
              {
                  if (inScriptedPause) return GameState.Paused;
  
                  if (KickStarter.playerMenus.ArePauseMenusOn ())
                  {
                      if (KickStarter.actionListManager.IsGameplayBlockedAndUnfrozen ())
                      {
                          return GameState.Cutscene;
                      }
                      return GameState.Paused;
                  }
  
                  if (inScriptedCutscene) return GameState.Cutscene;
                  if (KickStarter.mainCamera && KickStarter.mainCamera.IsShowingForcedOverlay ()) return GameState.Cutscene;
                  if (KickStarter.playerInteraction == null) return GameState.Paused;
                  if (KickStarter.playerInteraction.InPreInteractionCutscene) return GameState.Cutscene;
  
                  if (KickStarter.actionListManager == null) return GameState.Paused;
                  if (KickStarter.actionListManager.IsGameplayBlocked ())
                  {
                      return GameState.Cutscene;
                  }
  
                  if (KickStarter.playerInput == null) return GameState.Paused;
                  if (KickStarter.playerInput.IsInConversation (true))
                  {
                      return GameState.DialogOptions;
                  }
                  return GameState.Normal;
              }
          }
  
• stelabouras

  September 2024 edited September 2024

  Another way this can be approached without altering the gameState getter is having the AC_Trigger class listen for the OnBeforeChangeScene and maybe disable itself so it doesn't listen for the produced trigger events (e.g. OnTriggerExit2D), given that based on my tests the OnBeforeChangeScene event is produced before the OnTriggerExit2D one.

  Update:

  Turns out disabling the AC_Trigger component (enabled = false) or even de-activating the whole gameobject (gameObject.SetActive(false);) doesn't prevent the OnTrigger* events from being produced, so extra checks need to be added there to ensure that the component / GameObject are disabled before proceeding into calling the Interact() method. Strange.
• ChrisIceBox

  September 2024

  I would like to recreate the original issue before delving further into a potential fix, but I cannot so far.

  Can you PM me your Managers + scene file to test directly?
• stelabouras

  September 2024

  I will attempt to replicate the issue on the demo project and I will PM you the files.