Playtime

[AJXB]

function loadPlayTime(account) 
    local minutes = getAccountData(account, "playTime") 
    if (tonumber(minutes)) then 
        minutes = tonumber(minutes) 
        local hours = math.floor(minutes / 60) 
        if (hours > 0) then 
            setElementData(source, "playTime", hours.." Hours") 
        else 
            setElementData(source, "playTime", minutes.." Minutes") 
        end 
    else 
        setAccountData(account, "playTime", 0) 
        setElementData(source, "playTime", "0 Minutes") 
    end 
    playTimeTimer[source] = setTimer(incrementPlayTime, 60000, 0, source) 
end 
addEventHandler("onPlayerLogin", root, loadPlayTime) 

Save side:

function incrementPlayTime(player) 
    if (not player or not isElement(player)) then return end 
    if (isGuestAccount(getPlayerAccount(player))) then return end 
    local account = getPlayerAccount(player) 
    local minutes = getAccountData(account, "playTime") or 1 
    minutes = tonumber(minutes) 
    minutes = minutes + 1 
    local hours = math.floor(minutes / 60) 
    if (hours > 0) then 
        setElementData(player, "playTime", hours.." Hours") 
    else 
        setElementData(player, "playTime", minutes.." Minutes") 
    end 
    setAccountData(account, "playTime", minutes) 
end 

What's wrong with this?

[Castillo]

If you don't explain the problem, how are we supposed to know?

[AJXB]

It won't save/load

The time adds to ElementData

But when I reconnect the time is gone.

[gghvcffcv]

function loadPlayTime(account) 
    local minutes = getAccountData(account, "playTime") 
    if (tonumber(minutes)) then 
        minutes = tonumber(minutes) 
        local hours = math.floor(minutes / 60) 
        if (hours > 0) then 
            setElementData(source, "playTime", hours.." Hours") 
        else 
            setElementData(source, "playTime", minutes.." Minutes") 
       end 
    else 
        setAccountData(account, "playTime", 0) 
        setElementData(source, "playTime", "0 Minutes") 
    end 
    playTimeTimer[source] = setTimer(incrementPlayTime, 60000, 0, source) 
end 
addEventHandler("onPlayerLogin", root, loadPlayTime) 

Are you sure that are 3 end?

[Castillo]

The first argument of "onPlayerLogin" is the previous account, try changing this:

function loadPlayTime(account) 

to:

function loadPlayTime ( _, account ) 

[AJXB]

local playTimeTimer = {} 
  
function loadPlayTime(_,account) 
    local minutes = getAccountData(account, "playTime") 
    if (tonumber(minutes)) then 
        minutes = tonumber(minutes) 
        local hours = math.floor(minutes / 60) 
        if (hours > 0) then 
            setElementData(source, "playTime", hours.." Hours") 
        else 
            setElementData(source, "playTime", minutes.." Minutes") 
        end 
    else 
        setAccountData(account, "playTime", 0) 
        setElementData(source, "playTime", "0 Minutes") 
    end 
    playTimeTimer[source] = setTimer(incrementPlayTime, 60000, 0, source) 
end 
addEventHandler("onPlayerLogin", root, loadPlayTime) 
  
function stopPlayTime() 
    if (isTimer(playTimeTimer[source])) then killTimer(playTimeTimer[source]) end 
  
end 
addEventHandler("onPlayerLogout", root, stopPlayTime) 
addEventHandler("onPlayerQuit", root, stopPlayTime) 
  
function incrementPlayTime(player) 
    if (not player or not isElement(player)) then return end 
    if (isGuestAccount(getPlayerAccount(player))) then return end 
    local account = getPlayerAccount(player) 
    local minutes = getAccountData(account, "playTime") or 1 
    minutes = tonumber(minutes) 
    minutes = minutes + 1 
    local hours = math.floor(minutes / 60) 
    if (hours > 0) then 
        setElementData(player, "playTime", hours.." Hours") 
    else 
        setElementData(player, "playTime", minutes.." Minutes") 
    end 
    setAccountData(account, "playTime", minutes) 
end 
  

This is the code, somehow when the player is playing time adds, but when he reconnects, his time sets to 0

[Castillo]

Try debugging the script, add outputs to see what is not working.

Also, using a timer for each player is a really bad idea, timers are inefficient.

[AJXB]

Try debugging the script, add outputs to see what is not working.

Also, using a timer for each player is a really bad idea, timers are inefficient.

What do you think I should do with the timers?

[Castillo]

I would make a single timer to update the time of every player.

[AJXB]

I couldn't find a way to fix the current script, can you try to fix it before I change the timers?

[Castillo]

Did you do what I said? add outputs in the saving/loading, etc. That will make it a lot easier to spot the problem.