Attempting to allow players to give themselves a hat

Help and Feedback Scripting Support
1 
    player.Chatted:connect(function(msg)
        if (msg:lower() == "/e hat") then
            if (player.Character:FindFirstChild("Humanoid")) then
            local Copy = game.ReplicatedStorage:FindFirstChild("Hat1"):Clone()
            local player = game.Players:GetPlayerFromCharacter(player)
            Copy.Parent = player.Character
            end
        end
    end)
end)

I’m unsure what I am doing incorrectly, any advice would be appreciated. Thank you.

2

I am not sure of the problem here, but what I know is that you don’t have to define the player twice.

1 Like
3

A bad habit I fall into far too often.

4

Are you using Hat or Accessory?
If you use Hat, stop using it. It is a deprecated instance that has garbage functionality.

EDIT: Also, shorten your code when you can, as long as it is still readable.

local Copy = game.ReplicatedStorage:FindFirstChild("Hat1"):Clone()
local player = game.Players:GetPlayerFromCharacter(player)
Copy.Parent = player.Character

could be replaced with

game.ReplicatedStorage:FindFirstChild("Hat1"):Clone().Parent = player.Character
2 Likes
5

I am using an accessory. I’ve been trying to resolve the issue however I’m still not sure why it won’t work.

6

Does the accessory have a Handle part?
Do you have any errors in the output?

1 Like
7

To add an accessory/hat to a character, use the function AddAccessory inside of Humanoid, it works with both types since Hat is a descendant of Accessory and simulates the same behaviour

Character.Humanoid:AddAccessory(accessory)

2 Likes
8

No errors appear to show, and it has a handle.

9

I attempted something like this but still nothing.

   local humanoid = character:WaitForChild("Humanoid")
     player.CharacterAppearanceLoaded:connect(function(character)
 player.Chatted:connect(function(msg)
        if (msg:lower() == "/e hat") then
            if (player.Character:FindFirstChild("Humanoid")) then
            Humanoid:AddAccessory(game.ReplicatedStorage.Hat1) 
            end
        end
    end)
end)
10

Can you show us what happens, the properties of the accessory, and the child-parent structure of the accessory?

1 Like
11

The problem there is that your code is not structured correctly.
You are defining a Humanoid BEFORE the event if fired.

Your character and humanoid variable definition should be inside the event, not outside it

Player.Chatted:Connect(function(msg)
   local character = Player.Character
   if not character then return end

   Humanoid = character.Humanoid
   Humanoid:AddAccessory()
end)
1 Like
12

The accessory itself is not the issue. I have successfully been able to obtain the hat through different means (spawning with it on). I am just having trouble implementing it through a command system.
image

13

I’m still unsure what I am doing incorrectly.

   local humanoid = character:WaitForChild("Humanoid")
     player.CharacterAppearanceLoaded:connect(function(character)
Player.Chatted:Connect(function(msg)
 if (msg:lower() == "/e hat") then
   local character = Player.Character
   if not character then return end
      Humanoid = character.Humanoid
      Humanoid:AddAccessory(game.ReplicatedStorage.Hat1) 
   end
end)
14

Ok here’s the two things that’s wrong with your code

1. Chatted only need to be binded once, not every time the character is added
2. My original code from that post included an error, it has been fixed in the post

1 Like
15

I think I will call it a night. I’m just unable to do it correctly. Thank you for taking the time to respond to my thread.

16

That’s probably your issue. Most likely the variable isn’t garbage collected when you respawn, so it produces no error.