Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

replay fix + stage, gf and noteStyle defaults to their corresponding style if none was found in chart #824

Closed
wants to merge 12 commits into from

Conversation

Lucky-56
Copy link

No description provided.

@Lucky-56
Copy link
Author

Adds #820

@Lucky-56 Lucky-56 changed the title defaults stage to their corresponding stage if none was found in chart replay fix + stage defaults to their corresponding stage if none was found in chart Jun 11, 2021
@Lucky-56
Copy link
Author

Fixes #825

@Lucky-56
Copy link
Author

would be funny if you sqeezed this into 1.5.2 aswell (as the current pre-compiled version is broken)

as in for modders to copy paste easily
case 5: if (songLowercase == 'winter-horrorland') {stageCheck = 'mallEvil';} else {stageCheck = 'mall';}
case 6: if (songLowercase == 'thorns') {stageCheck = 'schoolEvil';} else {stageCheck = 'school';}
//i should check if its stage (but this is when none is found in chart anyway)
}

Choose a reason for hiding this comment

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

Should also be applying gfVersion and noteStyle because of weeks 4-6.

Copy link
Author

Choose a reason for hiding this comment

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

oh yeah, I'll try

@puyoxyz
Copy link
Collaborator

puyoxyz commented Jun 11, 2021

i swear to god im about to make a github action that comments on every pr saying to actually test it when approving changes and not just glance over them

@Lucky-56 Lucky-56 changed the title replay fix + stage defaults to their corresponding stage if none was found in chart replay fix + stage, gf and noteStyle defaults to their corresponding stage if none was found in chart Jun 11, 2021
@Lucky-56 Lucky-56 changed the title replay fix + stage, gf and noteStyle defaults to their corresponding stage if none was found in chart replay fix + stage, gf and noteStyle defaults to their corresponding style if none was found in chart Jun 11, 2021
Copy link

@ActualMandM ActualMandM left a comment

Choose a reason for hiding this comment

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

Issues:

  • Mods like B-Sides have their variables go back to the game's default when in replay mode.
  • Notes don't fully get changed. (Note.hx wasn't changed at all)
  • Compilation error. (look below) resolved
screenshots of issues

2021-06-11_08-03-54_Kade_Engine
2021-06-11_08-07-57_Kade_Engine
2021-06-11_08-06-37_Kade_Engine

@Lucky-56
Copy link
Author

Lucky-56 commented Jun 11, 2021

* Mods like B-Sides have their variables go back to the game's default when in replay mode.

what do you mean by that
while wait.. I didn't check it first (I'm such a 'well I did. I'm done.' type)

@ActualMandM
Copy link

In gameplay, B-Sides works as intended with the stage, gf, and notes. However in replay, it defaults back to stage, regular gf, and regular notes.
2021-06-11_08-07-57_Kade_Engine

@Lucky-56
Copy link
Author

Lucky-56 commented Jun 11, 2021

I currently have a problem understanding why, cause the replay just loads the JSON associated with the song name and that should load the stage'n'stuff right?
maybe it doesn't do the stage check

@Lucky-56
Copy link
Author

Lucky-56 commented Jun 11, 2021

I found the culprit when starting a replay it doesn't get the week correctly
it says week 0
image
so I need to somehow fix that

EDIT: haha found it it is as simple as removing .toLowerCase() as replays are no longer in lowercase in 1.1

@Lucky-56 Lucky-56 marked this pull request as draft June 11, 2021 16:37
@Lucky-56
Copy link
Author

actually tested these this time

@Lucky-56 Lucky-56 marked this pull request as ready for review June 11, 2021 17:13
@Lucky-56
Copy link
Author

Fixes #830

@Lucky-56 Lucky-56 closed this Jun 12, 2021
@Lucky-56 Lucky-56 deleted the branch KadeArchive:master June 12, 2021 10:03
@Lucky-56 Lucky-56 deleted the master branch June 12, 2021 10:03
@Lucky-56 Lucky-56 restored the master branch June 12, 2021 10:07
@Lucky-56 Lucky-56 deleted the master branch June 12, 2021 10:21
Kade-github added a commit that referenced this pull request Jun 15, 2021
REOPENED: #824 replay fix + modchart fix + stage, gf and noteStyle defaults to their corresponding style if none was found in chart
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants