You only created one instance of an Attachment object. On line 29, you destroy that object the first time .Touched fires, and therefore there are no more attachments to destroy (hence the NULL error). To fix this, create a new attachment whenever the .Touched event fires.
02 | local tool = script.Parent.Parent |
03 | local Blade1 = script.Parent.Parent.Union |
04 | local Blade2 = script.Parent.Parent.Union2 |
05 | local Blades = Blade1 or Blade2 |
06 | local DamageDis = script.Parent.Parent.Instances.DamageDisplay:Clone() |
07 | local Damage = script.Parent.Parent.Values.Damage |
08 | local DamageText = script.Parent.Parent.Instances.DamageDisplay.Damage:Clone() |
09 | local CanDamage = script.Parent.Parent.Values.CanDamage |
11 | tool.Activated:Connect(function() |
17 | Blades.Touched:Connect(function(hit) |
18 | local HumanoidRoot = hit.Parent:FindFirstChild("HumanoidRootPart") |
19 | if isUsing and HumanoidRoot and CanDamage.Value == true then |
20 | local Attachment = Instance.new("Attachment") |
21 | Attachment.Parent = HumanoidRoot |
22 | DamageDis.Parent = Attachment |
23 | DamageText.Parent = DamageDis |
24 | DamageText.Text = tostring(Damage.Value) |
25 | DamageDis.Enabled = true |
0