PDA

View Full Version : quest_globals


Netvoid
12-01-2007, 09:50 AM
Seems that post version 1062 the perl variables that contain the flag values for a character are all = undef.

When I downgrade to 1062 all is fine, but if I go to 1064 or 1065 npc's will only respond if the script has no reference to character flags from quest_globals.

This just me or anyone else noticing this?

Thanks

cavedude
12-01-2007, 10:59 AM
KLS: Fixed: qglobals will now not be exported for npcs that do not have the qglobal flag set in the perl parser.

Meaning, any NPC that uses a global has to have a 1 in its qglobal column in npc_types in order to use it. This breaks some stuff on PEQ, but I already have PoP working properly again, and everything else will be corrected as the reports come in.

The fixes will be in the next CVS dump of the db.

The reason this was fixed was simply to save on CPU load. Before, EVERY NPC could accept a global, now only the ones specified will.

Netvoid
12-01-2007, 11:14 AM
Thanks for the heads up on this, didn't realize the association.

Netvoid
12-02-2007, 12:00 PM
For anyone interested until Cavedude releases next PEQ this should turn the global vars back on for POP.


UPDATE npc_types SET qglobal = 1 WHERE name IN
(
'Elder_Poxbourne',
'Elder_Fuirstel',
'Barban_Cryptwalker',
'#Mavuin',
'#Thelin_Poxbourne',
'#Tylis_Newleaf',
'__Thelin_Poxbourne',
'A_Planar_Projection',
'Adler_Fuirstel',
'Adroha_Jezith',
'Askr_the_Lost',
'factory_access',
'Fahlia_Shadyglade',
'Giwin_Mirakon',
'Grand_Librarian_Maelin',
'Karana',
'Miak_the_Searedsoul',
'Sarhya_the_Dawnbreeze',
'Seer_Mal_Nae-Shi',
'Tylis_Newleaf'
)


Hope someone finds it handy.

cavedude
12-03-2007, 12:52 AM
There are more than that, I don't have names off-hand but I do know there should be at least 10-15 more. But, I plan on updating CVS today so it's a moot point.

Netvoid
12-03-2007, 03:16 PM
Yeah, figured a bit might be missed.. All I did was build a quick list of the quests files that had a reference to $pop* and scriped those results.

Anything not referring to $pop* was not included... Cool to hear of another release though.

cavedude
12-04-2007, 08:31 AM
CVS just updated.