Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cannot use vehicle keys because export is only for qbx_vehiclekeys #209

Closed
alberttheprince opened this issue Oct 24, 2023 · 5 comments · Fixed by #242
Closed

cannot use vehicle keys because export is only for qbx_vehiclekeys #209

alberttheprince opened this issue Oct 24, 2023 · 5 comments · Fixed by #242
Labels
bug Something isn't working need repro This bug report needs confirmation
Milestone

Comments

@alberttheprince
Copy link
Contributor

Summary

unless you use qbx_vehiclekeys, givekeys breaks

Reproduction

use any vehiclekey resource other than qbx_vehiclekeys. For example qb-vehiclekeys.

Expected behavior

expected behavior is that regardless of what vehiclekey system you use you should be able to trigger the event.

Actual behavior

Car ownership, and a number of other car related things break due to car keys not being given to players.

Additional context

Maybe move this out of core?

Last Updated

Today

Custom Resources

modified qb-vehiclekeys/custom vehiclekeys resource

Resource Rename

no

@alberttheprince alberttheprince added bug Something isn't working need repro This bug report needs confirmation labels Oct 24, 2023
@Manason
Copy link
Member

Manason commented Oct 24, 2023

We should probably just move the command out of core. Seems like the vehicle key resources should own those commands.

@Manason
Copy link
Member

Manason commented Oct 24, 2023

On second thought moving the command might break backwards compatibility. Maybe best to trigger a 'qbx_core:server:giveKeys' event then?

@SantyR0
Copy link

SantyR0 commented Oct 24, 2023

On second thought moving the command might break backwards compatibility. Maybe best to trigger a 'qbx_core:server:giveKeys' event then?

Sorry for asking, but where should be "qbx_core:server:giveKeys" be placed? Also, "use any vehiclekey resource other than qbx_vehiclekeys. For example qb-vehiclekeys." it seems that even in fresh recipes, even with the qbx-vehiclekeys this problem persists when taking vehicles out from impound/garage.

@Manason
Copy link
Member

Manason commented Oct 24, 2023

On second thought moving the command might break backwards compatibility. Maybe best to trigger a 'qbx_core:server:giveKeys' event then?

Sorry for asking, but where should be "qbx_core:server:giveKeys" be placed? Also, "use any vehiclekey resource other than qbx_vehiclekeys. For example qb-vehiclekeys." it seems that even in fresh recipes, even with the qbx-vehiclekeys this problem persists when taking vehicles out from impound/garage.

Instead of the export, triggering the event within the /giveKeys command in core would allow any vehicle key resource to consume the event to give keys

@Manason
Copy link
Member

Manason commented Oct 24, 2023

On third thought, the export does have the property that it will cause an error if it doesn't exist. This might be a good thing as it fails loudly so that vehicle key resources know they need to provide the export. Versus an event would fail silently if unimplemented. This does add a dependency on a vehicle key resource to core. Although that can easily be fixed by guarding it with a resource started if condition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need repro This bug report needs confirmation
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants