My shirt spawner script isn't working on spawn, any help?

My script that with a value, you get a shirt, isn’t working, I have no idea if its because i have 2 leaderstats, instead of just 1, but any help? Heres the script.

game.Players.PlayerAdded:connect(function(player)

player.CharacterAdded:connect(function(character)

if player:WaitForChild("leaderstats").Money12.Value>= 1 then

print("player.leaderstats.Money12.Value")

character.Shirt:Destroy()

end

lol = script.Parent.Shirt:Clone()

lol.Parent = character

end)

end)

There are a few potential issues here. I want to enumerate them and I'll finish the post with suggested changes.

1. It's highly suggested to throw the shirt into server storage instead. That way if you ever want to change the location of the script (I highly suggest the server script service) then it will be possible without changing the source. This has the additional benefits that traversing the object tree will be a lot more predictable (we will know for sure where the shirt is).
2. It's not enough to wait for the character to be added to the workspace. Often times, especially if there's a lag spike, the character will be added AND THEN appearance objects like shirts, pants, and hats will trickle in and be placed into the player. I think you should switch from CharacterAdded to CharacterAppearanceLoaded (I can't directly link the page since it's broken to me but it's visible in the player events list).
3. Shirts are not super unique objects. A lot of their behavior is actually defined by the ShirtTemplate property. This means that we can actually just copy the ShirtTemplate property of your object and paste it into the players shirt. We don't need to do any deletion. This does add additional complexity however and this is explained by the next point.
4. We don't know IF the player actually has a shirt at all. Default characters I believe do not have a shirt at all. They only have coloring of body parts. This means we will want to make an if statement.
5. Finally, I want to ask because its not clear from the script. Do you want a person to have a shirt if money12 is above 1 on respawn? Because that's the current behavior. We have to use more events if you want the shirt replaced as soon as money12 has a value above 1.

I have assumed that objects have stayed in the same spot with these edits here. Though I highly suggest implementing 1 and 2 yourself.

-- I'm throwing this in a variable so its easier to edit in the future. 
-- Its a good practice to put "constants" or values that will basically stay
-- the same but are specific here.
local shirt = script.Parent.Shirt
	
game.Players.PlayerAdded:connect(function(player)
		-- I have changed the event here.		
		player.CharacterAppearanceLoaded:connect(function(character)

		-- I have split up the leaderstats check because I
		-- think this may prove useful in the future. This 
		-- may for example be replaced with a changed event 
		-- for the money12 value.
		if player:FindFirstChild("leaderstats") then
			if player.leaderstats.Money12.Value >= 1 then
				if character:FindFirstChild("Shirt") then
					character.Shirt.ShirtTemplate = shirt.ShirtTemplate
				else
					local newshirt = Instance.new("Shirt")
					newshirt.Name = "Shirt"
					newshirt.ShirtTemplate = shirt.ShirtTemplate
					newshirt.Parent = character
				end
			end
		end
	end)
end)