Accessory - Removes action from pointers while not in slot

[10Dozen]

When merged this pull request will:

• fnc_switchAttachment operates only mounted items, but inventory action was available everywhere (and didn't work). Pull request changes action availabness to POINTER and OPTICS slots.

NOTE:
Currently if you place pointer with laser/flashlight MRT options (or optics) to e.g. untiform container - it will still have enabled 'Next rail item state' action in the item context menu. But when selected - item is not changed (because called function only works when item is attached to a gun).

[commy2]

These are supposed to work on all slots, not just pointers?

[10Dozen]

These are supposed to work on all slots, not just pointers?

Doesn't make sense to me to be able to enable flashlight while LLM-01 is in the backpack.
Maybe it will be useful for ACE map flashlight system, but not sure now.

EDIT:
ACE checks for all items
https://github.com/acemod/ACE3/blob/fa6da1ae7cb09c53c5ea6383322accd3c5f90e45/addons/map/functions/fnc_getUnitFlashlights.sqf#L20
(according to https://cbateam.github.io/CBA_A3/docs/files/common/fnc_uniqueUnitItems-sqf.html other params are all True by default)

So maybe switchAttachements function need rewrite to allow switching not only attached items, but in containers too.

[commy2]

Now it makes more sense to me, but I don't understand the need for this change. What is actually broken here?

[10Dozen]

Currently if you place pointer with laser/flashlight MRT options (or optics) to e.g. untiform container - it will still have enabled 'Next rail item state' action in the item context menu. But when selected - item is not changed (because called function only works when item is attached to a gun).

[commy2]

I see. Yeah, then this pr makes sense to me now.