Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WARNING SubsonicBackend does not implement playlists.as_list(). Please upgrade it. #13

Open
maxammann opened this issue May 6, 2015 · 13 comments

Comments

@maxammann
Copy link

WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.

appears while browsing the library. Also music doesn't show up.

Using Subsonic 5.2.1-KANG

@jake-g
Copy link

jake-g commented May 9, 2015

same warning reoccurs while browsing. Also I dont get any indication that subsonic is running during startup of the server. All my other plugins say something like "Starting ..."

Pretty sure im using version 4.7 with music cabinet

@evenreven
Copy link

Same issue for me. It looks like it's starting, but nothing from Subsonic shows up in any of the frontends (I've tried Moped, Musicbox and ncmpcpp, which all show Spotify correctly).

Here's the terminal output when I start Mopidy manually:

INFO     Starting Mopidy 1.0.8
INFO     Loading config from builtin defaults
INFO     Loading config from /home/MYUSER/.config/mopidy/mopidy.conf
INFO     Loading config from command line options
INFO     Enabled extensions: spotify, mpd, http, stream, m3u, subsonic, mpris, softwaremixer, moped, musicbox_webclient
INFO     Disabled extensions: local
INFO     Starting Mopidy mixer: SoftwareMixer
INFO     Starting Mopidy audio
INFO     Starting Mopidy backends: SpotifyBackend, StreamBackend, M3UBackend, SubsonicBackend
INFO     Mopidy uses SPOTIFY(R) CORE
INFO     Loaded 0 M3U playlists from /home/MYUSER/.local/share/mopidy/m3u
INFO     Connecting to Subsonic library http://192.168.1.11:8091 as user mopidy
INFO     Audio output set to "autoaudiosink"
INFO     Connected to Spotify
INFO     Starting Mopidy core
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
INFO     Starting Mopidy frontends: MprisFrontend, MpdFrontend, HttpFrontend
INFO     MPRIS server connected to D-Bus session bus
INFO     MPD server running at [::ffff:127.0.0.1]:6600
INFO     HTTP server running at [::ffff:127.0.0.1]:6680
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
INFO     Loaded 141 Spotify playlists
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.
WARNING  SubsonicBackend does not implement playlists.as_list(). Please upgrade it.

"mopidy" is a valid Subsonic user, and it seems that mopidy-subsonic correctly creates a py-sonic player, so the URL and port is obviously correct.

I use Arch Linux with a Mopidy install from the official repos. Mopidy-subsonic is from AUR and connects to Subsonic 5.2.1 on an Ubuntu 14.04 server. Subsonic is working perfectly through the web interface and through the Android app.

I can confirm that trying to browse a frontend will add yet another warning to the terminal output.

(I have no idea if the warning is related to the issue of the frontends not finding the Subsonic music or not. For all I know this could be two separate issues.)

EDIT: If I try to search, I get this output:

WARNING  SubsonicBackend does not implement library.search() with "exact" support. Please upgrade it.

@evenreven
Copy link

Actually, while the issue itself (the error message) is still present, removing the outdated AUR package and replacing it with the most recent Github commit solved the issue of the frontend not finding anything in a search.

However, I can still can't browse my music through a frontend - only search. I noticed this issue just now, indicating that this is strictly a search-based extension. Is this correct?

@maxammann
Copy link
Author

Here are the commits, which most likely fixed the two issues:

3f76e75

e6a6949#diff-8f64437659afb135e03ba97e5cc56624R29

Not sure whether browsing should be supported or not

@nac-zero
Copy link

I got same warning message;

WARNING SubsonicBackend does not implement playlists.as_list(). Please upgrade it.

with Mopidy1.1.1, mopidy-subsonic 1.0.0, subsonic 5.2.1.

@andor44
Copy link

andor44 commented Oct 15, 2015

Same issue with the exact same setup.

@remche
Copy link

remche commented Feb 8, 2016

Same issue here, all installed from pip.

Mopidy - Music server with MPD and Spotify support
INSTALLED: 1.1.2 (latest)
Mopidy-Subsonic - Subsonic extension for Mopidy
INSTALLED: 1.0.0 (latest)

In this mopidy-subsonic version, the below patches are already applied.

I traced the error if it helps..
traceback.txt

@jodal
Copy link
Contributor

jodal commented Feb 8, 2016

From https://github.com/rattboi/mopidy-subsonic/blob/master/mopidy_subsonic/client.py#L252-L255:

        if 'year' in data:
            track_kwargs['date'] = data['year']
        else:
            track_kwargs['date'] = 'none'

Should be changed to:

        if 'year' in data:
            track_kwargs['date'] = str(data['year'])
        else:
            track_kwargs['date'] = None

@remche
Copy link

remche commented Feb 8, 2016

Thanks jodal, it did it !
Any chance to see playlists.as_list() implemented ? Still can't browse media library using ncmpcpp :(
Thx guys for your work.

@unhammer
Copy link

@jodal tok meg friheiten: #18

With that change onto the git version of this, I can actually search and play media through ncmpcpp =D

@jolny
Copy link

jolny commented Mar 9, 2016

Hi!
A question, is it just my frontend or is search very slow?

@unhammer
Copy link

@jolny if it's not related to this bug, you should probably report a new one at https://github.com/rattboi/mopidy-subsonic/issues/new

@evanscottgray
Copy link

@remche -- I'm working on getting browse rocking for ncmpcpp in my fork right now. as of evanscottgray@2267bcf it is working but it needs a lot of cleanup before it is ready to PR.

after that the playlist stuff is next for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants