The Validator - Find errors quickly and with minimal pain!

[Jamie550]

Near-repeat of a previous error:

		modifier = {
			factor = 0.5
			NOT = { culture = liege }
		}

Fixed (along with culture_group and conquest_culture)

 

[richvh]

Found a false negative. Ran into a [Liege.Getxxx] bracket command in an integrated mod that didn't print out anything, and searching for examples in Vanilla, it seems that Liege.Getxxx needs to be preceded by Root., From(From)., or This. before it can resolve.

 

[Jamie550]

Found a false negative. Ran into a [Liege.Getxxx] bracket command in an integrated mod that didn't print out anything, and searching for examples in Vanilla, it seems that Liege.Getxxx needs to be preceded by Root., From(From)., or This. before it can resolve.

Currently the Validator just breaks each bracket command by dots and sees that each part is valid. Is it worth it to do more? Because it seems like some things only work in certain contexts, etc, and it would be a bit messy to handle (but doable, if worthwhile).

 

[Korbah]

Is there a chance you could add the little snippets to the first post on how to get the validator to load up notepad++ to the correct line that has the error?
On a new PC with a new installation of the validator without that command >.<

 

[richvh]

--- Error 1 of 1 ---
At <mod>\events\waylit_siege_pulse_events.txt [character_event[2]\option[3]\random_list\10\ROOT\death\death_reason] (Line 471, Column 22):
"death_escape" is not a valid DeathReason.

		ROOT = { death = { death_reason = death_escape } }

death_escape;Died attempting a daring escape;;;;;;;;;;;;;;x

As far as I can tell, death reason just sets up a localization key, so rather than checking death_reason against a fixed set of death reasons, you should check for a matching localization key.

 

[Wizzington]

In my experience death reasons are hardcoded and you can't add new ones. Have you actually tested it?

 

[richvh]

Hm, you're right, all the vanilla death_reasons are in the executable, I must have been thinking of death_by_trait which is extensible by adding in new traits.

 

[richvh]

common/retinue_subunits needs to be scanned for localization tags.

 

[Korbah]

Very small false positive, you can use short_name = yes in baronies, it's how I have "Arcane University" as a holding in the imperial city rather than "Temple of Arcane University".

 

[Jamie550]

common/retinue_subunits needs to be scanned for localization tags.

Fixed.

Very small false positive, you can use short_name = yes in baronies, it's how I have "Arcane University" as a holding in the imperial city rather than "Temple of Arcane University".

Fixed.

Is there a chance you could add the little snippets to the first post on how to get the validator to load up notepad++ to the correct line that has the error?
On a new PC with a new installation of the validator without that command >.<

Path: D:\Edit\Npp592\Notepad++.exe
Command line: {0} -n{1}
is what I have

 

[Jamie550]

v1.2 RC9 has been released
Download here: http://www.mediafire.com/?h3u09j0y8teui56

 

[Korbah]

There's a false positive for "active = yes/no" when validating diseases.

 

[richvh]

Also for second_graphical_culture at the culture_group level.

 

[Korbah]

And for

modifier = {
factor = 0.5 # Slow it down a bit
}

That appears ALOT in the vanilla factions decisions file, but validator reports it as requiring at least one non-factor element. I have no idea if it even does anything without another element, but thought I'd mention it.

 

[Jamie550]

Also for second_graphical_culture at the culture_group level.

Fixed

There's a false positive for "active = yes/no" when validating diseases.

Fixed

And for


That appears ALOT in the vanilla factions decisions file, but validator reports it as requiring at least one non-factor element. I have no idea if it even does anything without another element, but thought I'd mention it.

Apparently it does, and for some reason they don't just change the overall factor to speed up/slow down the event, for no known reason, even though it would be much clearer, so soon that check will be off by default

 

[Jamie550]

v1.2 RC10 has been released.
Download here: http://www.mediafire.com/?pyf720lef1qfiit

 

[tompalmer]

Wishlist : A way to disable one type of error, the bad thing with your program is to have thousands of errors of something WAD for us. (ex : X is 1700 Years old ; normal we start in 8300 and end in 9999)

So please

And also changelog ?

Thank you Jamie550

 

[Meneth]

Wishlist : A way to disable one type of error, the bad thing with your program is to have thousands of errors of something WAD for us. (ex : X is 1700 Years old ; normal we start in 8300 and end in 9999)

So please

And also changelog ?

Thank you Jamie550

You can change the ages that are considered "normal" in the settings file.

 

[Korbah]

false positives:
levy_size_reinforce_rate (is vanilla code)
short_name = yes (in baronies, already reported this but it seems to have crept back in to the latest RC)

 

[Jamie550]

false positives:
levy_size_reinforce_rate (is vanilla code)
short_name = yes (in baronies, already reported this but it seems to have crept back in to the latest RC)

Can you post both errors please?