Gamepass counting extra money when player dosen't own gamepass?

as soon as one player has the game pass it will set vip to true meaning as soon as one player has the gamepass inthe lobby all the other players are "vip" too bc you dont keep a "vip" per player but one for every player

so what you have to do is keep a "vip" for each player, so we create a table that keeps a record for each player if they are vip

local vip = {}
local gamePassService = game:GetService("GamePassService") -- always save it in the highest scope possible
local MarketplaceService = game:GetService("MarketplaceService")

game.Players.PlayerAdded:connect(function(plyr)
  -- if gamePassService:PlayerHasPass(plyr,12146561) then 
   --   table.insert(vip, plyr.UserId) 
  -- end
-- above function is deprecated as pointed out in the comments but i cant full replace it atm due to it using  assetid while the newer one use gamepass id so either you get the gamepass id and fill it in or you use the above
    if MarketplaceService:UserOwnsGamePassAsync(plyr.UserId, gamepassid) then  --fill in the gamepassid
        table.insert(vip, plyr.UserId) -- adds the userid of the player which has the gamepass
    end
end)

btw globals are bad always use local, connect is deprecated use Connect and indent your code

function onTouch(part)
    local searcher = part.Parent:FindFirstChild("Humanoid")
    if searcher ~= nil then
        local finder = game.Players:FindFirstChild(part.Parent.Name)
        if finder ~= nil and finder.Cheeze.Value == 3 then
            finder.leaderstats.Laps.Value += 1
            finder.leaderstats.Cash.Value += 500
            finder.Cheeze.Value = 0
            if table.find(vip, finder.UserId) then -- checks if it is in the table
                finder.leaderstats.Cash.Value += 300
            end
        end
    end 
end

script.Parent.Touched:Connect(onTouch)

btw you can also do var += 1 instead var = var +1, use more variables, findFirstChild is also deprecated

local plr = game.Players.LocalPlayer
local ID = 0 -- GamepassId
local MPS = game:GetService("MarketPlaceService")
local amnt = 25
script.Parent.Touched:Connect(function(hit)
local hit = hit.Parent -- If you want this this can be used as a kill brick but lets head on --
local RetrieveData = plr.leaderstats
local Cash = RetrieveData.Cash
Cash.Value = Cash.Value + amnt
haspass = MPS:UserOwnsGamePassAsync(plr.UserId, ID)
if haspass then
amnt = amnt * 2 -- if not do amnt *2 --
Cash.Value = Cash.Value + amnt

Ever since April 2018, you could no longer use PlayerHasPass(). You must now use UserOwnsGamePassAsync. Also, you don't use the player, you use it like this:

local MPS = gamel:GetService("MarketplaceService");

if MPS:UserOwnsGamePassAsync(player.UserId, id) then
    -- // code
end

Also PLEASE, PLEASE INDENT YOUR CODE! AND DON'T USE DEPRECATED FEATURES!

I'd also like you to check out VerdommeMans answer.