TalkAction Talkaction that teleports you to where you want :) My first script

[Hatsune Miku]

Credits to Bogart for helping me with some errors and teaching me how to fix them ​

So this script basicly teleports you to (1000x 1000y and 7z) from (1002x 1000y 7z) you can change thoose to whatever you want

If you want you can change it to anything else
If you want you can use this script in a quest Make a wall people can't pass and a little square, when people stand on it they have to say your "secret" code(talkaction) to get over to the other side

If you want people to say "!omgnoobwall" to get teleported to other side you can do that too

The Lua file

local pos = {x = 1002, y = 1000, z = 7}
local temple = {x = 1000, y = 1000, z = 7}
local k = getCreaturePosition(cid)
function onSay(cid, words, param)
	if (k.x == pos.x) and (k.y == pos.y) and (k.z == pos.z) then
		doTeleportThing(cid, temple)
		doPlayerSendTextMessage(cid, 21, "You have been teleported to the temple!")
		doSendMagicEffect(getPlayerPosition(cid), 31)
	end
return true
end

In talkactions.xml

<talkaction words="!omgnoobwall" event="script" value="x.lua"/>

Hope this helps someone Its my first script after all so no noob comments please

If you get any bugs please write here and I will try to fix it if I can altho I don't think you will get any
I am using TFS 0.3.6 distro, no bugs for me works fine
This is the distro I am using http://otland.net/f18/8-60-forgotten-server-0-3-6-crying-damson-v8-147913/

EDIT: Also if you got any suggestions for something a noob in lua can make please post I want to learn this as quick as possible so any suggestions is welcome

 

[azzkaban]

Approved

 

[Hatsune Miku]

Thanks

Bump Come on guys use this and give me some feedback on it

 

[Erexo]

if (k.x == pos.x) and (k.y == pos.y) and (k.z == pos.z) then

You can simply set that:

if (k == pos) then

 

[Hatsune Miku]

Oh lol thanks *goes to complain to Bogart *

- - - Updated - - -

updated first post

 

[tetra20]

Good Job Bro, might be helpful.here is a advice​

1-when using sendMagicEffect don't use 31 use CONST_ME_MAGIC there is much of them. they can be found in lib-constant​

2-try to make a table not only 1 place ​

thats what i can say​

 

[Hatsune Miku]

Well thanks alot but if I make a table it won't be exactly that place Maybe it will teleport me to the second or even third one. while in a quest I just wanted to get over the wall If you understand what I mean

Thanks about the sendmagiceffect I was lazy and just did it that way Might change the way I do stuff later

 

[hodleo]

does this rly works? it usually returns false because stack positions don't match

local k = getCreaturePosition(cid)
	if (k == pos) then

it'd be wiser to use:

if doComparePositions(pos, getCreaturePosition(cid)) then

 

[Hatsune Miku]

script worked fine until i changed it to what erexo posted.. and idk it should probably work then also thats what makes sense well I will update my first post again.. but then again... I haven't tested it after I edited if it happens to not work I will just turn it back to the one I made Cause I know that one works without any errors shortening a short script doesnt really matter for me

 

[Summ]

Comparing table does not work like this.
You can only check whether 2 tables are identical (the same reference):

local x = {x = 1, y = 2, z = 3}
return x == x -- should return true

return {x = 1, y = 2, z = 3} == {x = 1, y = 2, z = 3} -- should return false

 

[UpInSmoke]

The value of k is unused, you might aswell delete this line

local k = getCreaturePosition(cid)

 

[Hatsune Miku]

The value of k is unused, you might aswell delete this line

local k = getCreaturePosition(cid)

Uhm its used look closer and if not.. just read the previous comments from others they are commenting on how to "improve" the script exactly at that part

Comparing table does not work like this.
You can only check whether 2 tables are identical (the same reference):

local x = {x = 1, y = 2, z = 3}
return x == x -- should return true

return {x = 1, y = 2, z = 3} == {x = 1, y = 2, z = 3} -- should return false

Lol I changed it to the way I made the script you guys just made it more complicated to understand 3 guys telling me 3 different things lol xD

Mine works so I don't care about how it looks

 

[UpInSmoke]

Lol well now that you changed it back it is, but before it wasnt being used

 

[Hatsune Miku]

Might be true ohh well xD