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

Fixes an exploit that allows you to infinitely stack damage multipliers #8180

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Watermelon914
Copy link
Contributor

@Watermelon914 Watermelon914 commented Jan 20, 2025

About the pull request

As the title says.

Closes #8179

Explain why it's good for the game

Modifying gun damage this way is really buggy and led to a bug (#8179) that lets people stack a damage multiplier infinitely.

A more comprehensive solution would be to, instead, have a gun instance stored in this attachment so that it handles all its own variables, damage multiplers, et cetera, instead of trying to modify the original weapon. This would require an overhaul and is not something I'm opposed to someone doing, but this simple solution should suffice for fixing the exploit.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
fix: Fixed an exploit that lets you increase your weapon's damage multiplier infinitely.
/:cl:

@Watermelon914 Watermelon914 added Fix Fix one bug, make ten more Balance You need to be a professional veteran game maintainer to comprehend what is being done here. labels Jan 20, 2025
@Watermelon914 Watermelon914 changed the title Allows gun attachments to retain the gun's original damage Fixes an exploit that allows you to infinitely stack damage multipliers Jan 20, 2025
@Watermelon914 Watermelon914 removed the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
Status: Awaiting Review
Development

Successfully merging this pull request may close these issues.

Damage multiplier exploit
1 participant