[Question] Items that spawn temporary objects.

[jar0fair]

Try this. I basically just added back in the GetWorldObject you removed.

Code: [Select]

-- Cauldron
local CauldronItem = 200001
local CauldronObj = 186472
-- Chair
local ChairItem = 200002
local ChairObj = 186732
-- bed
local BedItem = 200003
local BedObj = 13948
-- vendor tent
local VendortentItem = 200004
local VendortentObj = 4000668

local spawn = {}

local function SpawnObj(plr, entry, despawn_time)
    local guid = plr:GetGUID()
    if spawn[guid] then
        local spawnedGobject = plr:GetMap():GetWorldObject(spawn[guid])
        if spawnedGobject then
            spawnedGobject:RemoveFromWorld(true)
            spawn[guid] = nil
        else
            plr:SendNotification("You need to be closer to the existing object to despawn it")
        end
    else
        local x, y, z, o = plr:GetLocation()
        local gobject = plr:SummonGameObject(entry, x, y, z, o, despawn_time)
        if gobject then
            spawn[guid] = gobject:GetGUID()
        end
    end
end

local function Cauldron(event, plr, item, target)
    SpawnObj(plr, CauldronObj, 0)
end

local function Chair(event, plr, item, target)
    SpawnObj(plr, ChairObj, 0)
end

local function Bed(event, plr, item, target)
    SpawnObj(plr, BedObj, 5)
end

local function VendorTent(event, plr, item, target)
    SpawnObj(plr, VendortentObj, 0)
end

RegisterItemEvent(CauldronItem, 2, Cauldron)
RegisterItemEvent(ChairItem, 2, Chair)
RegisterItemEvent(BedItem, 2, Bed)
RegisterItemEvent(VendortentItem, 2, VendorTent)


Nope, It doesn't remove the objects, and I am not sure why, because, it really seems like would, reading the code.

[Rochet2]

Nope, It doesn't remove the objects, and I am not sure why, because, it really seems like would, reading the code.

Are you getting any errors?
The script does nothing after spawning one gameobject? No more spawns or any messages?

[Rochet2]

Alright, I see the error now.
Change
local guid = plr:GetGUID()
to
local guid = plr:GetGUIDLow()

The difference here is that GetGUID returns an uint64 number which is represented as an object.
Each GetGUID returns a different object causing the lua table to map them to different locations.
GetGUIDLow returns a normal number.
To properly make the uint64 work as table key it might need to be converted to strings, but using lowguid is easier for this and might be faster.

[jar0fair]

Alright, I see the error now.
Change
local guid = plr:GetGUID()
to
local guid = plr:GetGUIDLow()

The difference here is that GetGUID returns an uint64 number which is represented as an object.
Each GetGUID returns a different object causing the lua table to map them to different locations.
GetGUIDLow returns a normal number.
To properly make the uint64 work as table key it might need to be converted to strings, but using lowguid is easier for this and might be faster.

Thank you so much! Both of you. It is finally working how I would like it to. Me and my entire team are very grateful.

[jar0fair]

Hia, back again.

After a lot of testing I discovered that if I log out, with one of my objects spawned, and it disappears, I can no long use the item anymore without reloading the lua engine. This is because the object is gone, and the code does not recognize it. So, I have written an additional function to nil the spawns on logout. However, it does not work. I am wondering if anyone sees any glaring mistakes in the logout function that I am somehow missing.

Code: [Select]

-- Name Here
local item = 200002
local obj = 22713

-- Code
local spawn = {}

local function SpawnObj(plr, entry, despawn_time)
    local guid = plr:GetGUIDLow()
    if spawn[guid] then
        local spawnedGobject = plr:GetMap():GetWorldObject(spawn[guid])
        if spawnedGobject then
            spawnedGobject:RemoveFromWorld(true)
            spawn[guid] = nil
        end
    else
        local x, y, z, o = plr:GetLocation()
        local gobject = plr:SummonGameObject(entry, x, y, z, o, despawn_time)
        if gobject then
            spawn[guid] = gobject:GetGUID()
        end
    end
end

local function logoff(plr, entry)
    guid = plr.GetGUIDLow()
    spawn[guid] = nil
end
-- Spawn Function
local function use(event, plr, item, target)
    SpawnObj(plr, obj, 0)
end
-- Registers
RegisterItemEvent(item, 2, use)
RegisterPlayerEvent(4, logoff)

[Rochet2]

Code: [Select]

local function logoff(plr, entry)
    guid = plr.GetGUIDLow()
    spawn[guid] = nil
end

Erm .. thats ..
The reason the code doesnt work is because your parameters are wrong logoff(plr, entry).
If you look here: http://eluna.emudevs.com/Global/Registe ... Event.html it says (event, player)

also make that guid variable local.
Also you call member functions with : not with a dot.

[jar0fair]

Thanks again, Rotchet, this works just how I needed it to, now. Couldn't have done it without you.