Skip to content

Allow client peds to enter/exit client vehicles#3678

Closed
gownosatana wants to merge 14 commits into
multitheftauto:masterfrom
gownosatana:cpeds
Closed

Allow client peds to enter/exit client vehicles#3678
gownosatana wants to merge 14 commits into
multitheftauto:masterfrom
gownosatana:cpeds

Conversation

@gownosatana

Copy link
Copy Markdown
Contributor

This pull request allows client created peds to enter client created vehicles.

setPedEnterVehicle(Element ped, [Element vehicle, bool passenger])
When no vehicle is specified ped will enter closest client vehicle

Test resource: test.zip

Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread vendor/google-breakpad/src/client/linux/handler/minidump_descriptor.h Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
@tederis tederis added the enhancement New feature or request label Aug 28, 2024
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
@FileEX

FileEX commented Sep 2, 2024

Copy link
Copy Markdown
Member

Shouldn't we also use the onClientPedVehicleExit / onClientPedVehicleEnter events for peds here?

@gownosatana

gownosatana commented Sep 2, 2024

Copy link
Copy Markdown
Contributor Author

Shouldn't we also use the onClientPedVehicleExit / onClientPedVehicleEnter events for peds here?

yep, it works here too, it's tested

@gownosatana

Copy link
Copy Markdown
Contributor Author

All done

Comment thread Client/mods/deathmatch/logic/CClientPed.cpp
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp
Comment thread Client/mods/deathmatch/logic/CClientPed.cpp Outdated
@gownosatana

gownosatana commented Sep 6, 2024

Copy link
Copy Markdown
Contributor Author

@tederis review please ✨

@FileEX

FileEX commented Sep 6, 2024

Copy link
Copy Markdown
Member

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

@gownosatana

Copy link
Copy Markdown
Contributor Author

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit

@gownosatana

Copy link
Copy Markdown
Contributor Author

Also I just want to add you're able to make comments and reviews but you're not one deciding here if pull request should be merged or not, thanks 🙏

@TracerDS

TracerDS commented Sep 6, 2024

Copy link
Copy Markdown
Contributor

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit

Actually, you didnt. You pointlessly resolved conversations without adding the requested changes

@gownosatana

gownosatana commented Sep 6, 2024

Copy link
Copy Markdown
Contributor Author

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit

Actually, you didnt. You pointlessly resolved conversations without adding the requested changes

I did in commit e461e70, I'm sorry for your blindness, other changes are approved by fileex.

@FileEX FileEX left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR already looks good, which unfortunately cannot be said about the others

@TracerDS

TracerDS commented Sep 6, 2024

Copy link
Copy Markdown
Contributor

I did in commit e461e70, I'm sorry for your blindness, other changes are approved by fileex.

Being rude to others does not benefit you whatsoever. You only added the type but did not remove the braces; i.e. You acknowledged 1 out of 3 conversations.

@Dutchman101

Dutchman101 commented Sep 6, 2024

Copy link
Copy Markdown
Member

@gownosatana Generally speaking, over all your recent PR's and code reviews on it, you are creating a lot of unnecesary drama, and you don't want to agree to do things the way this project has implemented.

You are this close to an organisation (repository) ban and the deletion of all your PR's, so im just warning you to keep yourself in check. Do not reply to this comment. Start showing some more respect, and cut the drama/combativeness.

You're not gonna get things the way you want it; the world or this project doesn't revolve around you. If you don't like that, you are free to just go away.

@CrosRoad95

This comment was marked as abuse.

@Dutchman101

Copy link
Copy Markdown
Member

This PR is being closed for the same reason as the comment explains on the other one closed.

@Dutchman101 Dutchman101 closed this Oct 5, 2024
@multitheftauto multitheftauto locked as too heated and limited conversation to collaborators Oct 5, 2024
@botder botder modified the milestone: 1.6.1 Dec 22, 2024
@botder botder added this to the 1.6.1 milestone Jan 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants