Yeah. The way you have it set up now, it sets a timer that ticks off and does nothing, then it creates another timer that calls an attack function over and over in a loop. I think you also want to call the attack functions directly rather than through the looping timer-by-function nodes. You just want the first timer to be ticking off.
your timer isnt actually doing anything the way you have it setup there. you need to connect the custom event to the switch. your probably going to have issue with the way that you are calling your other events too.