Attempt to call method 'SetArmor' (a nil value)

Question

So script works perfectly fine, but every time it does it's part there appears an error in console which is says this:

attempt to call method 'SetArmor' (a nil value)

Here's the code

local function ArmorRegeneration ()
  for k,v in pairs( player.GetAll() ) do
    if (v:IsValid()) then
      if v:Alive() and v:Armor() < 150 and ( !v.lastregen or v.lastregen < CurTime() - 1 ) then
        v.lastregen = CurTime()
        v:SetArmor( v:Armor() + 1 )
      end
    end
  end
end

Answer 1

Is this script supposed to be server side instead of client side? I've never done Garry's Mod development, but I figure this might be a similar issue: https://stackoverflow.com/a/58381218/3150484

Also, I'd recommend using ipairs here instead of pairs since the table indices of player.GetAll() are numbers and not strings.

Answer 2

I assume you are running the code both serverside and clientside. The error you receive is probably clientside (ie SetArmor is not a clientside function. You can't set a player's armor clientside. So, it returns nil), but what is working is the serverside part of the code, which is why the code still appears fine in-game.

If I remember correctly, clientside errors are displayed with yellow text in Garry's Mod, so that might be something to look out for to confirm this.

Consider running this script entirely serverside, or alternatively, add if(SERVER)then checks on parts of code that is only supposed to run serverside.