Replace breaks with Returns

currently when adding multiple jobs or items or gangs if there is a duplicate it breaks the loop instead of returning and going to the next item.
This commit is contained in:
KamuiKody
2022-10-19 01:46:04 -04:00
committed by GitHub
parent d89662950b
commit b9576a6084
+6 -6
View File
@@ -61,14 +61,14 @@ local function AddJobs(jobs)
message = 'invalid_job_name'
shouldContinue = false
errorItem = jobs[key]
break
return
end
if QBCore.Shared.Jobs[key] then
message = 'job_exists'
shouldContinue = false
errorItem = jobs[key]
break
return
end
QBCore.Shared.Jobs[key] = value
@@ -171,14 +171,14 @@ local function AddItems(items)
message = "invalid_item_name"
shouldContinue = false
errorItem = items[key]
break
return
end
if QBCore.Shared.Items[key] then
message = "item_exists"
shouldContinue = false
errorItem = items[key]
break
return
end
QBCore.Shared.Items[key] = value
@@ -244,14 +244,14 @@ local function AddGangs(gangs)
message = "invalid_gang_name"
shouldContinue = false
errorItem = gangs[key]
break
return
end
if QBCore.Shared.Gangs[key] then
message = "gang_exists"
shouldContinue = false
errorItem = gangs[key]
break
return
end
QBCore.Shared.Gangs[key] = value