fix: set GlobalOther=hero before ZS_* loops and dialog condition evaluation

After commit 69d7707 correctly fixed the GlobalOther save/restore in
ExecutePerception, a stale NPC (e.g. a Shadow guard from a recent perception)
remained in GlobalOther between perception calls. This caused two visible bugs:

1. NPCs turned their backs during conversation - routine states like ZS_*_Loop
   called B_AssessTalk/B_SmartTurnToNpc using 'other', which pointed to the stale
   Shadow instead of the player. NPCs would physically rotate toward that guard.

2. Important dialogs refused to trigger - conditions like Npc_GetDistToNpc(self,other)
   or guild attitude checks ran against the Shadow's position/guild, not the hero's,
   causing them to return false and skip the dialog entirely.

Fix in AiHandler: set GlobalOther=GlobalHero before executing ZS_* state functions.
This is just a sensible default - perception calls override GlobalOther via their
own save/restore, so combat/assess perceptions are unaffected.

Fix in DialogService: save/restore GlobalOther and explicitly set it to GlobalHero
around Info_*_Condition calls. In Gothic's convention, self=NPC and other=hero in
all dialog condition functions.

Author: BoroBongo

Assets/Gothic-Core/Scripts/Adapters/Npc/AiHandler.cs

@@ -79,9 +79,12 @@ private void Update()
             if (Properties.AnimationQueue.Count == 0)
             {
                 // We always need to set "self" before executing any Daedalus function.
+                // "other" defaults to hero here so routine states (ZS_*_Loop) have a sensible fallback.
+                // Perception calls (ExecutePerception) override GlobalOther themselves with their own save/restore.
                 if (NpcInstance != null)
                 {
                     Vm.GlobalSelf = NpcInstance;
+                    Vm.GlobalOther = Vm.GlobalHero;
                 }
 
                 DaedalusSymbol loopSymbol;

Assets/Gothic-Core/Scripts/Services/Player/DialogService.cs

@@ -99,9 +99,12 @@ public void StartDialog(NpcContainer npcContainer, bool initialDialogStarting)
 
                     // TODO - Should be outsourced to some VmManager.Call<int> function which sets and resets values.
                     var oldSelf = _gameStateService.GothicVm.GlobalSelf;
+                    var oldOther = _gameStateService.GothicVm.GlobalOther;
                     _gameStateService.GothicVm.GlobalSelf = npcContainer.Instance;
+                    _gameStateService.GothicVm.GlobalOther = _gameStateService.GothicVm.GlobalHero;
                     var conditionResult = _gameStateService.GothicVm.Call<int>(dialog.Condition);
                     _gameStateService.GothicVm.GlobalSelf = oldSelf;
+                    _gameStateService.GothicVm.GlobalOther = oldOther;
 
                     // Dialog condition is false
                     if (conditionResult == 0)
@@ -146,9 +149,12 @@ private bool TryGetImportant(NpcContainer npcContainer, out InfoInstance item)
 
                 // TODO - Should be outsourced to some VmManager.Call<int> function which sets and resets values.
                 var oldSelf = _gameStateService.GothicVm.GlobalSelf;
+                var oldOther = _gameStateService.GothicVm.GlobalOther;
                 _gameStateService.GothicVm.GlobalSelf = npcContainer.Instance;
+                _gameStateService.GothicVm.GlobalOther = _gameStateService.GothicVm.GlobalHero;
                 var conditionResult = _gameStateService.GothicVm.Call<int>(dialog.Condition);
                 _gameStateService.GothicVm.GlobalSelf = oldSelf;
+                _gameStateService.GothicVm.GlobalOther = oldOther;
 
                 if (conditionResult == 0)
                 {