As Intaris said, putting the Value signalling inside the event case was a bad idea and was just going to lead to infinite loops. The correct Value signalling you had in the other loop, yet you disabled it. Why?
Get rid of the Value signalling you have, and enable the one that you have disabled. Then your code would have worked.
As for your latest VI, you think it works, but you'll eventually run into problems. Events are used to trigger event structures that are in other loops. You have it in the same loop. That is pointless. You are just remotely triggering a structure that is right next to where you are triggering it. That event structure will run one for one with each time you If you ever added another event, then you generate the user event. Which sounds good at first. But if you ever add another event case and those start loading in the event queue. You can only process that event when you go and trigger another user event. You are going to flood your event queue with user events and they will just fall further and further behind.
