Pharmacy - Coagulation bug

[kolmipilot]

Coagulation does not work for players if the Allow clotting for AI setting is disabled.

How to reproduce:

1. In the settings, disable Allow clotting for AI and disable Coagulation.
2. Shoot a player — no wound starts clotting.

Code-based problem:
The coagulation event handler is removed even if the unit is a player.

When I added this trace to the code:
if (!(GVAR(coagulation_allowOnAI)) && ACE_Player != _unit) exitWith { TRACE_1("ClotWound: Stopping per frame handler for unit %1", _unit); [_idPFH] call CBA_fnc_removePerFrameHandler; };

This trace appeared in the RPT file:
1:18:01 [KAT] (pharma) TRACE: 7169 ClotWound: Stopping per frame handler for unit %1: _unit=B Alpha 1-1:1 (kolmipilot) x\kat\addons\pharma\functions\fnc_clotWound.sqf:111

Full RPT: https://pastebin.com/j6uWr017
Base file: https://github.com/KAT-Advanced-Medical/KAM/blob/dev-Tomcat/addons/pharma/functions/fnc_clotWound.sqf

With ChatGPT’s help, changing the statement to this fixed the issue:
(!(GVAR(coagulation_allowOnAI)) && {!isPlayer _unit})

[Cplhardcore]

1. Chatgpt is bad, dont use it, plus its supposed fix, doesnt work for our usecase
2. line 109 is correct, and does what it needs to, which is exits the PFH if the unit is not the current ACE_Player.
3. the issue with coagulation not working is different, and its an issue with RCed units and ACE_player changing, which breaks the PFH

[Cplhardcore]

Specifically the issue is this in ace common
["unit", { ACE_player = (_this select 0); }, true] call CBA_fnc_addPlayerEventHandler;
Which switches ace player to the RCed unit, making the player body break the PFH
There is no easy fix for this without a lot of work, so IMO just have it as a known bug