YEP_Z_PassiveCases

[Frostorm]

So I'm implementing a custom passive case for this plugin (the lunatic mode added before the following)
// -------------------------------
// Add new effects above this line
// -------------------------------

I wanted to ask if my code was correct since I was mostly just following the provided syntax...

} else if (data.match(/ONE[ ]HANDED)) {
  if (user.isActor()) {
    condition = !user.equips()[1];
  } else {
    condition = false;
}

So in the state notebox I would put:

<Passive Condition Cases>
One Handed
</Passive Condition Cases>

Correct?

 

[caethyril]

For this line:

} else if (data.match(/ONE[ ]HANDED)) {

I would suggest this instead:

} else if (data.match(/ONE HANDED/i)) {

Explanation: /ONE HANDED/i is a regular expression. (You forgot to put a / at the end of yours, which will cause an error.) It represents the exact sequence of characters "ONE HANDED". The i immediately after the expression is called a "flag": in this case, it means "case-insensitive", i.e. this will match anything like one handed, ONE HANDED, One Handed, etc. More details on regular expressions can be found here:

Regular expressions

Regular expressions are patterns used to match character combinations in strings. In JavaScript, regular expressions are also objects. These patterns are used with the exec() and test() methods of RegExp, and with the match(), matchAll(), replace(), replaceAll(), search(), and split() methods of...

developer.mozilla.org

 

[Frostorm]

@caethyril Thank you for pointing out my mistakes.