Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Attempt fixes to reported erring scripts #1085

Merged
merged 2 commits into from
Dec 17, 2022

Conversation

Jacky720
Copy link
Contributor

@Jacky720 Jacky720 commented Oct 5, 2022

Description

Remove Chapter 2 errors from DeltaMILK, fix other errors on DeltaHATE caused by changing variable names. Make it handle games that aren't named "UNDERTALE" a bit more gracefully, too.

Caveats

DeltaHATE still doesn't work on Chapter 2.

Notes

N/A

@github-actions
Copy link

github-actions bot commented Oct 5, 2022

else if (GameName.Contains("survey_program"))
deltamode = true;
else
deltamode = ScriptQuestion("Is this Deltarune Chapter 1 or a mod thereof?") ? true : false;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ScriptQuestion already returns a bool, why do a ? true : false after it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Somehow I was unsure it was a bool, but was sure it could be used in a ternary. In retrospect, of course it's a bool. I'll fix that.

@@ -2,6 +2,7 @@

EnsureDataLoaded();

// Is actually incompatible. Something broke when I (Space Core) tried to combine the UT and SURVEY_PROGRAM code.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add todo to fix?

@Grossley Grossley merged commit bf0f80e into UnderminersTeam:master Dec 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants