Skip to content

Server side weapon reloading #3936

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

Merged
merged 7 commits into from
Jan 10, 2025
Merged

Conversation

Nico8340
Copy link
Contributor

@Nico8340 Nico8340 commented Jan 9, 2025

Description

This pull request enhances the synchronization of weapon reloading for both peds and player with making it server sided. Refactors existing functions, introduces a new one, and adds two new events as well. Closes #1525.

Function

POP:

bool isPedReloadingWeapon(element ped)

OOP:
Method: ped:isReloadingWeapon()
Variable ped.reloadingWeapon

Events

"onPedWeaponReload"
"onPlayerWeaponReload"

Example

reloadtest.zip

Nico8340 and others added 4 commits January 9, 2025 18:50
Co-authored-by: Marek Kulik <noreply@bug32.com>
Co-authored-by: Marek Kulik <noreply@bug32.com>
@Nico8340 Nico8340 requested a review from botder January 10, 2025 05:23
@Nico8340 Nico8340 requested a review from tederis January 10, 2025 07:06
@botder botder merged commit e71f482 into multitheftauto:master Jan 10, 2025
6 checks passed
@botder botder added the enhancement New feature or request label Jan 10, 2025
@botder botder added this to the 1.6.1 milestone Jan 10, 2025
@Nico8340 Nico8340 deleted the reloading branch January 10, 2025 20:22
MTABot pushed a commit that referenced this pull request Jan 10, 2025
e71f482 Add server side weapon reloading (PR #3936, Fixes #1525)
@Nico8340 Nico8340 mentioned this pull request Jan 18, 2025
@lopezloo
Copy link
Member

lopezloo commented May 5, 2025

Resolves #2747

@lopezloo lopezloo linked an issue May 5, 2025 that may be closed by this pull request
1 task
@lopezloo lopezloo mentioned this pull request May 5, 2025
1 task
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.

Synchronize weapon reload task Serverside for isPedReloadingWeapon
4 participants