Cloning Script| Not Working? [Edited]

**The below answer is given under the assumption the script in question is a LocalScript and is in the player's PlayerGui, Backpack or Character. **

• Model instances don't have a 'Children' property.
• You should use elseif statements instead of if statements after the first if statement.
• This will only work when they first spawn - If that's what you want, then it's not an issue.
• By simply moving the children of the two models, you prevent their reuse; you need to clone.

local Player = game.Players.LocalPlayer
repeat wait() until Player.Character ~= nil and Player.TeamColor ~= BrickColor.new("White")

local AwareModel = game.ReplicatedStorage.Aware
local SoldierModel = game.ReplicatedStorage.Soldier

function GiveParts(Model)
    for _, Object in pairs(Model:GetChildren()) do 
        Object:Clone().Parent = Player.Character
    end
end

if Player.TeamColor == BrickColor.new("Crimson") then
    GiveParts(AwareModel)
elseif Player.TeamColor == BrickColor.new("Electric Blue") or Player.TeamColor == BrickColor.new("Pearl") then
    GiveParts(SoldierModel) 
end

Line 7: Created a function to give the model's contents to the player; easier than typing out the same thing for the three different TeamColors.

Line 8: Iterating through the given model, so that we're able to give the player the correct "kit"

Line 15: Both TeamColors receive the same, so we might as well just use 'or' rather than using a completely different if statement for the same thing.