Teleport to Spawn if lastBed doesnt exist

[moiko89]

I will teleport the Player to WorldSpawn if they lastBed doesnt exist.
But the Player.TeleportToCoords doesnt work for me, i copy the SpawnTeleport Code from plugins/core/spawn.lua

Code:

19:37:01] LUA: Plugins/Test1/main.lua:49: error in function 'TeleportToCoords'.
     argument #1 is 'string'; 'cEntity' expected.

[19:37:01] Stack trace:
[19:37:01]   [C](-1): TeleportToCoords
[19:37:01]   Plugins/Test1/main.lua(49): (no name)
[19:37:01] Stack trace end
[19:37:01] Error in <attached> calling function <callback>()

PHP Code:

PLUGIN = nil

function Initialize(Plugin)
    Plugin:SetName("Test1")
    Plugin:SetVersion(1)

    -- Hooks
    cPluginManager.AddHook(cPluginManager.HOOK_PLAYER_SPAWNED, OnPlayerSpawned)
    PLUGIN = Plugin -- NOTE: only needed if you want OnDisable() to use GetName() or something like that

    LOG("Initialised " .. Plugin:GetName() .. " v." .. Plugin:GetVersion())
    return true
end

function OnDisable()
    LOG(PLUGIN:GetName() .. " is shutting down...")
end

function OnPlayerSpawned(Player) -- See API docs for parameters of all hooks
    local lastBedPos = Player:GetLastBedPos()
    local world=Player:GetWorld()
    local block=world:GetBlock(lastBedPos.x, lastBedPos.y, lastBedPos.z)

    local WorldIni = cIniFile()
    WorldIni:ReadFile(Player:GetWorld():GetIniFileName())

    local SpawnX = WorldIni:GetValue("SpawnPosition", "X")
    local SpawnY = WorldIni:GetValue("SpawnPosition", "Y")
    local SpawnZ = WorldIni:GetValue("SpawnPosition", "Z")

    if (block == 26) then
        -- do stuff
    else
        Player.TeleportToCoords(SpawnX, SpawnY, SpawnZ)
    end
end 


[xoft]

The error message basically says it all. You're calling TeleportToCoords with wrong arguments, it expects a cEntity as its first parameter, but you're giving it a string.
Note that in Lua, there are two "calling conventions":
obj.fn("a") -- calls obj.fn with param "a"
obj:fn("a") -- calls obj.fn with params obj, "a"
The second is what Cuberite is using everywhere and is expecting from the plugins.

The SpawnX/Y/Z need to be read with a GetValueI() function, so that they are numbers, not strings.

[xoft]

One more friendly advice: Use E_BLOCK_BED instead of 26.

[Seadragon91]

A list of block, item ids and more constants can you find here Globals

[moiko89]

If i use

PHP Code:

    local SpawnX = WorldIni:GetValueI("SpawnPosition", "X")
    local SpawnY = WorldIni:GetValueI("SpawnPosition", "Y")
    local SpawnZ = WorldIni:GetValueI("SpawnPosition", "Z") 


i become this msg

Code:

[20:06:37] LUA: Plugins/Test1/main.lua:27: attempt to call method 'GetValuelI' (a nil value)
[20:06:37] Stack trace:
[20:06:37]   Plugins/Test1/main.lua(27): (no name)
[20:06:37] Stack trace end
[20:06:37] Error in <attached> calling function <callback>()

can you give me an example?

[Seadragon91]

Do you want to read the spawn position from the world ini?
If yes you could directly use the functions: GetSpawnX, GetSpawnY and GetSpawnZ from cWorld.

if not about world spawn.The error means that WorldIni is not initialized. Post the code where you are initializing WorldIni.

Edit: Ah sorry didn't saw the whole code in first post, will look at it.

[moiko89]

Do you want to read the spawn position from the world ini?
If yes you could directly use the functions: GetSpawnX, GetSpawnY and GetSpawnZ from cWorld.

if not about world spawn.The error means that WorldIni is not initialized. Post the code where you are initializing WorldIni.

Edit: Ah sorry didn't saw the whole code in first post, will look at it.

Server Start Log

Code:

[20:41:41] --- Started Log ---
[20:41:41] Cuberite Jenkins clang x64 Release (master) build id: #578
[20:41:41] from commit id: c83b051c1a3e68fc9315ffffb277dc3da86cf02a built at: Thu  8 Sep 21:30:46 UTC 2016
[20:41:41] Creating new server instance...
[20:41:41] Reading server config...
[20:41:41] Starting server...
[20:41:41] Compatible clients: 1.8.x, 1.9.x, 1.10.x
[20:41:41] Compatible protocol versions 47, 107, 108, 109, 110, 210
[20:41:41] WebServer: The server will run in unsecured HTTP mode.
[20:41:41] Put a valid HTTPS certificate in file 'webadmin/httpscert.crt' and its corresponding private key to 'webadmin/httpskey.pem' (without any password) to enable HTTPS support
[20:41:41] Loaded 465 crafting recipes
[20:41:41] Loaded 22 furnace recipes and 56 fuels
[20:41:41] Loaded 88 brewing recipes
[20:41:41] -- Loading Plugins --
[20:41:41] Initialised Core v.15
[20:41:41] Initialized TransAPI v.1
[20:41:41] Initialized ChatLog v.3
[20:41:41] Initialised Test1 v.1
[20:41:41] -- Loaded 4 Plugins --
[20:41:42] Did not find an acceptable spawnpoint. Generated a random spawnpoint position at {0.00, 168.00, 0.00}
[20:41:42] WebAdmin is running on port(s) 8080
[20:41:42] Startup complete, took 1662ms!

Code with GetSpawn

PHP Code:

local SpawnX = world:GetSpawnX()
local SpawnY = world:GetSpawnY()
local SpawnZ = world:GetSpawnZ()
Player.TeleportToCoords(SpawnX, SpawnY, SpawnZ) 


Error with GetSpawn

Code:

[20:39:31] WorldSpawnX:-2.1523170155827 WorldSpawnY:65 WorldSpawnZ:-32.89404058136
[20:39:31] LUA: Plugins/Test1/main.lua:37: error in function 'TeleportToCoords'.
    argument #1 is 'number'; 'cEntity' expected.

[20:39:31] Stack trace:
[20:39:31]   [C](-1): TeleportToCoords
[20:39:31]   Plugins/Test1/main.lua(37): (no name)
[20:39:31] Stack trace end
[20:39:31] Error in <attached> calling function <callback>()

What i can do?

[Seadragon91]

Change
Player.TeleportToCoords(SpawnX, SpawnY, SpawnZ)
to
Player:TeleportToCoords(SpawnX, SpawnY, SpawnZ)

Has to be a colon :

[moiko89]

Change
Player.TeleportToCoords(SpawnX, SpawnY, SpawnZ)
to
Player:TeleportToCoords(SpawnX, SpawnY, SpawnZ)

Has to be a colon :

Waaah.... now i see the different in xoft's post too.....

obj.fn("a") -- calls obj.fn with param "a"
obj:fn("a") -- calls obj.fn with params obj, "a"



I understand... thanksick:

[xoft]

I'm quite concerned with the "GetValuelI" not found, even with the INI file not read, the method should be there.