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

Game Design Improvements for vehicle keys #46

Closed
alberttheprince opened this issue May 6, 2024 · 16 comments · Fixed by #72
Closed

Game Design Improvements for vehicle keys #46

alberttheprince opened this issue May 6, 2024 · 16 comments · Fixed by #72
Assignees
Labels
enhancement New feature or request

Comments

@alberttheprince
Copy link
Contributor

alberttheprince commented May 6, 2024

The problem

Suggesting these features be added or reworked:

qb-vehiclekeys feature set is antiquated/needs to be added to and improved.

Ideal solution

  • Proper car starting animation (depending on vehicle, motorcycle, cars, etc)
  • engine start sound (most key resources don't play the full engine start up sound on restart
  • give key command
  • % chance world spawn cars are locked in world (config option)
  • List in config of vehicles that require lockpicks to unlock (ox lib minigame)
  • vehicles that require an advanced lockpick to unlock (harder minigame than the lockpick)
  • List of Cars that can't be unlocked at all
  • Cars with no locks/accessible to everyone (for vehicles like wheelchairs, bicycles, etc)
  • List of weapons that can and can't steal keys from drivers, with a setting for chance that some weapons are more likely
  • set change that peds will drive off: for example, 25% chance a ped tries to drive off, 10% they get and fight, 65% they give you their keys, or they try to escape
  • Police alert settings: i.e. chance to alert police when lockpicking or hotwiring
  • shared keys list for jobs
  • Lock PD/EMS/Job vehicles automatically on leaving cars
  • Config option to allow people to get into cars that are turned on and drive off
  • Remote car start config option (turn off remote start as an option)
  • Whitelist or "whitelist" of vehicles with no doorlocks, but need to be ox_lib circle game/lock picked to to unlock. Useful for things like motorcycles, which don't have doors but should require lockpicking.

Alternative solutions

N/A

Additional context

These changes would make qbx_vehiclekeys a much better resource. As it stands, qb-vehiclekeys has a pretty antiquated design and should have features added to make it more modern and useable.

I have some of these changes in the version of qb-vehiclekeys that popcorn rp uses, although it's not the greatest implementation and most of it is janky.

@alberttheprince alberttheprince added the enhancement New feature or request label May 6, 2024
@qbox-duck qbox-duck bot added this to Issues May 6, 2024
@github-project-automation github-project-automation bot moved this to Todo in Issues May 6, 2024
@TonybynMp4
Copy link
Contributor

TonybynMp4 commented May 6, 2024

  • Proper car starting animation (depending on vehicle, motorcycle, cars, etc)
  • engine start sound (most key resources don't play the full engine start up sound on restart)

Those two just sounds like bugs / the awful way vehiclekeys.. works

  • give key command

Fuck commands we're not NP, target option maybe?

  • % chance world spawn cars are locked in world (config option)

Yes please

  • List in config of vehicles that require lockpicks to unlock (ox lib minigame)

So a big ol' list of 95% of vehicles? 🤨

  • shared keys list for jobs

Vehicle DB refactor + vehiclekeys rework could allow job owned vehicles 👀

  • Lock PD/EMS/Job vehicles automatically on leaving cars

I feel like that kinda sucks, if you forget to Lock it's on you, get good bro? 🤔

  • Config option to allow people to get into cars that are turned on and drive off

Should be a config only if we're trying to please NP fanbois, if a car is on it's on duh

  • Remote car start config option (turn off remote start as an option)

Wouldn't be a vehiclekeys thing IMO

@alberttheprince
Copy link
Contributor Author

Those two just sounds like bugs / the awful way vehiclekeys.. works

They're not really bugs, I think, they just need to be added in as they're not there. I'd have to check with @Manason, but we added these a long time ago and it just has to do with forcing the player to do the animations. Not sure.

I think there's a specific native for this?

Fuck commands we're not NP, target option maybe?

Commands are good depending on the server style, not necessarily NP specific. A lot of servers use commands or are used to them. Qbox uses them for /transfervehicle for example in the vehicleshop. Would be good to have these as available options in the config. Or if you're in a vehicle.

So a big ol' list of 95% of vehicles? 🤨

Let me clarify, it would be a list of vehicles that always start locked. So for example, police motorcycle would always spawn locked and requiring a lockpick or the rhino tank when spawned would require advanced lockpick. Vehicles not on this list would default to the % chance world lock.

Vehicle DB refactor + vehiclekeys rework could allow job owned vehicles 👀

it already works like this, but it's only for police/ambulance job. Better to make it something like job = police in the config, for users anyways.

I feel like that kinda sucks, if you forget to Lock it's on you, get good bro?

Depends. For example on our server, we use the vehicle glovebox as our weapon storage for cops. So this would be useful depending on the type of server/rp level.

Wouldn't be a vehiclekeys thing IMO (remote start)

This is already how qb-vehiclekeys work, it's always on. Would be nice to turn it off so people need to turn their car on/off when sitting in there.

@artur-michalak
Copy link
Contributor

Proper car starting animation (depending on vehicle, motorcycle, cars, etc) 👎

I have not found a native solution to this problem (sadly). Preparing animations for each vehicle can be too time-consuming.

engine start sound (most key resources don't play the full engine start up sound on restart 👍

give key command👍

% chance world spawn cars are locked in world (config option) 👍

List in config of vehicles that require lockpicks to unlock (ox lib minigame) 👍

vehicles that require an advanced lockpick to unlock (harder minigame than the lockpick)

need to make a difficult mini-game

List of Cars that can't be unlocked at all👍

Cars with no locks/accessible to everyone (for vehicles like wheelchairs, bicycles, etc)

to do

List of weapons that can and can't steal keys from drivers, with a setting for chance that some weapons are more likely👍

set change that peds will drive off: for example, 25% chance a ped tries to drive off, 10% they get and fight, 65% they give you their keys, or they try to escape

need for analysis

Police alert settings: i.e. chance to alert police when lockpicking or hotwiring

Can you describe it in more detail?

shared keys list for jobs👍

Lock PD/EMS/Job vehicles automatically on leaving cars

to do

Config option to allow people to get into cars that are turned on and drive off

When getting into the vehicle, should you get the keys to it?

Remote car start config option (turn off remote start as an option)👍

if the car is running after leaving the garage, it is set in the garages script.

Whitelist or "whitelist" of vehicles with no doorlocks, but need to be ox_lib circle game/lock picked to to unlock. Useful for things like motorcycles, which don't have doors but should require lockpicking.👍

I marked what should be done. Most of the features should already be available, feel free to test and make comments.

@alberttheprince
Copy link
Contributor Author

Hi Artur, Thanks for your work on this!

I have not found a native solution to this problem (sadly). Preparing animations for each vehicle can be too time-consuming.

I posted a bandaid solution some time ago in Qbox DIscord snippets, it may give you an idea. Basically I set up 3 different animations for different vehicle categories, although the way I did it was really amateur/not clean.

need to make a difficult mini-game

For the sake of Qbox, probably just a more difficult circle game. We use a native data track mini game to hack to both lock pick the car and the keys of high value vehicles. https://github.com/utkuali/datacrack

need for analysis (weapon list)

What do you need? As in you need the list? Basically just forcing players to use gun, not melee weapons to steal cars.

Can you describe it in more detail? (police alerts)

At the moment vehicle keys will trigger when a person is hot wiring a car, not breaking into it. Change this so if someone breaks into or gets into a world vehicle they don't own there is a % chance police can be alerted.

When getting into the vehicle, should you get the keys to it?

Yes but only if the car has been left on. For example: a player leaves their engine running and doors unlocked, I sit in the seat and I would get keys and be able to drive off.

if the car is running after leaving the garage, it is set in the garages script. (remote start)

More specifically at the moment players can turn their engine off from a distance (they press engine off and the car will turn off or on from a distance, not sure but I think it's something like 5 meters? This should be something you can disable.

Will test what has been done when I have time and let you know.

@artur-michalak
Copy link
Contributor

artur-michalak commented Jun 28, 2024

More specifically at the moment players can turn their engine off from a distance (they press engine off and the car will turn off or on from a distance, not sure but I think it's something like 5 meters? This should be something you can disable.

I see, qb-vehiclekeys works like this. It looks like a bug, so in qbx_vehiclekeys it is possible to start the engine only while in the vehicle. Should it be possible to start the car remotely?

What do you need? As in you need the list? Basically just forcing players to use gun, not melee weapons to steal cars.

What I meant to say was that the chance of a ped escaping requires analysis

List of weapons that can and can't steal keys from drivers, with a setting for chance that some weapons are more likely

Works fine

@artur-michalak
Copy link
Contributor

artur-michalak commented Jul 4, 2024

this branch has most of these features
https://github.com/Qbox-project/qbx_vehiclekeys/tree/renew
Let me know if you catch any problems

@alberttheprince
Copy link
Contributor Author

I see, qb-vehiclekeys works like this. It looks like a bug, so in qbx_vehiclekeys it is possible to start the engine only while in the vehicle. Should it be possible to start the car remotely?

I think remote on/off was unintended in qb-vehiclekeys, but I think some people do prefer this system. I don't, but always assumed it was functionality that people wanted/used.

What I meant to say was that the chance of a ped escaping requires analysis

I think a simple break down like:

75% chance to give up, 15% chance to drive off, 10% chance to get out and fight would be dynamic enough. By giving players a chance they get injured or have to kill the driver it drives RP for other interactions like EMS, LSPD, etc.

@artur-michalak
Copy link
Contributor

artur-michalak commented Jul 7, 2024

I think remote on/off was unintended in qb-vehiclekeys, but I think some people do prefer this system. I don't, but always assumed it was functionality that people wanted/used.

I would like to keep the script as simple as possible, so I would add such a feature only when needed

I think a simple break down like

I mean it is not a simple task, depending on the native functions that you need to find

@artur-michalak
Copy link
Contributor

Lock PD/EMS/Job vehicles automatically on leaving cars

blocked by overextended/ox_lib#618

@artur-michalak
Copy link
Contributor

At the moment vehicle keys will trigger when a person is hot wiring a car, not breaking into it. Change this so if someone breaks into or gets into a world vehicle they don't own there is a % chance police can be alerted.

I think it's handled

@alberttheprince
Copy link
Contributor Author

I mean it is not a simple task, depending on the native functions that you need to find

I think this is a feature of qb-vehicle keys already; there is a chance of the ped driving over you. I just think it doesn't actually work.

@artur-michalak
Copy link
Contributor

I think this is a feature of qb-vehicle keys already; there is a chance of the ped driving over you. I just think it doesn't actually work.

The features that were in qb-vehiclekeys work. There is no option here to control who tries to drive off, who fights.

@alberttheprince
Copy link
Contributor Author

The features that were in qb-vehiclekeys work. There is no option here to control who tries to drive off, who fights.

I guess I misunderstood, I think at the moment the chance is just if the ped gives you keys or tries to drive off. I'm not sure why I thought there was a chance for them to fight you.

@artur-michalak
Copy link
Contributor

then in that case branch "renew" should have all functions

@Manason
Copy link
Member

Manason commented Jul 9, 2024

then in that case branch "renew" should have all functions

Looks like it's marked as draft. Is it ready for review?

@artur-michalak
Copy link
Contributor

I think it's too complex. I will gradually upload it to the main. Functionally it should be ready to use.

@artur-michalak artur-michalak mentioned this issue Jul 19, 2024
2 tasks
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants