Is there anything wrong with this script?

I believe I have it set up correctly, but I don't know if I should use 'plr' or 'user' because the player is already defined as 'plr'?

local PointsService = game:GetService("PointsService")
function onJoin(plr)
    wait(3600)
    Game:GetService("BadgeService"):AwardBadge(plr.userId,31678972)
            PointsService:AwardPoints(plr.userId, 7)

end
game.Players.PlayerAdded:connect(onJoin)

Sadly, as I own the badge and can't get it, i can't test this out unless i want to stay in the game for 1 hour.