Shadnix-was-taken / BeatSaber-SongPlayHistoryContinued

Beat Saber local leaderboard plugin
MIT License
16 stars 4 forks source link

The Unmerged Changes from #6 #8

Closed qe201020335 closed 2 days ago

qe201020335 commented 1 year ago

This is a re-opened pr because I created a new branch to not spam this pr with all my future commits.

Changes:

Copied From #6 :

The max raw score is actually not accessible at all outside of the game play context and it only exists in the ScoreController. Well, I guess I have to track and pass the value myself then. SiraUtil and Zenject to the rescue!

For all the existing records, if the max scores can be calculated with the old method, i.e. there is no slider or burst in the map, I simply calculate the max score with the note count Record.LastNote.

The OldschoolBeatSaber mod will break this. Simply installing the mod will cause the SPH hover hint ui not to load. There won't be any exceptions, just like SPH is not installed but new records will still be saved. This seems to be because OldschoolBeatSaber provides a re-implementation of some old-school elements of HMUI and SPH's reflection may be finding the wrong types.