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 327700 - [PATCH] DAAP song list is missing "Year"
[PATCH] DAAP song list is missing "Year"
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: DAAP
HEAD
Other All
: Normal enhancement
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-01-19 13:31 UTC by Alex Lancaster
Modified: 2006-01-21 05:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Adds support for extracting "Year" from DAAP stream (1.17 KB, patch)
2006-01-19 13:33 UTC, Alex Lancaster
committed Details | Review

Description Alex Lancaster 2006-01-19 13:31:58 UTC
Please describe the problem:
The DAAP song list is currently missing the year even if the ID3 year tag is
encoded in the file.  This is because the year information is not currently
being extracted into the rhythmbox data structure.

This patch implements the year extraction for files taken from a DAAP stream.

Steps to reproduce:
1. rhythmbox
2. connect to a DAAP source
3. look at year of track
4. check that DAAP source track has year encoded in it (e.g. check it in iTunes
or other rhythmbox source)

Actual results:
"Year" field is always "Unknown" even if the Year is present on the source machine.

Expected results:
The year should be filled in.

Does this happen every time?
Yes

Other information:
Comment 1 Alex Lancaster 2006-01-19 13:33:39 UTC
Created attachment 57638 [details] [review]
Adds support for extracting "Year" from DAAP stream
Comment 2 Alex Lancaster 2006-01-20 05:15:10 UTC
Tag as "enhancement" (although it could arguably be called a "bug" as well).
Comment 3 James "Doc" Livingston 2006-01-21 05:57:09 UTC
Committed to cvs, with a few code-style things fixed (indentation, space before opening parenthesises)