fix issues when loading randomcycle from disabled folder

[sapphonie]

currently, when loading randomcycle, it runs Timer_RandomizeNextmap twice, leading to odd behavior where the map gets changed twice after timelimit is reached. This fixes that issue by only allowing the timer to be run once per map.

[peace-maker]

So OnConfigsExecuted is executed multiple times when a plugin is loaded late?

[KyleSanderson]

So OnConfigsExecuted is executed multiple times when a plugin is loaded late?

If the map changes twice from our data model there can be multiples of these timers running because this isn't killed with fire when the level ends. To fix this one I think we just need TIMER_FLAG_NOMAPCHANGE and nuke the global as it isn't needed and welcomes glitches.

There was an ancient issue with OnConfigsExecuted firing multiple times, and that could indeed be the issue here (if you want to step through with a debugger that would be cool). However, I'd suspect there's multiple map plugins loaded at the moment, so the server is changing levels twice (which also happened to me, over 10 years ago).

I do believe the current PR resolves the issue, but I don't believe it's the correct fix.

[peace-maker]

Thank you for bringing this up! Please consider the following change if you have the time!

[KyleSanderson]

The flag is in the wrong arg.