Code:
if((containi(Sprava, "q") != -1) ||
(containi(Sprava, "w") != -1) ||
(containi(Sprava, "e") != -1) ||
(containi(Sprava, "r") != -1) ||
(containi(Sprava, "t") != -1) ||
(containi(Sprava, "z") != -1) ||
(containi(Sprava, "u") != -1) ||
(containi(Sprava, "i") != -1) ||
(containi(Sprava, "o") != -1) ||
(containi(Sprava, "p") != -1) ||
(containi(Sprava, "a") != -1) ||
(containi(Sprava, "s") != -1) ||
(containi(Sprava, "d") != -1) ||
(containi(Sprava, "f") != -1) ||
(containi(Sprava, "g") != -1) ||
(containi(Sprava, "h") != -1) ||
(containi(Sprava, "i") != -1) ||
(containi(Sprava, "k") != -1) ||
(containi(Sprava, "l") != -1) ||
(containi(Sprava, "m") != -1) ||
(containi(Sprava, "n") != -1) ||
(containi(Sprava, "b") != -1) ||
(containi(Sprava, "v") != -1) ||
(containi(Sprava, "c") != -1) ||
(containi(Sprava, "y") != -1) ||
(containi(Sprava, "x") != -1))
Really? Why not just check if the amount is numeric?
Code:
if(equali(Sprava,"ALL") || equali(Sprava,"all"))
This + the else will never be called because you're preventing the rest of the code from executing by doing what you did above.