Remove Folder When Player Leave the game

[MrRBX_Dev (Someone)]

Hi!
How to remove folder from workspace when player leave from the game?

This is my script:

local prompt = script.Parent
local appPlayer = nil
local plot = script.Parent.Parent.Parent.Parent

prompt.Triggered:Connect(function(player1)
	appPlayer = player1
	player1.PlayerData.IsPlayerHavePlot.Value = true
	script.Parent.Parent.Parent.Parent.Claimed.Value = true
	script.Parent.Enabled = false
	local office = game.ReplicatedStorage.Models.Offices.OfficeLvl1
	local clone = office:Clone()
	clone.Parent = plot.Offices
	local model = clone
	model:MoveTo(plot.Position)
end)

game.Players.PlayerRemoving:connect(function(player) -- This part
	if player.Name == appPlayer then
		prompt.Enabled = true
		script.Parent.Parent.Parent.Parent.Claimed.Value = false
		plot.Offices:Destroy()
		Instance.new("Folder", plot).Name = "Offices"
	end
	end)

[Moleza]

game.Players.PlayerRemoving:connect(function(player) -- This part
Instance.new("Folder", plot).Name = "Offices"

this line, why in the world you will create a Folder, if the player is leaving?
you dind’t wanted to Delete it? so why you are creating it…

[MrRBX_Dev (Someone)]

I want to delete this folder to remove all models in this folder

[MrRBX_Dev (Someone)]

Ok, I solved this now

Thanks for all replies

[Babybunnyiscute19]

Don’t use Instance.new("Folder", plot), it’s a lot slower and can do some harm to your game, I would link a topic explaining this but I can’t find it, just use:

local folder = Instance.new("Folder")
folder.Name = "Offices"
folder.Parent = plot

[Limited_Unique]

PSA: Don't use Instance.new() with parent argument

I’ve discovered a pretty bad performance issue in one of top games that has to do with Instance.new and wanted to write about this, since this is not obvious unless you know the system inside out. Tthere are several ways to create a ROBLOX object in Lua: local obj = Instance.new(‘type’); fill obj fields local obj = Instance.new(‘type’, parent); fill obj fields local obj = util.Create(‘type’, { field1 = value1, … }) If you care at all about performance, please only use the first option - I wi…

[Limited_Unique]

local prompt = script.Parent
local appPlayer = nil
local plot = script.Parent.Parent.Parent.Parent
local players = game:GetService("Players")
local storage = game:GetService("ReplicatedStorage")

prompt.Triggered:Connect(function(player)
	appPlayer = player
	player.PlayerData.IsPlayerHavePlot.Value = true
	script.Parent.Parent.Parent.Parent.Claimed.Value = true
	script.Parent.Enabled = false
	local office = storage.Models.Offices.OfficeLvl1
	local clone = office:Clone()
	clone.Parent = plot.Offices
	local model = clone
	model:MoveTo(plot.Position)
end)

players.PlayerRemoving:connect(function(player) -- This part
	if player.Name == appPlayer.Name then
		prompt.Enabled = true
		script.Parent.Parent.Parent.Parent.Claimed.Value = false
		plot.Offices:Destroy()
	end
end)

You were attempting to compare a player instance with a player name (which is a string value) which would always end up as evaluating as false.