Why are you using vLong1 & vShort2? (which should be '==' in an if-statement if that was needed).

if(x == y > z) makes no logical sense

Check the condition example on https://zorro-project.com/manual/en/advisor.htm