refactor(callbacks): remove the cb argument

BREAKING CHANGE: Registered callbacks now return values rather than sending them as a callback.

Previously the cb needed to be called to return to the client, now it will happen once the function closes. i.e.

cb(string, string, false) -> return string, string, false

fxmanifest.lua

@@ -23,7 +23,7 @@ game         'gta5'
 --[[ Resource Information ]]--
 name         'pe-lualib'
 author       'Linden'
-version      '1.2.1'
+version      '1.3.0'
 repository   'https://github.com/project-error/pe-lualib'
 description  'A library of shared functions to utilise in other resources.'
 

imports/callbacks/client.lua

@@ -29,8 +29,8 @@ local ServerCallback = table.create(0, 2)
 ServerCallback.Await = function(resource, event, delay, ...)
 	if callbackTimer(event, delay) then
 		local promise = promise.new()
-		event = RegisterNetEvent(startCallback(resource, event, ...), function(...)
+		event = RegisterNetEvent(startCallback(resource, event, ...), function(response)
-			promise:resolve({...})
+			promise:resolve(response)
 			RemoveEventHandler(event)
 		end)
 		return table.unpack(Citizen.Await(promise))
@@ -44,8 +44,8 @@ end
 --- Sends an event to the server and triggers a callback function once the response is returned.
 ServerCallback.Async = function(resource, event, delay, cb, ...)
 	if callbackTimer(event, delay) then
-		event = RegisterNetEvent(startCallback(resource, event, ...), function(...)
+		event = RegisterNetEvent(startCallback(resource, event, ...), function(response)
-			cb(...)
+			cb(table.unpack(response))
 			RemoveEventHandler(event)
 		end)
 	end

imports/callbacks/server.lua

@@ -1,17 +1,14 @@
 local ServerCallback = {}
 
-	---@param name string
+---@param name string
-	---@param callback function
+---@param callback function
-	--- Registers an event handler and callback function to respond to client requests.
+--- Registers an event handler and callback function to respond to client requests.
-	ServerCallback.Register = function(name, callback)
+ServerCallback.Register = function(name, callback)
 	name = ('__cb_%s:%s'):format(GetCurrentResourceName(), name)
 
 	RegisterServerEvent(name, function(id, ...)
-			local source = source
+		TriggerClientEvent(name..id, source, {callback(source, ...)})
-			callback(source, function(...)
-				TriggerClientEvent(name..id, source, ...)
-			end, ...)
 	end)
-	end
+end
 
 return ServerCallback