Project Ironfist Scripting Documentation Page

On the Scripting Documentation page:

This is specified for "GetSecondarySkill":

GetSecondarySkill(hero, skill, level)
hero: The hero to get the secondary skill of. (See, for example, GetHero).
skill: The ID of the secondary skill. (See section "Skills.")

Returns a number 0-3 for None, Basic, Advanced, and Expert, respectively, representing the level of the skill the hero has.

*

I think "level" is not necessary in the expression.

GetSecondarySkill(hero, skill) should be sufficient. And it works this way in scripts.

*

In category "Locations":

For the "Shrine of the First Circle", the location ID is: "LOCATION_SHRINE_FIRST".

For the "Shrine of the Second Circle", the location ID is: "LOCATION_SHRINE_SECOND_ORDER".

For the "Shrine of the Third Circle", the location ID is: "LOCATION_SHRINE_THIRD_ORDER".

Should it be "LOCATION_SHRINE_FIRST_ORDER" for the "Shrine of the First Circle"?

***** Edited *****

A few remarks:

In category "Locations":

A word "misspelled":

LOCATION_DESRT_TENT --> LOCATION_DESERT_TENT

Currently, the expression "LOCATION_DESRT_TENT" works normally written this way in scripts.

Locations missing (PoL):

- Air Altar
- Fire Altar
- Water Altar
- Earth Altar
- Barrow Mounds
- Hut of the Magi
- Eye of the Magi
- Stables
- Arena
- Mermaid
- Sirens

*

Resources to be separated:

LOCATION_RESOURCE_WOOD
LOCATION_RESOURCE_MERCURY
LOCATION_RESOURCE_ORE
LOCATION_RESOURCE_SULFUR
LOCATION_RESOURCE_CRYSTAL
LOCATION_RESOURCE_GEMS
LOCATION_RESOURCE_GOLD

*

Mines to be separated:

LOCATION_MINE_ORE
LOCATION_MINE_SULFUR
LOCATION_MINE_CRYSTAL
LOCATION_MINE_GEMS
LOCATION_MINE_GOLD

Noted.