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

Medical - Fix script error when bandaging #9956

Merged
merged 4 commits into from
Apr 19, 2024

Conversation

tbeswick96
Copy link
Contributor

@tbeswick96 tbeswick96 commented Apr 18, 2024

When merged this pull request will:

  • On bandage success callback, overwrite the value of createLitter set by treatment to 1 for bandage effectiveness change bandage effectiveness to not use the same args index as createLitter.
  • I could not see where an actual value for bandage effectiveness was being passed through, so could also remove this value and its usages entirely
  • See discord thread for more context and explanation

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

addons/medical_treatment/functions/fnc_bandage.sqf Outdated Show resolved Hide resolved
@johnb432 johnb432 added this to the 3.17.1 milestone Apr 19, 2024
@johnb432 johnb432 added the kind/bug-fix Release Notes: **FIXED:** label Apr 19, 2024
Copy link
Contributor

@LinkIsGrim LinkIsGrim left a comment

Choose a reason for hiding this comment

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

need to update docs @https://github.com/acemod/ACE3/blob/master/docs/wiki/framework/events-framework.md?plain=1#L45

can do so myself.

changelog/release might want to mention we're breaking API for bugfix? I don't think anyone actually uses this yet though

addons/medical_treatment/functions/fnc_bandage.sqf Outdated Show resolved Hide resolved
Co-authored-by: Grim <69561145+LinkIsGrim@users.noreply.github.com>
@johnb432
Copy link
Contributor

johnb432 commented Apr 19, 2024

changelog/release might want to mention we're breaking API for bugfix? I don't think anyone actually uses this yet though

Doesn't hurt, but docs were outdated even before this issue, so idk how much it will help.

@LinkIsGrim LinkIsGrim changed the title Medical - Fix bandage effectiveness type Medical - Fix script error when bandaging Apr 19, 2024
@LinkIsGrim LinkIsGrim merged commit 23842aa into acemod:master Apr 19, 2024
5 checks passed
@tbeswick96 tbeswick96 deleted the fix-bandage-effectiveness-type branch April 19, 2024 21:22
Krzyciu added a commit to Arma4Ever/A4ES_A3 that referenced this pull request Apr 20, 2024
SzwedzikPL pushed a commit to Arma4Ever/A4ES_A3 that referenced this pull request Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants