What core / lua engine are you using?
On trinitycore all objects have a guid - however Im unsure if you can use it to get the object.
In any case when spawing an object the spawn function usually returns a reference to the object.
This reference could be used to identify and maybe access the object later on.

However such access is dangerous without guaranteeing the object still exists (dangling pointer).
A grid search with checking the guid might be a better way if needing access after time has passed.

Can you post what you have already?

Working on coding Eluna multithreaded version.
Should work on a PR as well .. and hoover since its saturday.

You dont actually need to do anything to your script to have the crash logs etc I just said. :3

Some notes:
Do not store variables like _stock unless you REALLY have to. They are subject to race conditions in some cases and they are shared between users using the gossip on any NPCs using the same script.
So even if you have 2 players on same NPC using the script, they may mess up eachother's choices in the script. Its better to use the sender and action to pass data around gossip and if something needs to be stored outside, store it to the player class or similar. (depends on situation)
I think its a waste sometimes when something like GOSSIP_SENDER_MAIN is used as sender.

Spell, aura, creature and gameobject AI scripts are different and in those you can store member variables to the script class. But do not store anything to the base script classes you use like CreatureScript and PlayerScript and so.

Use the override keyword if possible. It will make sure the script operates correctly in the future and makes your own typos and other mistakes show.

Since gossip is handled in thread safe environment, it delays ALL actions on the server for the time the script runs. For that reson it would be important to keep it efficient. And one thing that is not efficient at all is database querying in real time. PExecute (and Execute) should be asynchronous (delayed/queued) though, so that /should/ be alright.
Load the needed data on start up or player log in or similar so its loaded beforehand and doesnt need loading all the time when using the script
That way you wont probably need to resort to hacks like this; SELECT %u FROM.
Database actions should be file reads, so having an SSD would speed the database. File IO is way slower than RAM access.

Post the database SQL code and data with the script. Its not always easy to just read a random script and see whats wrong and testing it is not a walk in the park without the database structure and code. Also the database structure could be faulty - many use bad database table structures without being aware how it may affect their script.
For example what data do you have in stock table?

Note that those talk and similar functions can be spammed. You can hit some action and everyone gets their chat and screen spammed with text.
Note that investment can be 99999. was this intended?

When testing, I dont have any crash.

Note that UPDATE `character_stockdata` SET `1` = %u WHERE `iD` = %f
here you mixed %u and %f.
Note that in SellStock you use result1 before checking if it is null. Check EVERY query! (if (!result) return)


PS. I hope you are using some sane IDE like Visual Studio to code your scripts.

This has been posted in some topics here already, but here it is as a release : )
Thanks a lot to Valkryst for reporting bugs and just using the modification.

Additional note: You can dress any existing races including playable and the naga and vrykul etc.

Ppl have asked once in a while if you could dress NPCs some gear and items etc. Usually people answer that you need to make a patch or similar.
Through some core editing and the mage spell "mirror image" (not actually using the spell, but the packet and flags) I made it possible to dress NPCs with gear.
Its a rather simple idea.

You can make an NPC, set the displays of items you want him to have and his race and gender (defines displayID) as well as facial features and skin color.
All this is done in the database.
The values for skins and features range from 0 to 12 or something .. not quite sure how many there are.

Read more & get it:
http://rochet2.github.io/Dress-NPCs.html

SQL does math with bigint - int64.
So if there is any problems with the result2 being wrong, its likely that you would need to use GetInt64.
Read more here: https://github.com/ElunaLuaEngine/Eluna/issues/89

Create this small script that works with the spell
http://www.wowhead.com/spell=62743/undercity-banner
as an example to save the spawned object to the database.
https://gist.github.com/Rochet2/d6212b5a2289eeac2d56

What it does?
It stops the default behavior and uses the spell's data to make a permanent spawn.

Quote from: "Valkryst"

This core modification by Rochet2 will do what you want.

Does this work for 4.3.4?

It is only made for 3.3.5 and 6.x (TC) and it be added to 4.3.4 as well. However it might need a bit of tweaking to do that.

Hmm, seems the item is a spell visual.
Its not even positioned right in the hand to be like an equipped weapon. And the model is different from the item model.

Found it with ID 6358 in spellvisualeffectname.dbc

WIth some understanding and client editing I guess it could be possible to for example edit this spell to use some other item model or something.
Never really modded myself though. And dont see how the visual and the spell link together at all ..