Butiksside
The Binding of Isaac: Rebirth
Alt Diskussioner Skærmbilleder Illustrationer Broadcasts Videoer Værksted Nyheder Guider Anmeldelser

The Binding of Isaac: Rebirth

Butiksside
The Binding of Isaac: Rebirth >  Værksted > AceHands værksted > What's That Song? > Diskussioner
What's That Song?
Beskrivelse Diskussioner1 Kommentarer49 Ændringsbemærkninger
☢ Nato Potato ☢ 15. aug. 2024 kl. 22:35
Add support for mods that use jukebox (also add support for variant music tracks)
after finding this mod I did a bit of experimenting, with the idea that it shouldn't matter whether someone is using this mod or Jukebox or whether the soundtrack mod uses the WhatsThatSongAPI:AddSoundtrack function or the AddTitlesToJukebox function.

I got it working on my end and will be updating the soundtrack menu and jukebox mods.

Basically this update would automatically add support for every soundtrack mod that uses jukebox, rather than the mods needing to be updated to support Whats that song? or you having to manually add more track lists.

The updates I made to this mod are as follows, no pressure to implement these updates and obviously feel free to implement things in a different way than I did
(you can show code on steam using the [code] ... [/code] tags)



In the PopulateMusicIDs function I added a check for the new soundtrack menu feature that allows multiple different versions of a track type
trackID = Isaac.GetMusicIdByName(st .. " " .. musicTable.TrackTypes[ii]) local variants = (Isaac.GetMusicIdByName(st .. " " .. musicTable.TrackTypes[ii] .. " 2") > 0) --check if soundtrack has multiple different tracks for the same track type if variants and trackID < 0 then trackID = Isaac.GetMusicIdByName(st .. " " .. musicTable.TrackTypes[ii] .. " 1") end -- Resolve referred soundtrack IDs if type(v) == "string" and v:sub(1, 1) == "[" and v:sub(-1, -1) == "]" then local resolvedSoundtrack = nil for idx, soundtrack in pairs(musicTable.Soundtracks) do if v:sub(2, -2) == soundtrack then resolvedSoundtrack = soundtrack break end end if resolvedSoundtrack ~= nil and musicTable.SoundtrackTitles[resolvedSoundtrack][ii] ~= nil then musicIDs[trackID] = musicTable.SoundtrackTitles[resolvedSoundtrack][ii] or fallbackID else musicIDs[trackID] = musicTable.SoundtrackTitles.Rebirth[ii] end else if variants then --load the different variants for this track type for iii=1, #musicTable.SoundtrackTitles[st][ii] do trackID = Isaac.GetMusicIdByName(st .. " " .. musicTable.TrackTypes[ii] .. " " .. iii) musicIDs[trackID] = musicTable.SoundtrackTitles[st][ii][iii] end else musicIDs[trackID] = musicTable.SoundtrackTitles[st][ii] or fallbackID end end

At the post_render function I added a check that lets Whats that song? know if the soundtrack menu is playing a song that doesn't get returned by GetCurrentMusicID
WhatsThatSong:AddCallback(ModCallbacks.MC_POST_RENDER, function() musicID = musicManager:GetCurrentMusicID() --if soundtrack menu is using layer/jingle to play something that won't show up on GetCurrentMusicID if JukeBoxTrackOverride and JukeBoxTrackOverride > 0 then musicID = JukeBoxTrackOverride end

and finally if a soundtrack mod is using jukebox to add its title list, this mod can grab the list
--get track titles from mod with jukebox function if Titles == nil then Titles = true end function AddTitlesToJukebox(tableid, name, displayname, soundtrack) WhatsThatSongAPI:AddSoundtrack(name, soundtrack) end
Sidst redigeret af ☢ Nato Potato ☢; 15. aug. 2024 kl. 22:41
< >
Viser 1-4 af 4 kommentarer
☢ Nato Potato ☢ 16. aug. 2024 kl. 4:29 
also wouldn't hurt to copy the built in track titles from the jukebox titles.lua list into your 'tablehell' file
#1
AceHand  [udvikler] 17. aug. 2024 kl. 9:20 
Hey, and WOW! This is exceedingly amazing, to put simply. I'll be sure to implement these features posthaste. Thanks!

P.S. Big fan of your work. Thank you for taking an interest, and keep doing what you do :)
#2
☢ Nato Potato ☢ 18. aug. 2024 kl. 2:18 
Oh, just realised this doesn't check if a song imported from jukebox list is a jingle or not, could result in jingles showing up as a currently playing track

should be easy fix though since track order is always the same, in the populate function check if ii is one of the jingle indexes, if it is just import "" and skip to the next one instead of reading the list entry
Sidst redigeret af ☢ Nato Potato ☢; 18. aug. 2024 kl. 2:21
#3
AceHand  [udvikler] 18. aug. 2024 kl. 12:38 
Good catch, that had completely slipped my mind as well. It'll be fixed soon!
Sidst redigeret af AceHand; 18. aug. 2024 kl. 13:31
#4
< >
Viser 1-4 af 4 kommentarer
Per side: 1530 50