The problem resides in these lines:
Code:
case pbHasAnyStatus?
when :POISON then newForm = 1
end
You should use "case" only when you want to check a value and have different outcomes depending on that value. Also note that there is a syntax error; there is no "then" after a "when". If you write on one line, you have to replace "then" with ";".
In your case here, you could write:
Code:
case @status
when :POISON ; newForm = 1
when :BURN ; newForm = 2 # This is an example, you don't need to define a form for burn.
end
Now, the function "pbHasAnyStatus?" only returns true or false if the Pokémon has a status or not, but it doesn't return said status. Now, you want to trigger the ability only if the Pokémon is poisoned, which you can check with the dedicated function "poisoned?":
Code:
if poisoned?
newForm = 1
end
Now here is more information on the code. I don't know which version of Essentials you are using; I'll assume you are using v20.
When defining an ability or an item, it's a good idea to check out the instances of Battle::AbilityEffects (they were called "BattleHandlers" in V18). Battle::AbilityEffects are a very elegant way to define effects that abilities may have (similar code exists for items).
In your example, you want to trigger a form change when your Pokémon gets poisoned. What other abilities are triggered when a status is inflicted? There is one I can think of: Synchronize. So look for the code of Synchronize (on RPG Maker, Ctrl+Shift+F to search all the scripts at once).
You can find its code:
Code:
Battle::AbilityEffects::OnStatusInflicted.add(:SYNCHRONIZE,
proc { |ability, battler, user, status|
# And lots of code
}
}
You can copy this for your "HUNGRYFORNOODLES":
Code:
Battle::AbilityEffects::OnStatusInflicted.add(:HUNGRYFORNOODLES,
proc { |ability, battler, user, status|
# We're filling this code here
}
}
Now AbilityEffects use different keywords compared to the other functions. You shouldn't write "return", but "next".
So, if you want your ability to trigger only when poison is inflicted, you should go "next" if the status is not poison:
Code:
Battle::AbilityEffects::OnStatusInflicted.add(:HUNGRYFORNOODLES,
proc { |ability, battler, user, status|
next if status != :POISON
# We're filling this code here
}
}
And then the rest of your code should fit in, with some adaptation:
Code:
Battle::AbilityEffects::OnStatusInflicted.add(:HUNGRYFORNOODLES,
proc { |ability, battler, user, status| # "user" refers to the Pokémon that is using the move that poisoned you. The targeted Pokémon is "battler"
next if status != :POISON # Don't trigger this code if the status is not poison.
newForm = 1 # The new form is necessarily 1, according to your code.
if battler.form != newForm # don't trigger if the Pokémon has already changed its form
battler.battle.pbShowAbilitySplash(battler)
battler.pbChangeForm(newForm, _INTL("{1} transformed!", battler.pbThis))
battler.battle.pbHideAbilitySplash(battler) # Put this after the code you want to activate, so the player sees the effect before the ability splash disappears.
end
}
}
You should be good to go :)
