In your function, you have it set to where when anything touches it, it will get passed as the player and then the script will attempt to edit the Points of the object that touched it.
To fix this, you should add an if statement which checks if the object that touched it has a humanoid, if it does then you can declare the player variable.
Checking if an object has a humanoid ensures that it is a player who is touching it and so it won't return nil.
1 | script.Parent.Touched:Connect(function(hit) |
2 | if hit.Parent:FindFirstChildWhichIsA('Humanoid') then |
3 | local player = game.Players:GetPlayerFromCharacter(hit.Parent) |
However, I'm not sure what you're trying to do by 'player.PlayerGui.Enabled' because that isn't a property of PlayerGui and you cannot access PlayerGui using a ServerScript, so you should read up on RemoteFunctions here and use that instead.
0