Skip to content

Conversation

MegadreamsBE
Copy link
Member

@MegadreamsBE MegadreamsBE commented Sep 3, 2025

This PR makes it possible to mark a client-side vehicle to be damageable. By default client-side vehicles cannot receive damage, this PR does not change that default behavior.

When false is passed to SetVehicleDamageProof for a local vehicle, it will now become damageable.

@Fernando-A-Rocha
Copy link
Contributor

I think the createVehicle function has enough arguments. Wouldn't it be wise to make them be damageable with like setVehicleDamageProof false?

@MegadreamsBE
Copy link
Member Author

I think the createVehicle function has enough arguments. Wouldn't it be wise to make them be damageable with like setVehicleDamageProof false?

It crossed my mind, but I'd still need to have two variables internally anyways... felt silly in that regards. I could of course change the default behavior, but many servers probably wouldn't appreciate that.

I'll see what I can do.

@MegadreamsBE
Copy link
Member Author

I think the createVehicle function has enough arguments. Wouldn't it be wise to make them be damageable with like setVehicleDamageProof false?

It crossed my mind, but I'd still need to have two variables internally anyways... felt silly in that regards. I could of course change the default behavior, but many servers probably wouldn't appreciate that.

I'll see what I can do.

I could just set the variable that handles damage proof to true by default for a local vehicle.... Okay, I'll make the change

@MegadreamsBE
Copy link
Member Author

@Fernando-A-Rocha Updated! Your message made me realize the easy solution to making it work through SetVehicleDamageProof. Thanks!

Copy link
Contributor

@Fernando-A-Rocha Fernando-A-Rocha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems ok to me, less complicated than the initial concept 😅 but other devs can share their opinion

@FileEX FileEX added the enhancement New feature or request label Sep 4, 2025
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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants