Hi,
I'm using Pathfinder 1E.
There appears to be a bug when a critical threat occurs during an attack with an ATK effect in place in the combat tracker. When the critconfirm is rolled, the effect is added in (as it should be) however when the final crit result is reported, it adds the effect in again, effectively doubling whatever the effect was. Here is a screenshot showing the issue:
Attachment 50670
As can be seen there, I added an ATK:1, which is reflected in the initial attack rolls ("EFFECTS+1") which for the die roll of 20, adds a total of +4 (+1 for the effect and +3 for the char's str bonus). Then, the crit confirm roll is made, and a 1 was rolled, and +4 was added (which includes the ATK:1 effect) for a result of 5. Then, the final chat line showing the crit result adds in "EFFECTS +1", changing the result to 6, which is incorrect.
I suspect the issue is related to the code in manager_action_attack.lua", around line 482 where it takes into account any effects for the crit confirm result. Should it really be doing this since the crit confirm roll already did?
If I unload the extension, things behave correctly, as can be seen in this screenshot. The critconfirm roll and the final critconfirm result are the same, as they should be:
Attachment 50672
EDIT: looking at the two screenshots, there is also a discrepancy in the reporting of the crit confirm roll when the extension is loaded - it adds the effect in (as can be seen by the +4 added to the roll), but doesn't include the text "EFFECTS +1", which might be clue as to why the extension then adds the effect in again in the final crit result reporting line. When the extension is not loaded (as seen in the second screenshot) the "EFFECTS +1" message is added to the critconfirm roll and is not present in the final crit result line.
Cheers,
Brian