I am using a regular server script, which it can not get the users playerID.
script.Parent.MouseButton1Click:Connect(function(Player)
local playerId = Player.UserId
script.Parent.Parent.No1.Value:WaitForChild('Eng').P1.Disabled = false
wait(2)
script.Parent.Parent.No1.Value:WaitForChild('Eng').P1.Disabled = true
script.Parent.Parent.No1.Value:WaitForChild('Eng').P2.Disabled = false
wait(2)
script.Parent.Parent.No1.Value:WaitForChild('Eng').P2.Disabled = true
script.Parent.Parent.No1.Value:WaitForChild('Eng').P3.Disabled = false
wait(2)
script.Parent.Parent.No1.Value:WaitForChild('Eng').P3.Disabled = true
script.Parent.Parent.No1.Value:WaitForChild('Eng').P4.Disabled = false
wait(2)
script.Parent.Parent.No1.Value:WaitForChild('Eng').P4.Disabled = true
script.Parent.Parent.No1.Value:WaitForChild('Eng').P5.Disabled = false
wait(2)
script.Parent.Parent.No1.Value:WaitForChild('Eng').P5.Disabled = true
script.Parent.Parent.No1.Value:WaitForChild('Eng').P6.Disabled = false
script.Parent.Parent.Confirm.BackgroundColor3 = Color3.new(0, 255, 0)
In the developer console it was saying "attempt to index local 'Player' (a nil value)

4 comments
why are you trying to find a Player inside of a ClickDetector...?
you cannot just make up and create arguments for a function; they are predetermined.
It is not a click dectector, this is located in a GUI.
oh then still, it's a different argument