Bug on a script

[Charlie Fleed]

You know sometimes reading the instructions helps...

# Passive skills can also be used to equip weapons and armors, and to be able to use skills.
# Configure it like that :
#
# When [weapon/armor id] then return [passive skill id]
# When [skill id] then return [passive skill id]
#
# Note that passive skills overwrite class restrictions : if a warrior gets a "Equip Staffs"
# passive ability, he will be able to equip staffs.
#
# On a sad note, all weapons/armors need to be set to a skill in order to be equipped.
# If you put "nil" or "0", the game will crash.
# So set a skill for each different weapon or armor. Default is 100
# The same applies for skills. Skills won't crash, but won't be able to be used. Which is bad.
# However if you put "when skill X then return skill X", the skill will be usable if you have it, which
# means you won't have to learn another passive skill.

Do you see this piece of code in the script?

Code:

def self.req_skill(id)
    case id
      # START Skill Configuration
      when 8 then return 101
    end
    return
  end

You should put all the "dependencies" here, like this:

Code:

def self.req_skill(id)
    case id
      # START Skill Configuration
      when 8 then return 101
      when 1 then return 1 # Heal requires Heal
      when 2 then return 2 # Greater Heal requires Greater Heal
      when 3 then return 2 # Mass Heal requires Greater Heal
    end
    return
  end

However, I suggest to change it a little bit:

Code:

def self.req_skill(id)
    case id
      # START Skill Configuration
      when 8 then return 101
      when 1 then return 1 # Heal requires Heal
      when 2 then return 2 # Greater Heal requires Greater Heal
      when 3 then return 2 # Mass Heal requires Greater Heal
    end
    return id
  end

See the "return id" line? Now the default behavior is that a skill only needs itself to be learned in order to be used.

And if you don't need this feature at all, change it like this:

Code:

def self.req_skill(id)
    return id
  end