Tool Clone Error

Help and Feedback Scripting Support
#1

Im trying to make a clone tool for my game, but when I try to it displays an error.

Client:

local mouse = game.Players.LocalPlayer:GetMouse()
local db = false


mouse.Button1Down:Connect(function()
	if db == false and mouse.Target.Anchored == false then
		db = true

		local Target = mouse.Target

		Target.CanCollide = false
		Target.Parent = workspace

		mouse.Button1Down:Connect(function()


			if mouse.Target.Anchored == false then
				local newterget = mouse.Target:Clone()
				newterget.Parent = workspace
				newterget.Anchored = true
				
				mouse.TargetFilter = newterget

				mouse.Move:Connect(function(hit)

					newterget.Position = mouse.Hit.Position
					newterget.Position = Vector3.new(math.round(newterget.Position.X,1),math.round(newterget.Position.Y,1),math.round(newterget.Position.Z,1))
					newterget.Position = newterget.Position + Vector3.new(0,newterget.Size.Y / 2,0)

					mouse.Button1Up:Connect(function(hit)

						script.Parent.RemoteEvent:FireServer(newterget)
						script.Parent.SFX:Play()


					end)

				end)
			end


		end)

		wait(2)
		db = false
	end

end)

server:

script.Parent.RemoteEvent.OnServerEvent:Connect(function(player, part)
	local cln = part:Clone()
	cln.Anchored = false
	cln.Parent = workspace
	cln.CanCollide = true
end)

error:
image

#2

Why the double event functions?
image
image

Cloning in a client script?
image

#3

Too tired to help you much here but heads up that instead of doing
if db == false and mouse.Target.Anchored == false then
you can just do
if (not db) and (not mouse.Target.Anchored) then

#4

The issue might be to do with the actual part that you’re sending through to the server, because you’re trying to clone something from the client
image

#5

yea but i need the exact properties to clone from the part from the client so what should I do

#6

Properties of what?
(limit)(limit)(limit)

#7

the variable newtarget in the client script is the part thats being cloned

#8

what exactly is

mouse.Target
#9

its the mosues target, like what part its hovering over

#10

The code is very terrible but,

this right here is the culprit.
you created a LocalPart
and you fired the RemoteEvent with that LocalPart

a Local part is a part that only exist on the client. and Trying to fire it to server the server will get it back as nil.
so the LocalPart created from

mouse.Target
you should save the part it was cloned from as a variable

local Target=mouse.Target
local ClonedObject=Target:Clone()--replaced newterget

and right here

replace newterget with the new Target
here is the new code

local mouse = game.Players.LocalPlayer:GetMouse()
local db = false


mouse.Button1Down:Connect(function()
	if db == false and mouse.Target.Anchored == false then
		db = true

		local Target = mouse.Target

		Target.CanCollide = false
		Target.Parent = workspace

		mouse.Button1Down:Connect(function()


			if mouse.Target.Anchored == false then
				local Target = mouse.Target
				local ClonedObject = Target:Clone()
				ClonedObject.Parent = workspace
				ClonedObject.Anchored = true

				mouse.TargetFilter = ClonedObject

				mouse.Move:Connect(function(hit)

					ClonedObject.Position = mouse.Hit.Position
					ClonedObject.Position = Vector3.new(math.round(ClonedObject.Position.X,1),math.round(ClonedObject.Position.Y,1),math.round(ClonedObject.Position.Z,1))
					ClonedObject.Position = ClonedObject.Position + Vector3.new(0,ClonedObject.Size.Y / 2,0)

					mouse.Button1Up:Connect(function(hit)

						script.Parent.RemoteEvent:FireServer(mouse.Target)
						script.Parent.SFX:Play()


					end)

				end)
			end


		end)

		wait(2)
		db = false
	end

end)

I didnt debug this code but if you want me to just message me again.

1 Like
#11

I had to adjust some stuff but it clones now correctly Thank you

#12

This topic was automatically closed 14 days after the last reply. New replies are no longer allowed.