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

Scanner: Image extraction is not working with GST1.0 #1469

Closed
dublok opened this issue Feb 24, 2016 · 2 comments
Closed

Scanner: Image extraction is not working with GST1.0 #1469

dublok opened this issue Feb 24, 2016 · 2 comments

Comments

@dublok
Copy link
Contributor

@dublok dublok commented Feb 24, 2016

mopidy-local-images does not extract the cover-art from music files anymore (since GST1.0).

This issue has two sides:

mopid
`convert_taglist'() ignores image tags.
https://github.com/mopidy/mopidy/blob/develop/mopidy/audio/tags.py#L52
I miss something like

            elif isinstance(value, Gst.Sample):
                # do the image things here
                result[tag].append(bytes(value.get_buffer())) # ???

mopidy-local-images
Does not support GST1.0.
https://github.com/mopidy/mopidy-local-images/blob/develop/mopidy_local_images/library.py#L171
GST1.0 Gst.Buffer has no data attribute

@tkem
Copy link
Member

@tkem tkem commented Feb 26, 2016

AFAICS, if image tags are added as bytes objects as proposed, mopidy-local-images would fall back on using the object "as is" when there's no data attribute.

Loading

@jodal jodal added this to the v2.0.1 - Bug fixes milestone Mar 1, 2016
@jodal jodal changed the title Image extraction is not working with GST1.0 Scanner: Image extraction is not working with GST1.0 Mar 1, 2016
jodal added a commit that referenced this issue Mar 26, 2016
@jodal
Copy link
Member

@jodal jodal commented Mar 26, 2016

Fixed by PR #1472.

Loading

@jodal jodal closed this Mar 26, 2016
tmyersjstar added a commit to tmyersjstar/mopidy that referenced this issue Apr 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants