Currently skills triggered by spellslinger only considered the cooldown imposed on them by spell slinger in addition to their own but not the cooldown of the main skill gem. This works fine as as long as the cooldown of the main effect is equal to the cooldown of the support effect as they both will go off at the same time. Problem is there currently exist conditional cooldown recovery mods which could affect only one of the cooldowns making this assumption no longer hold. This pr basically extends the logic from https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/7558 to solve the issue effectively taking a max of the cooldown of the main gem and the effective cooldown of the triggered skill while taking the correct cooldown recovery mods into consideration.
Description of the problem being solved:
Currently skills triggered by spellslinger only considered the cooldown imposed on them by spell slinger in addition to their own but not the cooldown of the main skill gem. This works fine as as long as the cooldown of the main effect is equal to the cooldown of the support effect as they both will go off at the same time. Problem is there currently exist conditional cooldown recovery mods which could affect only one of the cooldowns making this assumption no longer hold. This pr basically extends the logic from https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/7558 to solve the issue effectively taking a max of the cooldown of the main gem and the effective cooldown of the triggered skill while taking the correct cooldown recovery mods into consideration.
Steps taken to verify a working solution:
Link to a build that showcases this PR: