Jump to content

[solved] BJaheir 68: ADD_TRANS_ACTION instead of ALTER_TRANS


Recommended Posts

EDIT: ah, don't bother. BGII:EE uses DO ~IncrementGlobal("LoveTalk","LOCALS",1), too, so people need to know what they are doing, anyways.

 

 

--

 

BJaheir state 68 misses the closing variable. The dlg state starts with:

 

IF WEIGHT #0 ~Global("JaheiraMatch","GLOBAL",0)
Global("LoveTalk","LOCALS",8)
!StateCheck(Player1,STATE_SLEEPING)~ THEN BEGIN 68 // from:
SAY #20307 /* ~I will not tolerate this waste of time. Let's get moving! We've much to do!~ [JAHEIR69] */

 

 

Fixpack uses this in soa-dlg.d:

 

ALTER_TRANS bjaheir BEGIN 68 END BEGIN 0 1 2 END BEGIN
"ACTION" ~IncrementGlobal("LoveTalk","LOCALS",1)~
END

 

 

This is a nitpick, but would you change this to

 

ADD_TRANS_ACTION bjaheir
BEGIN 68 END
BEGIN END
~SetGlobal("LoveTalk","LOCALS",9)~

 

instead.

1. If a mod uses the current Fixpack fix, any added transactions by other mods are lost.

2. If a modder notices this but sticks to the ~IncrementGlobal("LoveTalk","LOCALS",1)~ this leads to hickups if more than one mod tries to close the variable (or does it on top of Fixpack).

Link to comment

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...