Why does it say index nil with ' userId?

I had some difficulties to understand your Script, but I think I found a solution, I haven't tested it yet, but I think it should work.

local Market = game:GetService("MarketplaceService")
local Button = Workspace.TYCOONS.FirstTycoon.Buttons.Robux.Button
local Id = -- GamePass Id

local ECValue = Workspace.TYCOONS.FirstTycoon.Essentials.Conveyor.Value

Button.Touched:Connect(function(Hit)
    if Hit.Parent:FindFirstChild("Humanoid") then
        local Player = game.Players:GetPlayerFromCharacter(Hit.Parent)
        if Player and owner.Value == Player.Name then
            if market:UserOwnsGamePassAsync(Player.UserId, id) then
                if ECValue.Value == false then
                    ECValue.Value = true
                end
            end
        end
    end
end)

You confused the variables, Players and Player, changing their values, so the Script returned UserId = Nil, because Player, was the same as the Players Service, while Players, was the player itself.

Try this... If it doesn't work, I need you to point out the error so I can know how to fix it more easily.