search

rattboi / mopidy-subsonic

Mopidy backend extension for playing music from Subsonic Music Streamer
MIT License
17 stars 14 forks source link

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

Open maxammann opened 9 years ago

maxammann commented 9 years ago 
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 commented 9 years ago

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 commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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

https://github.com/rattboi/mopidy-subsonic/commit/3f76e75f26e4291d1db97ddbd98929d93d3a9801

https://github.com/rattboi/mopidy-subsonic/commit/e6a694946d381d433fd4ad00d89951a2af8426b5#diff-8f64437659afb135e03ba97e5cc56624R29

Not sure whether browsing should be supported or not

nac-zero commented 8 years ago

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 commented 8 years ago

Same issue with the exact same setup.

remche commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

@jodal tok meg friheiten: https://github.com/rattboi/mopidy-subsonic/pull/18

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

jolny commented 8 years ago

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

unhammer commented 8 years ago

@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 commented 8 years ago

@remche -- I'm working on getting browse rocking for ncmpcpp in my fork right now. as of https://github.com/evanscottgray/mopidy-subsonic/commit/2267bcf4666dc4ac86456c8402ae646e5b0c774b 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.