STATE_REALLY_DEAD in UDMPSWN.BCS is not reliable

Comments

• Erg Member Posts: 1,756

  August 2015 edited August 2015

  Yes, the block should be moved from the dying creature to somewhere else, e.g. the area script, as you did or as I did here. In my case, the Dead() command was enough, but it may be preferable to add additional checks (STATE_REALLY_DEAD, etc.) just in case.
  
  Concerning reproducibility, see this or, for the Enhanced Editions, this.
  
  Edit: Sorry, I didn't take in account that there are several differences between your case and mine. In particular, in my case the creature was supposed to die only once. In your case Dead() will not work because it will return TRUE always once the first instance is dead. I think you should use Died("udmind10") instead.
  

  Post edited by Erg on August 2015

  0
• swit Member, Translator (NDA) Posts: 495

  August 2015 edited August 2015

  one more thing - in BG2 this script block in UDMPSWN.BCS looks like this:

  
  IF
    Die()
  THEN
    RESPONSE #100
      SetGlobalTimer("SpawnIllTimer","GLOBAL",TEN_ROUNDS)
      SetGlobal("spawnIll","AR2400",0)
  END
  

  which means that there should be infinite Mind Flyer spawn, not only 1 respawn like in BG2:EE. Not sure why CDDieReplace has been added to the script. That is why some people are confused with current behaviour:
  https://forums.beamdog.com/discussion/25072/i-ran-out-of-mind-flayers
  https://forums.beamdog.com/discussion/19686/mind-flayer-tunnels-spoilers-question
  https://forums.beamdog.com/discussion/28357/trapped-in-illithid-layer-bug-spoiler
  

  1
• Erg Member Posts: 1,756

  August 2015 edited August 2015

  swit said:

  Not sure why CDDieReplace has been added to the script.

  Yes, it looks like a mistake. That single Mind Flyer is supposed to keep respawning, so players don't necessarily have to transform into the Slayer, if they want to avoid the reputation loss.
  
  Edit: in theory CDDieReplace should work if used locally to the creature, because it will be reset to zero once the creature is respawned, but it will not work if used locally to the area unless you add SetGlobal("CDDieReplace","AR2400",0) to the area script when the creature is respawned. Also see my edit above about replacing Dead() with Died().
  
  I cannot test in BGII:EE as I don't have it, but something like this should work:

  IF
    Global("CDDieReplace","AR2400",0)
    OR(2)
      StateCheck("udmind10",STATE_REALLY_DEAD)
  Died("udmind10")
  THEN
    RESPONSE #100
      SetGlobalTimer("SpawnIllTimer","GLOBAL",TWELVE_MINUTES)
      SetGlobal("spawnIll","AR2400",0)
      SetGlobal("CDDieReplace","AR2400",1)
  END

  followed but something like

  IF
  	GlobalTimerExpired("SpawnillTimer","GLOBAL")
  	GlobalGT("udMind","GLOBAL",17)
  	Global("spawnill","AR2400",0)
  THEN
  	RESPONSE #100
  		SetGlobal("spawnill","AR2400",1)
  SetGlobal("CDDieReplace","AR2400",0)
  		CreateCreatureDoor("udmind10",[4195.1681],4)
  END
  

  Post edited by Erg on August 2015

  0
• Erg Member Posts: 1,756

  August 2015 edited August 2015

  One more thing:
  
  the modified script has now both variables "spawnill" and "CDDieReplace" on AR2400, but only one should be really required.
  
  Besides, maybe I'm overthinking it, but I'm not sure it is a good idea to use STATE_REALLY_DEAD here (IESDP says that CreateCreatureDoor requires 3.5 seconds for the creature to appear, and I'm not sure what StateCheck("udmind10",STATE_REALLY_DEAD) would return in that interval) and Died() should be enough, so maybe something like this in the area script is better:

  IF
    Global("CDDieReplace","AR2400",0)
    Died("udmind10")
  THEN
    RESPONSE #100
      SetGlobalTimer("SpawnIllTimer","GLOBAL",TWELVE_MINUTES)
      SetGlobal("CDDieReplace","AR2400",1)
  END
  
  IF
    GlobalTimerExpired("SpawnillTimer","GLOBAL")
    GlobalGT("udMind","GLOBAL",17)
    Global("CDDieReplace","AR2400",1)
  THEN
    RESPONSE #100
      SetGlobal("CDDieReplace","AR2400",0)
      CreateCreatureDoor("udmind10",[4195.1681],4)
  END

  0