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 337876 - TagList does not retreive tags by key if 2 or more tags with the same name are present
TagList does not retreive tags by key if 2 or more tags with the same name ar...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-python
git master
Other All
: Normal major
: 0.10.4
Assigned To: GStreamer Maintainers
Johan (not receiving bugmail) Dahlin
Depends on:
Blocks:
 
 
Reported: 2006-04-10 03:54 UTC by Артём Попов
Modified: 2006-04-10 14:48 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
Patch that fixes the problem (367 bytes, patch)
2006-04-10 03:56 UTC, Артём Попов
none Details | Review

Description Артём Попов 2006-04-10 03:54:32 UTC
Please describe the problem:
TagList.keys() returns only tags that have exactly ONE value.
Tags with a same name that are specified 2 or more times are not listed in keys
at all.

Steps to reproduce:
 Typically this problem could happen with vorbis-style comments, i. e.:
$ ogginfo starbase.ogg
...
        ARTIST=Tore Aune Fjellstad
        ARTIST=Espen Gätzschmann
...
        GENRE=Soundtrack
        GENRE=Media
...

TagList.keys() would not list Genre or Artist tags at all.

Actual results:


Expected results:
TagList.keys should contain proper keys for multi-value tags and return a list
or tuple of expected values.

Does this happen every time?


Other information:
Comment 1 Артём Попов 2006-04-10 03:56:15 UTC
Created attachment 63074 [details] [review]
Patch that fixes the problem

Fortunately this problem is very easily fixed (see the patch) :)
Comment 2 Edward Hervey 2006-04-10 14:48:55 UTC
2006-04-10  Edward Hervey  <edward@fluendo.com>

	* gst/gsttaglist.override: (tag_foreach_func_list): 
	Return tag value if present more than once (instead of exactly once).
	Fixes #337876