help saving skins???

[Luke_Ferrara]

function onPlayerQuit ( )
local playeraccount = getPlayerAccount ( source )
if ( playeraccount ) then
local playerskin = getElementModel ( source )
setAccountData ( playeraccount, "skin", playerskin )
end
end
 
function onPlayerLogin ( )
local playeraccount = getPlayerAccount ( source )
if ( playeraccount ) then
local playerskin = getAccountData ( playeraccount, "skin" )
if ( playerskin ) then
setPedSkin ( source, playerskin )
end
end
end
 
addEventHandler ( "onPlayerQuit", getRootElement ( ), onPlayerQuit )
addEventHandler ( "onPlayerLogin", getRootElement ( ), onPlayerLogin )

This should work but I have no idea why it is not, I got this code from another post however that was to save money so I figured if i changed what the code looks for and saves it will save the players skin however it doesn't, it sets the skin to 0 and doesn't update when it is changed

[50p]

Before you get/setAccountData, you should check if player's account isGuestAccount.

[Luke_Ferrara]

function onPlayerQuit ( )
local playeraccount = getPlayerAccount ( source )
if isGuestAccount ( playeraccount ) then
outputConsole ( "For full features you must register!", playerSource )
else
local playerskin = getElementModel ( source )
setAccountData ( playeraccount, "skin", playerskin )
end
end
 
function onPlayerLogin ( )
local playeraccount = getPlayerAccount ( source )
if isGuestAccount ( playeraccount ) then
outputConsole ("Register", playersource)
 
else
local playerskin = getAccountData ( playeraccount, "skin" )
setPedSkin ( source, playerskin )
 
end
end
 
addEventHandler ( "onPlayerQuit", getRootElement ( ), onPlayerQuit )
addEventHandler ( "onPlayerLogin", getRootElement ( ), onPlayerLogin )

okay now it saves the skin but doesn't call it when player logs in, I have it check for guest account idk whats going wronge

[50p]

onPlayerLogin passes 2 account arguments to your function. You can use them instead of getPlayerAccount.

account thePreviousAccount, account theCurrentAccount, bool autoLogin

theCurrentAccount will be the account player logged into. You can then use getAccountData( currentAccount, "skin" ).

And please learn to debug scripts. Debugging scripts will help you so much that you won't even have to ask questions here.

[Luke_Ferrara]

okay thanks 50 Ill start trying to debug