Won't clone frame?

I am trying to make a SurfaceGui shop that gets all items in a Folder, clones the frame and changes the frame.ItemName(TextLabel).Text to the name of it. But for some reason I can't get this script to work.

local sp = script.Parent
local folder = sp.ScrollingFrame
local player = script.Parent.Parent.Parent.Parent

function refresh()
    folder:ClearAllChildren()
    for _,item in pairs(game.ServerStorage.ShopItems:GetChildren()) do 
        local frame = script.Frame:Clone()
        frame.Parent = folder
        frame.ItemName.Text = item.Name " : "..item.Price.Value..""
        frame.Name = item.Name
        frame.Purchase.MouseButton1Click:connect(function(c)
            if player:FindFirstChild("globalstats").Points.Value < item.Price.Value
                then
                return false
            else
                player:FindFirstChild("globalstats").Points.Value = player:FindFirstChild("globalstats").Points.Value - item.Price.Value
                local itemt = item.Tool:Clone()
                itemt.Parent = player.Backpack
                itemt.Name = item.Name
            end
        end)
    end
end

game.Players.PlayerAdded:connect(function(p)
    wait(1)
    refresh()
end)

Output show's no error's, can anyone tell me why it won't work?