Skip to content

feat: add GetOnDuty exports#1162

Closed
Cocodrulo wants to merge 17 commits intoqbcore-framework:mainfrom
Cocodrulo:feature/availablejobs
Closed

feat: add GetOnDuty exports#1162
Cocodrulo wants to merge 17 commits intoqbcore-framework:mainfrom
Cocodrulo:feature/availablejobs

Conversation

@Cocodrulo
Copy link
Contributor

@Cocodrulo Cocodrulo commented Jan 9, 2025

Description

Added GetOnDuty exports both client and server side. In order to add this QBCore.Functions.GetOnDuty was added client-side and a callback was added server-side.

-- Client side
local cops_func = QBCore.Functions.GetDutyCount('police') -- int
local cops_exp = exports['qb-core']:GetDutyCount('police') -- int

-- Server side
local cops_func = QBCore.Functions.GetDutyCount('police') -- int
local cops_exp = exports['qb-core']:GetDutyCount('police') -- int

Checklist

  • I have personally loaded this code into an updated qbcore project and checked all of its functionality.
  • My code fits the style guidelines.
  • My PR fits the contribution guidelines.

@Mustachedom
Copy link
Contributor

one thing i would add is a check to see if job has a type attached

so like

if QBCore.Shared.Jobs[job].type == nil then 
  check logic for name 
else
   Check logic for type 
 end

so that way it will pool all people that can do those jobs

or even a bool as a second param so people can do something like

QBCore.Functions.GetOnDuty('police', true)
and it would check if param2 then check by type assigned to police
if false or nil then check by name

@Cocodrulo
Copy link
Contributor Author

Added! (I think)

@alfaritsii
Copy link

Suggestion: Utilize QBCore:Client:OnPlayerLoaded and QBCore:Client:OnJobUpdate to update the number of duty jobs and store it in the global statebag instead of loop from the table when the function is called, that will make the data can be accessed from both the server side and client side.

@Cocodrulo
Copy link
Contributor Author

Well, idk if thats a better solution, it to messy I think, but i'll look into it?

@JericoFX
Copy link
Contributor

the global state bag is a good addition, but is shud make into the core itself so you can return later GlobalState[jobname]

@Cocodrulo
Copy link
Contributor Author

IDK really, I think is messier, but I'll look into it

@Cocodrulo
Copy link
Contributor Author

Added the use of statebags, let me know what you think

@GhzGarage
Copy link
Member

I added exports for all qb functions

@GhzGarage GhzGarage closed this Jun 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants