This spell has no cooldown

[wafuboe]

Hello Otland.
I recently installed this spell to my server, but the problem i encounter is that the player can use it everytime it wants no cooldown, the script just goes faster and the server crashes.

What it does:
It teletransport to all visible creatures and deals damage while being invulnerable to it while the spell is cast.

local function targetEffect(cid)
    local player = Player(cid)
    if not player then
        return
    end
    local effect =  CONST_ANI_REDSTAR
    local orig = player:getPosition()
    local d1, d2 = {z = orig.z}, {z = orig.z}
    d1.x = orig.x - 5
    d2.x = orig.x + 5
    for i = -2, 2 do
        d1.y = orig.y + i
        d2.y = d1.y
        orig:sendDistanceEffect(d1, effect)
        orig:sendDistanceEffect(d2, effect)
    end
    d1.y = orig.y - 3
    d2.y = orig.y + 3
    for i = -4, 4 do
        d1.x = orig.x + i
        d2.x = d1.x
        orig:sendDistanceEffect(d1, effect)
        orig:sendDistanceEffect(d2, effect)
    end
end
local function backOldPosition(cid, oldPosition)
    local player = Player(cid)
    if not player then
        return
    end
  
    player:teleportTo(oldPosition)
    player:setGhostMode(false)
end
local function jumpEffect(cid, target)
    local player = Player(cid)
    if not player then
        return
    end
    local target = Creature(target)
    if target then
        player:getPosition():sendDistanceEffect(target:getPosition(), CONST_ANI_EXPLOSION)
    end
end
local function jumpOnTarget(cid, target, targetCount, oldPosition)
    local player = Player(cid)
    if not player then
        return
    end
    local target = Creature(target)
    if target then
        addEvent(targetEffect, 100, cid)
        player:teleportTo(target:getPosition())
        player:setGhostMode(true)
        doTargetCombatHealth(cid, target, COMBAT_PHYSICALDAMAGE, -400, -500, CONST_ME_ASSASSIN)
        if targetCount == 0 then
            addEvent(backOldPosition, 400, cid, oldPosition)
        end
    end
end
local function extractRandomValuesFromTable(tbl) --By Printer(This will make sure not to select a value twice from the table)
    if #tbl == 0 then
        return false
    end
          return table.remove(tbl, math.random(#tbl))
end
function onCastSpell(creature, var)
    local targets = {}
    local playerPos = creature:getPosition()
    local spectators = Game.getSpectators(playerPos, false, false, 0, 10, 0, 10)
    for i = 1, #spectators do
        local specs = spectators[i]
        if specs ~= creature and not specs:isNpc() then
            targets[#targets+1] = specs
        end
    end
    if #targets == 0 then
        creature:sendCancelMessage('There is no targets in sight.')
        playerPos:sendMagicEffect(CONST_ME_POFF)
        return false
    end
    local targetCount = #targets
    for i = 1, #targets do
        local randTarget = extractRandomValuesFromTable(targets)
        addEvent(jumpOnTarget, i * 400, creature:getId(), randTarget:getId(), targetCount - i, playerPos)
        addEvent(jumpEffect, i * 300, creature:getId(), randTarget:getId())
    end
    return false
end

<instant group="attack" spellid="24" name="Multi Shunpo" words="exevo gran shen" lvl="60" mana="400" prem="1" cooldown="40000" groupcooldown="4000" selftarget="1" needlearn="0" script="attack/multi shunpo.lua">
        <vocation name="Ninja" />
        <vocation name="Silent Ninja" />
        </instant>

 

[Xikini]

The spell only returns false, never true.

 

[wafuboe]

The spell only returns false, never true.

should i replace all flase with true?

 

[Restles]

No. change line 88

return false

to

return true