Solved TFS 1.2 8.6 Boss arena

[ssiwyy158]

Hello.
When the event ends an error appears in the console:

bossarenalib.lua code :

bossarenaconfig = { minPlayers = 1, --Min players req to start the event - Pastebin.com

Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.

pastebin.com

 

[Lessaire]

ok, first for my sanity, a refactor to kill the super long line

-    local specs = Game.getSpectators(bossarenaconfig.waitingRoomPosition.centerPos, false, true, 0, bossarenaconfig.waitingRoomPosition.radiusX, 0, bossarenaconfig.waitingRoomPosition.radiusY)
+    local lobby = bossarenaconfig.waitingRoomPosition
+    local specs = Game.getSpectators(lobby.centerPos, false, true, 0, lobby.radiusX, 0, lobby.radiusY)

ok,

function resetEvent() --  <---- in this function
    local lobby = bossarenaconfig.waitingRoomPosition
    local specs = Game.getSpectators(lobby.centerPos, false, true, 0, lobby.radiusX, 0, lobby.radiusY)
    if #specs >= 1 then
        for i = 0, #specs do
            if specs[i]:getStorageValue(bossarenaconfig.joinedStorage) == 1 then
                specs[i]:teleportTo(specs[i]:getTown():getTemplePosition())
                specs[i]:setStorageValue(bossarenaconfig.joinedStorage, 0)
            end
        end
    end
--  in this block ⇲
    local specs2 = Game.getSpectators(Position(528, 776, 7), false, true, 39, 39, 55, 55)
    if #specs2 >= 1 then
        for i = 0, #specs2 do
--          this invocation is incorrect
            if specs2[i]:getStorageValue(bossarenaconfig.joinedStorage) == 1 then
                specs2[i]:teleportTo(specs2[i]:getTown():getTemplePosition())
                specs2[i]:setStorageValue(bossarenaconfig.joinedStorage, 0)
            end
        end
    end

My initial suspicion is this programmer is not familiar with Lua very well and that i = 1 because Lua is 1-indexed not 0-indexed, but then previous loop should have broke first... so my second suspicion is about this targeting hardcoded positions, but that could be some real-map thing.

Ok, I searched GitHub and see lots of i = 1 loops when it comes to Game.getSpectators so I would go with that. Change all the get spectator for-loops in that function to use i = 1.

Here you go.

 

[ssiwyy158]

ok, first for my sanity, a refactor to kill the super long line

-    local specs = Game.getSpectators(bossarenaconfig.waitingRoomPosition.centerPos, false, true, 0, bossarenaconfig.waitingRoomPosition.radiusX, 0, bossarenaconfig.waitingRoomPosition.radiusY)
+    local lobby = bossarenaconfig.waitingRoomPosition
+    local specs = Game.getSpectators(lobby.centerPos, false, true, 0, lobby.radiusX, 0, lobby.radiusY)

ok,

function resetEvent() --  <---- in this function
    local lobby = bossarenaconfig.waitingRoomPosition
    local specs = Game.getSpectators(lobby.centerPos, false, true, 0, lobby.radiusX, 0, lobby.radiusY)
    if #specs >= 1 then
        for i = 0, #specs do
            if specs[i]:getStorageValue(bossarenaconfig.joinedStorage) == 1 then
                specs[i]:teleportTo(specs[i]:getTown():getTemplePosition())
                specs[i]:setStorageValue(bossarenaconfig.joinedStorage, 0)
            end
        end
    end
--  in this block ⇲
    local specs2 = Game.getSpectators(Position(528, 776, 7), false, true, 39, 39, 55, 55)
    if #specs2 >= 1 then
        for i = 0, #specs2 do
--          this invocation is incorrect
            if specs2[i]:getStorageValue(bossarenaconfig.joinedStorage) == 1 then
                specs2[i]:teleportTo(specs2[i]:getTown():getTemplePosition())
                specs2[i]:setStorageValue(bossarenaconfig.joinedStorage, 0)
            end
        end
    end

My initial suspicion is this programmer is not familiar with Lua very well and that i = 1 because Lua is 1-indexed not 0-indexed, but then previous loop should have broke first... so my second suspicion is about this targeting hardcoded positions, but that could be some real-map thing.

Ok, I searched GitHub and see lots of i = 1 loops when it comes to Game.getSpectators so I would go with that. Change all the get spectator for-loops in that function to use i = 1.

Here you go.

Still the same

 

[Lessaire]

change this portion of the script to spit out debug info

    local specs2 = Game.getSpectators(Position(528, 776, 7), false, true, 39, 39, 55, 55)
    if #specs2 >= 1 then
        for i = 1, #specs2 do
            print(1, i)
            print(2, specs2[i])
            print(3, specs2[i]:getStorageValue)
            print(4, bossarenaconfig)
            print(5, bossarenaconfig.joinedStorage)
            print(6, specs2[i]:getStorageValue(bossarenaconfig.joinedStorage))
            if specs2[i]:getStorageValue(bossarenaconfig.joinedStorage) == 1 then
                specs2[i]:teleportTo(specs2[i]:getTown():getTemplePosition())
                specs2[i]:setStorageValue(bossarenaconfig.joinedStorage, 0)
            end
        end
    end

 

[Lessaire]

Um, what?
This code you pasted should not be the state of your code.

This should be

Also: You didn't even do all the fixes? It's still using index of 0!

    local specs = Game.getSpectators(lobby.centerPos, false, true, 0, lobby.radiusX, 0, lobby.radiusY)
    if #specs >= 1 then
        for i = 0, #specs do
            if specs[i]:getStorageValue(bossarenaconfig.joinedStorage) == 1 then
                specs[i]:teleportTo(specs[i]:getTown():getTemplePosition())
                specs[i]:setStorageValue(bossarenaconfig.joinedStorage, 0)
            end
        end
    end