After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 625823 - new Last.fm radio API playlists not parsed correctly
new Last.fm radio API playlists not parsed correctly
Status: RESOLVED FIXED
Product: totem-pl-parser
Classification: Core
Component: General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: totem-pl-parser-maint
totem-pl-parser-maint
Depends on:
Blocks:
 
 
Reported: 2010-08-02 12:43 UTC by Jamie Nicol
Modified: 2010-08-02 13:24 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
excerpt from new API response (1.11 KB, text/plain)
2010-08-02 12:43 UTC, Jamie Nicol 		Details
excerpt from old API response, for comparison (1.08 KB, text/plain)
2010-08-02 12:44 UTC, Jamie Nicol 		Details
full response from new api (6.44 KB, text/plain)
2010-08-02 13:03 UTC, Jamie Nicol 		Details
full response from old api (5.91 KB, text/plain)
2010-08-02 13:04 UTC, Jamie Nicol 		Details

Description Jamie Nicol 2010-08-02 12:43:14 UTC 
Created attachment 166968 [details]
excerpt from new API response

The new radio API stores Last.fm specific info in the xspf playlist differently to the old API.

Rhythmbox (Jamie Nicol's 2010 GSoC branch) makes use of the TOTEM_PL_PARSER_FIELD_ID and TOTEM_PL_PARSER_FIELD_DOWNLOAD_URI fields. Both are parsed correctly from the old API response but not from the new API response.

If possible, it would be great if totem-pl-parser could parse correctly from both old and new style responses, as Rhythmbox uses both APIs.
Comment 1 Jamie Nicol 2010-08-02 12:44:32 UTC 
Created attachment 166969 [details]
excerpt from old API response, for comparison
Comment 2 Bastien Nocera 2010-08-02 12:51:55 UTC 
Could you please attach a full XSPF file for both types of playlists (as in, one that's actually parseable instead of snippets)?
Comment 3 Jamie Nicol 2010-08-02 13:03:53 UTC 
Created attachment 166972 [details]
full response from new api
Comment 4 Jamie Nicol 2010-08-02 13:04:37 UTC 
Created attachment 166973 [details]
full response from old api
Comment 5 Bastien Nocera 2010-08-02 13:24:20 UTC 
commit c1e5c2f15f4b0c523743cfeae0c7e636d9d468d1
Author: Bastien Nocera <hadess@hadess.net>
Date:   Mon Aug 2 14:22:56 2010 +0100

    Add support for Last.fm's new XSPF extensions
    
    Support the TOTEM_PL_PARSER_FIELD_ID and
    TOTEM_PL_PARSER_FIELD_DOWNLOAD_URI fields in newer last.fm XSPF
    playlists.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=625823