[Lua] Profession stone

[Summer]

Im trying to make profession stone's for every profession a stone that takes the players profession skill from 1 to 300 when u click it but cant get it to work

this is what i got

function Item_Trigger(item, event, player)
    Item_menu(item, player)
end

function Item_menu(item, player)
 
    if (player:IsInCombat() == true) then
    player:SendAreaTriggerMessage("You are in combat!") 
    else                          
    item:GossipCreateMenu(3543, player, 0)
    item:GossipMenuAddItem(4, "Enchanting Stone", 1, 0)
    item:GossipSendMenu(player)
end


function gearonselect(item, event, player, id, intid, code)
 
    if (intid == 1) then
    player:AdvanceSkill(333, 299);
    player:GossipComplete()
end

end
end

RegisterItemGossipEvent(90900,1,"Item_Trigger")
RegisterItemGossipEvent(90900,2,"gearonselect")

but it dosent work plz help me

[The-Eradicator]

Read the sticky.

[Kiev]

One thing when posting something like this, post a picture of the error, or else were just looking at code and wasting time.

[Aellya]

You have too many 'end' just before your first RegisterItemGossipEvent() that might be causing the script file to finish before it registers?

I don't know much about lua scripts (only lua addons) so i might be wrong.

:edit:

Looks like you have the right number of 'end's but the function gearonselect() is inside the Item_menu() function, was this intentional?

[Ground Zero]

Fixed. I don't know why you had a useless function at the top.
You registered your function to the wrong name.

function Item_menu(item, player)
    if (player:IsInCombat() == true) then
        player:SendAreaTriggerMessage("You are in combat!")
    else
    item:GossipCreateMenu(3543, player, 0)
    item:GossipMenuAddItem(4, "Enchanting Stone", 1, 0)
    item:GossipSendMenu(player)
    end
end


function gearonselect(item, event, player, id, intid, code)

    if (intid == 1) then
        player:AdvanceSkill(333, 299)
        player:GossipComplete()
    end
end
RegisterItemGossipEvent(90900, 1, "Item_menu")
RegisterItemGossipEvent(90900, 2, "gearonselect")