Galathée Posted March 24, 2009 Share Posted March 24, 2009 I hope I'm not pointing out things that have been repeated over and over, but I just finished the French translation of Garrick's files and noticed a couple of coding issues... Here they come ! I hope they're clear like that, as I'm not used to the proper way to refer to them... 1) In the Garrick's part of the pcinit.d file, the GarrickPID2.x scripts, while present, are never accessed : + ~RandomNum(8,1)~ + @3621 + GarrickPID1.1+ ~RandomNum(8,2)~ + @3621 + GarrickPID1.2 + ~RandomNum(8,3)~ + @3621 + GarrickPID1.3 + ~RandomNum(8,4)~ + @3621 + GarrickPID1.4 + ~RandomNum(8,5)~ + @3621 + GarrickPID1.5 + ~RandomNum(8,6)~ + @3621 + GarrickPID1.6 + ~RandomNum(8,7)~ + @3621 + GarrickPID1.7 + ~RandomNum(8,8)~ + @3621 + GarrickPID1.8 + ~RandomNum(8,1)~ + @3622 + GarrickPID1.1 + ~RandomNum(8,2)~ + @3622 + GarrickPID1.2 + ~RandomNum(8,3)~ + @3622 + GarrickPID1.3 + ~RandomNum(8,4)~ + @3622 + GarrickPID1.4 + ~RandomNum(8,5)~ + @3622 + GarrickPID1.5 + ~RandomNum(8,6)~ + @3622 + GarrickPID1.6 + ~RandomNum(8,7)~ + @3622 + GarrickPID1.7 + ~RandomNum(8,8)~ + @3622 + GarrickPID1.8 I believe, considering its content, that the @3622 line should lead to those files instead of the GarrickPID1.1-1.8 ones. 2) Very small thing : in the pcinit.tra file, line @3684, there is a <SIRMADAM> variable that should be <SIRMAAM>, I believe. 3) Another little thing, in the file x#garar.d, the coding for line @21 is written as if spoken by Garrick, while it seems, considering what is said, that Quayle should be the one saying it. == ~%GARRICK_JOINED%~ IF ~InParty("quayle") InMyArea("quayle") !StateCheck("quayle",CD_STATE_NOTVALID)~ THEN @21 And that's all ! Link to comment
cmorgan Posted March 24, 2009 Share Posted March 24, 2009 Awesome reporting - I am establishing the next development version. I can confirm all three of these. Repaired, v18_prerelease \bg1npc_18_prerelease\phase3\pid\x#pcinit.d IF ~IsGabber(Player1) %BGT_VAR%~ THEN BEGIN GACH SAY @3614 ++ @3615 EXIT + ~Global("X#GACHdone","GLOBAL",1) RandomNum(4,1)~ + @3616 + GACHFailure1 + ~Global("X#GACHdone","GLOBAL",1) RandomNum(4,2)~ + @3616 + GACHFailure2 + ~Global("X#GACHdone","GLOBAL",1) RandomNum(4,3)~ + @3616 + GACHFailure3 + ~Global("X#GACHdone","GLOBAL",1) RandomNum(4,4)~ + @3616 + GACHFailure4 + ~!Global("X#GACHdone","GLOBAL",1) RandomNum(5,1)~ + @3617 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) RandomNum(5,2)~ + @3617 + GACHFailure1 + ~!Global("X#GACHdone","GLOBAL",1) RandomNum(5,3)~ + @3617 + GACHFailure2 + ~!Global("X#GACHdone","GLOBAL",1) RandomNum(5,4)~ + @3617 + GACHFailure3 + ~!Global("X#GACHdone","GLOBAL",1) RandomNum(5,5)~ + @3617 + GACHFailure4 + ~!Global("X#GACHdone","GLOBAL",1) CheckStatGT(Player1,15,INT) RandomNum(2,1)~ + @3618 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) CheckStatGT(Player1,15,INT) RandomNum(2,2)~ + @3618 + GACHFailure1 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,INT) RandomNum(5,1)~ + @3618 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,INT) RandomNum(5,2)~ + @3618 + GACHFailure1 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,INT) RandomNum(5,3)~ + @3618 + GACHFailure2 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,INT) RandomNum(5,4)~ + @3618 + GACHFailure3 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,INT) RandomNum(5,5)~ + @3618 + GACHFailure4 + ~!Global("X#GACHdone","GLOBAL",1) CheckStatGT(Player1,15,WIS) RandomNum(2,1)~ + @3619 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) CheckStatGT(Player1,15,WIS) RandomNum(2,2)~ + @3619 + GACHFailure2 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,WIS) RandomNum(5,1)~ + @3619 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,WIS) RandomNum(5,2)~ + @3619 + GACHFailure1 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,WIS) RandomNum(5,3)~ + @3619 + GACHFailure2 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,WIS) RandomNum(5,4)~ + @3619 + GACHFailure3 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,WIS) RandomNum(5,5)~ + @3619 + GACHFailure4 + ~!Global("X#GACHdone","GLOBAL",1) CheckStatGT(Player1,15,CHR) RandomNum(2,1)~ + @3620 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) CheckStatGT(Player1,15,CHR) RandomNum(2,2)~ + @3620 + GACHFailure3 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,CHR) RandomNum(5,1)~ + @3620 + GACHSuccess + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,CHR) RandomNum(5,2)~ + @3620 + GACHFailure1 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,CHR) RandomNum(5,3)~ + @3620 + GACHFailure3 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,CHR) RandomNum(5,4)~ + @3620 + GACHFailure2 + ~!Global("X#GACHdone","GLOBAL",1) !CheckStatGT(Player1,15,CHR) RandomNum(5,5)~ + @3620 + GACHFailure4 + ~RandomNum(8,1)~ + @3621 + GarrickPID1.1 + ~RandomNum(8,2)~ + @3621 + GarrickPID1.2 + ~RandomNum(8,3)~ + @3621 + GarrickPID1.3 + ~RandomNum(8,4)~ + @3621 + GarrickPID1.4 + ~RandomNum(8,5)~ + @3621 + GarrickPID1.5 + ~RandomNum(8,6)~ + @3621 + GarrickPID1.6 + ~RandomNum(8,7)~ + @3621 + GarrickPID1.7 + ~RandomNum(8,8)~ + @3621 + GarrickPID1.8 + ~RandomNum(8,1)~ + @3622 + GarrickPID2.1 + ~RandomNum(8,2)~ + @3622 + GarrickPID2.2 + ~RandomNum(8,3)~ + @3622 + GarrickPID2.3 + ~RandomNum(8,4)~ + @3622 + GarrickPID2.4 + ~RandomNum(8,5)~ + @3622 + GarrickPID2.5 + ~RandomNum(8,6)~ + @3622 + GarrickPID2.6 + ~RandomNum(8,7)~ + @3622 + GarrickPID2.7 + ~RandomNum(8,8)~ + @3622 + GarrickPID2.8 ++ @3623 + GarrickVoice END In bg1npc_18_prerelease\phase2\dlg\x#garar.d CHAIN ~X#GARAR~ ArchGarri @17 == ~%GARRICK_JOINED%~ @18 == ~X#GARAR~ @19 == ~%GARRICK_JOINED%~ @20 == ~%QUAYLE_JOINED%~ IF ~InParty("quayle") InMyArea("quayle") !StateCheck("quayle",CD_STATE_NOTVALID)~ THEN @21 == ~X#GARAR~ @22 == ~%EDWIN_JOINED%~ IF ~InParty("edwin") InMyArea("edwin") !StateCheck("edwin",CD_STATE_NOTVALID)~ THEN @23 == ~%GARRICK_JOINED%~ @24 END IF ~~ THEN DO ~ActionOverride("archibald",GiveItem("X#GARBK",Player1)) SetGlobal("X#GarrickQuest","GLOBAL",1) SetGlobalTimer("X#GarrickBookTalkTimer","GLOBAL",10)~ EXIT bg1npc_18_prerelease\tra\english\x#pcinit_tmp.tra @3684 = ~Why, <SIRMAAM>, I would love to converse, but pressing matters await... ~ A check of the ClanDLAN materials shows they fixed this token during translation. I suspect it was reported, and slipped through my fingers . Searching all translation files in all directories for < to recheck all tokens again across the project. (It will take some time to do that.) For the eventual documentation, <li> <a href="http://forums.gibberlings3.net/index.php?showtopic=17012&st=0&gopid=145893entry145893">Galathée's translation repairs to Garrick's dialog </a> </li> Link to comment
Recommended Posts
Archived
This topic is now archived and is closed to further replies.