Skip to content

Commit 874524f

Browse files
committed
make variant shit actually work
was being sent as a blank string for some reason
1 parent 31e3ebb commit 874524f

5 files changed

Lines changed: 12 additions & 12 deletions

File tree

engine/source/imaginative/backend/music/Conductor.hx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -545,7 +545,7 @@ class Conductor extends FlxBasic implements IBeat {
545545
FlxG.sound.loadHelper(audio, 1, soundGroup);
546546
audio.persist = true;
547547

548-
data = getMetadata('content/songs/$song/audio${variant == null ? '' : '-$variant'}');
548+
data = getMetadata('content/songs/$song/audio${variant.isNullOrEmpty() ? '' : '-$variant'}');
549549
applyBPMChanges();
550550

551551
#if FLX_PITCH pitch = pitch; #end
@@ -589,7 +589,7 @@ class Conductor extends FlxBasic implements IBeat {
589589
public function addVocalTrack(song:String, suffix:String, ?variant:String, ?afterLoad:FlxSound->Void):FlxSound {
590590
var file:ModPath = Paths.vocal(song, suffix, variant);
591591
if (!file.isFile) {
592-
log('Failed to find ${suffix.isNullOrEmpty() ? 'base ' : ''}vocal track for song "$song"${variant == null ? '' : ', variant "$variant"'}${suffix.isNullOrEmpty() ? '' : ' with a suffix of "$suffix"'}.', WarningMessage);
592+
log('Failed to find ${suffix.isNullOrEmpty() ? 'base ' : ''}vocal track for song "$song"${variant.isNullOrEmpty() ? '' : ' of variant "$variant"'}${suffix.isNullOrEmpty() ? '' : ' with a suffix of "$suffix"'}.', WarningMessage);
593593
return null;
594594
}
595595
var vocals:FlxSound = FlxG.sound.list.add(new FlxSound());
@@ -649,7 +649,7 @@ class Conductor extends FlxBasic implements IBeat {
649649
if (tracks.empty())
650650
addVocalTrack(song, '', variant);
651651
} catch(error:haxe.Exception) {
652-
log('Chart parse for song "$song"${variant == null ? '' : ', variant "${FunkinUtil.getDifficultyDisplay(variant)}"'} failed.', ErrorMessage);
652+
log('Chart parse for song "$song"${variant.isNullOrEmpty() ? '' : ' of variant "${FunkinUtil.getDifficultyDisplay(variant)}"'} failed. (error:$error)', ErrorMessage);
653653

654654
var tracks:Array<FlxSound> = [];
655655
// loads main suffixes

engine/source/imaginative/backend/system/Assets.hx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -615,13 +615,13 @@ class Assets {
615615
* @param variant The variant key.
616616
* @return ChartData ~ The chart data.
617617
*/
618-
public static function chart(song:String, difficulty:String, variant:String = 'normal'):ChartData {
618+
public static function chart(song:String, difficulty:String, ?variant:String):ChartData {
619619
final list:ChartDataList = charts.get(Paths.file('content/songs/$song').format(), false);
620620
if (list._fields().empty()) {
621621
_log('[Assets] No list instance found. (song:$song)', DebugMessage);
622622
return null;
623623
}
624-
if (variant != 'normal') {
624+
if (!variant.isNullOrEmpty()) {
625625
final list:ChartDataList = list.variants.get(variant);
626626
if (list._fields().empty()) {
627627
_log('[Assets] No list instance found. (song:$song, variant:$variant)', DebugMessage);

engine/source/imaginative/backend/system/Paths.hx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,7 @@ class Paths {
532532
* @return ModPath ~ The path data.
533533
*/
534534
inline public static function chart(song:String, difficulty:String, ?variant:String):ModPath
535-
return json('content/songs/$song/charts/${variant == null ? '' : '$variant/'}$difficulty');
535+
return json('content/songs/$song/charts/${variant.isNullOrEmpty() ? '' : '$variant/'}$difficulty');
536536
/**
537537
* Gets the path of a character json from "../content/objects/characters/".
538538
* @param file The mod path.
@@ -585,16 +585,16 @@ class Paths {
585585
* @return ModPath ~ The path data.
586586
*/
587587
inline public static function inst(song:String, ?variant:String):ModPath
588-
return audio('content/songs/$song/audio/${variant == null ? '' : '$variant/'}Inst');
588+
return audio('content/songs/$song/audio/${variant.isNullOrEmpty() ? '' : '$variant/'}Inst');
589589
/**
590590
* Gets the path of a songs vocal track from "../content/songs/{song}/audio/".
591591
* @param song The song folder name.
592592
* @param suffix The suffix tag.
593593
* @param variant The variation key.
594594
* @return ModPath ~ The path data.
595595
*/
596-
inline public static function vocal(song:String, suffix:String, ?variant:String):ModPath
597-
return audio('content/songs/$song/audio/${variant == null ? '' : '$variant/'}Voices${suffix.isNullOrEmpty() ? '' : '-$suffix'}');
596+
inline public static function vocal(song:String, ?suffix:String, ?variant:String):ModPath
597+
return audio('content/songs/$song/audio/${variant.isNullOrEmpty() ? '' : '$variant/'}Voices${suffix.isNullOrEmpty() ? '' : '-$suffix'}');
598598
/**
599599
* Gets the path of a song from "../music/".
600600
* @param file The mod path.

engine/source/imaginative/states/PlayState.hx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -828,7 +828,7 @@ class PlayState extends BeatState {
828828
storyMode = true;
829829
ArrowField.enemyPlay = ArrowField.enableP2 = false;
830830
renderChart(songList[0], difficulty, variant);
831-
_log('[PlayState] Rendering level "${level.name}", rendering songs ${[for (song in levelData.songs) song.name].cleanDisplayList()} under difficulty "${FunkinUtil.getDifficultyDisplay(difficulty)}"${variant == null ? '.' : ' in variant "$variant".'}', SystemMessage);
831+
_log('[PlayState] Rendering level "${level.name}", rendering songs ${[for (song in levelData.songs) song.name].cleanDisplayList()} under difficulty "${FunkinUtil.getDifficultyDisplay(difficulty)}"${variant.isNullOrEmpty() ? '.' : ' in variant "$variant".'}', SystemMessage);
832832
}
833833

834834
/**
@@ -844,7 +844,7 @@ class PlayState extends BeatState {
844844
ArrowField.enemyPlay = playAsEnemy;
845845
ArrowField.enableP2 = p2AsEnemy;
846846
renderChart(setSong = song, curDifficulty = difficulty, curVariant = variant);
847-
_log('[PlayState] Rendering song "$song" under difficulty "${FunkinUtil.getDifficultyDisplay(difficulty)}"${variant == null ? '.' : ' in variant "$variant".'}', SystemMessage);
847+
_log('[PlayState] Rendering song "$song" under difficulty "${FunkinUtil.getDifficultyDisplay(difficulty)}"${variant.isNullOrEmpty() ? '.' : ' in variant "$variant".'}', SystemMessage);
848848
}
849849

850850
/**

engine/source/imaginative/states/menus/FreeplayMenu.hx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ class FreeplayMenu extends BeatState {
232232
infoTextGroup.add(infoTextBox);
233233

234234
songNameText = new FlxText(10, 10, boxWidth - 20, 'Song: crAzy');
235-
variantText = new FlxText(10, songNameText.y + songNameText.height + 17, boxWidth - 20, 'Variant: Normal');
235+
variantText = new FlxText(10, songNameText.y + songNameText.height + 17, boxWidth - 20, 'Variant: [NONE]');
236236
difficultyText = new FlxText(10, variantText.y + variantText.height + 20, boxWidth - 20, 'Normal');
237237
sideArrowsText = new FlxText(10, difficultyText.y, difficultyText.width, '< >');
238238
for (text in [songNameText, variantText, difficultyText, sideArrowsText]) {

0 commit comments

Comments
 (0)