What is wrong with this script? (Please Answer)

Okay, when the script's parent is touched two functions execute: OnTouched and Ontouch.

OnTouched: Checks if what hit is a player, if so it checks if the player has leaderstats, if so it takes 68000 from the player's money.

OnTouch: Checks if what hit is a player, if so it spawns the car.

Instead of two functions we need one function that checks if what hit is a player (I always check for character first, just a preference thingy) and if that is true we check for leaderstats and if we have them we take 68000 from the player's money and spawn in the car. That would look like this:

function OnTouched(hit)
    if hit.Parent:FindFirstChild("Humanoid") then -- it's a character
        local player = game.Players:GetPlayerFromCharacter(hit.Parent)
        if player then -- it is also a player, not an npc
            if player:FindFirstChild("leaderstats") then -- the player has statistics ;D
                if player.leaderstats.Money.Value >= 68000 then
                    player.leaderstats.Money.Value = player.leaderstats.Money.Value - 68000
                    local Clone = game.Workspace["Nissan 240SX(Car)"]:Clone()
                    Clone.Parent = game.Workspace
                    Clone:MoveTo(Vector3.new(-26, 4.2, -878))
                end
            end
        end
    end
end

script.Parent.Touched:connect(OnTouched)