Natalia acts as if Y'Berion is alive

[OrcPriestPsychologist]

Describe the bug
Natalia acts as if Y'Berion is alive.

Expected behavior
Natalia doesn't act as if Y'Berian is alive when he is dead.

Steps to reproduce the issue

1. Be in Chapter 3 or further.
2. Visit the temple of the Swamp Camp
3. Look at Natalia's behaviour or check her schedule(J key in the test mode)

Additional context
The function that should switch her behaviour is B_Story_PrepareRitual()

gothic-1-community-patch/scriptbase/_work/Data/Scripts/Content/Story/Chapters/B_Story_PrepareRitual.d

Lines 41 to 42 in 2892a60

	B_ExchangeRoutine (Gur_1205_Chani, "dead");
	B_ExchangeRoutine (Gur_1207_Natalia, "dead");

It works for Chani but doesn't work for Natalia.

I guess the reason is a wrong routine's name in GUR_1207_Natalia.d

gothic-1-community-patch/scriptbase/_work/Data/Scripts/Content/Story/NPC/GUR_1207_Natalia.d

Lines 62 to 66 in 2892a60

	FUNC VOID Rtn_dead_1421()
	{
	TA_Babe_SitAround (00,00,06,00, "PSI_TEMPLE_ROOMS_IN_03");
	TA_Babe_SitAround (06,00,00,00, "PSI_TEMPLE_ROOMS_IN_03");
	};

If I understand correctly how routines work the name should be RTN_dead_1207 because Natalia's id is 1207.

[AmProsius]

It is correct that the routine name does not match the NPC's ID, but that is no error. The routine 1421 is correctly started for NPC Natalia.

Scratch that.

[OrcPriestPsychologist]

The routine 1421 is correctly started for NPC Natalia.

How can I check the running procedure? Because looks like for me she still has the procedure Rtn_start_1421() and not Rtn_dead_1421():

And Chani has the correct schedule:

[AmProsius]

How can I check the running procedure? Because looks like for me she still has the procedure Rtn_start_1421() and not Rtn_dead_1421()

Sorry, there might have been a misunderstanding. Just from the script's point of view, the routine is correctly started with the ID of 1421:

gothic-1-community-patch/scriptbase/_work/Data/Scripts/Content/Story/NPC/GUR_1207_Natalia.d

Line 35 in 2892a60

daily_routine = Rtn_start_1421;

Just to clarify that the routine's ID doesn't necessarily need to match the NPC's ID. But we'll investigate this further.

[OrcPriestPsychologist]

the routine's ID doesn't necessarily need to match the NPC's ID

I think they should match for the case when B_ExchangeRoutine() is called. I assume B_ExchangeRoutine() combines the NPC's ID and a passed string to get the name of a procedure.

gothic-1-community-patch/scriptbase/_work/Data/Scripts/Content/Story/Chapters/B_Story_PrepareRitual.d

Lines 41 to 42 in 2892a60

	B_ExchangeRoutine (Gur_1205_Chani, "dead");
	B_ExchangeRoutine (Gur_1207_Natalia, "dead");

For Chani it does "1205" + "dead" = "Rtn_dead_1205". Then finds the procedure and assigns it.

For Natalia it does "1207" + "dead" = Rtn_dead_1207". Such procedure doesn't exist. So the schedule switch in ch. 3 doesn't happen.

[OrcPriestPsychologist]

Looks similar to #124

[AmProsius]

You are totally right!

Provided fix:

gothic-1-community-patch/scriptbase/_work/Data/Scripts/Content/Story/NPC/GUR_1207_Natalia.d

Line 10 in 2892a60

id = 1207;

changed to

    id          =   1421;

I've checked that no other NPC uses the ID 1421.