SET_2DA_ENTRY broken in v230

[plainab]

okay I made this change in a party ai script I made for IWD.  converted it to use in BG (well what I could anyway)

Code: [Select]

COPY_EXISTING ~scrlev.ids~ ~override~
 SET_2DA_ENTRY 4 0 2 ~3~
 SET_2DA_ENTRY 5 0 2 ~4~
 SET_2DA_ENTRY 6 0 2 ~5~
 SET_2DA_ENTRY 7 0 2 ~6~
BUT_ONLY_IF_IT_CHANGESIt runs perfectly fine under weidu version 210 (the version that the IWD script had)

it changes this:

Code: [Select]

0 OVERRIDE
1 AREA
2 SPECIFICS
4 CLASS
5 RACE
6 GENERAL
7 DEFAULT
into this:

Code: [Select]

0 OVERRIDE
1 AREA
2 SPECIFICS
3 CLASS
4 RACE
5 GENERAL
6 DEFAULT

but when it updates to version 230 I receive this error:

Code: [Select]

ERROR: Cannot find 7 rows with at least 2 columns.ERROR: [scrlev.ids] -> [override] Patching Failed (COPY) (Failure("Cannot Set 2DA Entry"))
If I change the code to this:

Code: [Select]

COPY_EXISTING ~scrlev.ids~ ~override~
 SET_2DA_ENTRY 3 0 2 ~3~
 SET_2DA_ENTRY 4 0 2 ~4~
 SET_2DA_ENTRY 5 0 2 ~5~
 SET_2DA_ENTRY 6 0 2 ~6~
BUT_ONLY_IF_IT_CHANGESit will process but changes it incorrectly to this:

Code: [Select]

0 OVERRIDE
1 AREA
3 SPECIFICS
4 CLASS
5 RACE
6 GENERAL
7 DEFAULT
So it obviously isn't an issue where it is counting from 0 instead of 1 since telling it to change line 3 instead of 4 changed the third line.  The issue seems to be that it isn't recognizing the last line (in this case the 7th).  I've even tried it with a column count of 1 and it still errors out on line 7.

the reason I correct the script level numbers is because I actually assign scripts via script to the class and race levels on the party members and if the numbers are not correct they get assigned incorrectly... (tempted to drop it into the fixpack but can't cause of this...)

I don't know which version since 210 it crapped out on, but can we get this to work properly again? 

[the bigg]

The file doesn't have 8 lines, so SET_2DA_ENTRY 7 0 2 must fail. It appears that the second (and correct) code doesn't work because the file begins with an empty line. SET_2DA_ENTRY_LATER/NOW works correctly.

[plainab]

are you saying to use SET_2DA_ENTRY_NOW rather than just SET_2DA_ENTRY?



and on a side note...  the scrlvl.ids file when 'fixed' in BG:ToTSC actually breaks and scripts don't get set right but in IWD1 it has to be fixed to get the correct script slots.  go figure!

[the bigg]

Fixed in git.