Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Endsection handling #332

Merged
merged 3 commits into from
Sep 19, 2022
Merged

Endsection handling #332

merged 3 commits into from
Sep 19, 2022

Conversation

fireblizzard
Copy link
Contributor

Hazard Course ends the game via the trigger_endsection entity, and possibly some other mods or campaigns. This PR just handles this game ending method so that the timer is stopped and LiveSplit stops too, because previously we had to stop both manually when running Hazard Course and it was a bit annoying, specially when you're new to running the thing and don't know what's going on with the timer/splits.

Smiley mentioned that there's some case where sending the player to the menu via trigger_endsection is not meant to end the game, but only one level or something, and then the player goes back to chapter selection or whatever and continues from there. For this reason, it won't automatically end the game on bxt_timer_autostop 1 unless it's Hazard Course. The player will have to set bxt_timer_autostop 2 if they actually want the game to end, so they have this workaround and they don't need to wait for someone to hardcode their mod's trigger targetname for it to work out of the box.

@YaLTeR YaLTeR merged commit b2343de into YaLTeR:master Sep 19, 2022
@YaLTeR
Copy link
Owner

YaLTeR commented Sep 19, 2022

Thanks

SmileyAG added a commit to SmileyAG/BunnymodXT that referenced this pull request Sep 22, 2022
* Perform game-ending tasks on trigger_endsection

* Apply autostop for Hazard Course if targetname == 'tr_endchange'

* EndSection: stop timer w/o required conditions in bxt_timer_autostop 2

Co-authored-by: SmileyAG <redstyle45@gmail.com>
Co-authored-by: SmileyAG <58108407+SmileyAG@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants