Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add ability to make client-side vehicles damageable #4411

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

Open
MegadreamsBE wants to merge 2 commits into multitheftauto:master
base: master
Choose a base branch
Loading
from MegadreamsBE:feature/client-side-vehicle-damage

Conversation

@MegadreamsBE
Copy link
Member

@MegadreamsBE MegadreamsBE commented Sep 3, 2025
edited
Loading

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.

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?

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.

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

Copy link
Member Author

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

Fernando-A-Rocha reacted with heart emoji

Copy link
Contributor

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

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

Reviewers

1 more reviewer

@Fernando-A-Rocha Fernando-A-Rocha Fernando-A-Rocha approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /