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

Arsenal - Conditional on sorts #7889

Merged
merged 4 commits into from
Nov 2, 2020

Conversation

BrettMayson
Copy link
Member

When merged this pull request will:

@PabstMirror PabstMirror added this to the 3.13.5 milestone Sep 1, 2020
@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Sep 1, 2020
@PabstMirror
Copy link
Contributor

Open arsenal and click on vest:

private _sortConfig = _sorts select (_sorts findIf {(_x select 0) isE>
Error position: <select (_sorts findIf {(_x select 0) isE>
Error Zero divisor
File z\ace\addons\arsenal\functions\fnc_sortPanel.sqf..., line 101

@PabstMirror PabstMirror modified the milestones: 3.13.5, 3.14.0 Oct 29, 2020
@BrettMayson
Copy link
Member Author

Which arsenal did you use? Main menu, editor, in game

I can have this fixed in the next few hours

@BrettMayson
Copy link
Member Author

@PabstMirror Fixed

@PabstMirror PabstMirror modified the milestones: 3.14.0, 3.13.5 Oct 30, 2020
Copy link
Contributor

@PabstMirror PabstMirror left a comment

Choose a reason for hiding this comment

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

tested working

@PabstMirror
Copy link
Contributor

Looks good, I'm not sure if there is anything else we would want to pass to the condition code?
We can always add params later if needed

@PabstMirror PabstMirror merged commit 9226035 into acemod:master Nov 2, 2020
@BaerMitUmlaut
Copy link
Member

_x params ["_sortName", "_displayName", "", "_condition"]

Missing semicolon!

@BrettMayson BrettMayson deleted the arsenal_quantity branch September 1, 2022 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ACE Arsenal: Can't sort by amount of items in loadout anymore.
3 participants