Captives - Add modded cable tie support

[SpicyBagpipes]

When merged this pull request will:

• Title, child classes will be compatible

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}.

[PabstMirror]

need to update ACE_captives_fnc_doApplyHandcuffs as well

[SpicyBagpipes]

need to update ACE_captives_fnc_doApplyHandcuffs as well

Just want first instance of kind of ace_cabletie or child found removed?

[johnb432]

Why not go with the approach found here?
https://github.com/acemod/ACE3/blob/master/addons/trenches/functions/fnc_hasEntrenchingTool.sqf

[SpicyBagpipes]

Why not go with the approach found here? https://github.com/acemod/ACE3/blob/master/addons/trenches/functions/fnc_hasEntrenchingTool.sqf

Don't see any benefit to it in this instance as no backpacks etc for entrenching tools and anyone who has already tried to create their own likely inherits from cable tie

[johnb432]

Don't see any benefit to it in this instance as no backpacks etc for entrenching tools and anyone who has already tried to create their own likely inherits from cable tie

Reasonable.

The only benefit I see in adding a function is for public usage, but I'm not sure how needed such a function would be.

[commy2]

I propose making it an attribute. I don't like using inheritance for this.

ace_captives_is_cable_tie = 1;

[Drofseh]

I think isRestraint might be better than isCabletie.
It's more generic and fits better for rope or handcuff items

[SpicyBagpipes]

I think isRestraint might be better than isCabletie. It's more generic and fits better for rope or handcuff items

As is not exposed to end users I think isCabletie is better as is in keeping with ace item name

[SpicyBagpipes]

Though not opposed to adding a rope restraint or something to expand on it? 🤷‍♀️

[BaerMitUmlaut]

Attribute is currently not set on the default cable ties.

[SpicyBagpipes]

Can do this for all the other "tools" currently present (think it's just the defusal kit left with no modded support) if desired, changes only add .0012ms to the condition.

Makes no sense to leave the defusal kit out if all the rest have modded support imo

[Drofseh]

I think isRestraint might be better than isCabletie. It's more generic and fits better for rope or handcuff items

As is not exposed to end users I think isCabletie is better as is in keeping with ace item name

Right, I'm saying it should be different than the item name because if I add a handcuff item in the future it will be odd to call it a cable tie.

[SpicyBagpipes]

Right, I'm saying it should be different than the item name because if I add a handcuff item in the future it will be odd to call it a cable tie.

Done

Handled

Handled