Feature/delegating lookup

[dscalzi]

The DelegatedAudioLookup function allows the track lookup behavior to be injectable by the client application. This may be desired when isrc/name match lookup is not good enough, and additional logic needs to be applied.

Sample usage:

playerManager.registerSourceManager(new SpotifySourceManager(
        g.getSpotifyClientId(),
        g.getSpotifyClientSecret(),
        g.getSpotifyCountryCode(),
        playerManager,
        mirroringAudioTrack -> {
            final String ytsearch = "ytsearch:";

            // Try by isrc
            AudioItem item = AudioReference.NO_TRACK;
            if(mirroringAudioTrack.getISRC() != null) {
                item = mirroringAudioTrack.loadItem(
                        ytsearch + mirroringAudioTrack.getISRC());
                String title = null;
                if(item instanceof InternalAudioTrack casted) {
                    title = casted.getInfo().title;
                }
                else if(item instanceof AudioPlaylist casted) {
                    title = casted.getTracks().get(0).getInfo().title;
                }
                if(title != null) {
                    if(!title.toLowerCase().contains(mirroringAudioTrack.getInfo().title.toLowerCase())) {
                        log.info("Title Mismatch, Expected: {}. Actual: {}.", mirroringAudioTrack.getInfo().title, title);
                        item = AudioReference.NO_TRACK;
                    }
                }
            }
            // Try name + isrc
            if(item == AudioReference.NO_TRACK) {
                item = mirroringAudioTrack.loadItem(
                        ytsearch + mirroringAudioTrack.getInfo().title + " " + mirroringAudioTrack.getISRC());
            }

            if(item == AudioReference.NO_TRACK) {
                log.error("Failed to find track");
            }

            return item;
        }
));

[topi314]

I moved & renamed some minor stuff. Like this it should be good to go. Let me know your thoughts

[dscalzi]

I moved & renamed some minor stuff. Like this it should be good to go. Let me know your thoughts

I'm alright with the name change, but it's hard to tell what changed in the diff. Did you intend to change all the spaces to tabs?

[topi314]

yeah that was more or less intended. You can hide whitespace changes here

[dscalzi]

Works for me, go ahead and merge.

As a side note, 4 spaces is much more standard than using tabs, but feel free to use whatever you prefer.

[topi314]

ty for the contribution