Bug 328230 - Cover fetching only works from last.fm since 2.8
Summary: Cover fetching only works from last.fm since 2.8
Status: CONFIRMED
Alias: None
Product: amarok
Classification: Applications
Component: Tools/Cover Manager (show other bugs)
Version: 2.8-git
Platform: Kubuntu Linux
: NOR normal
Target Milestone: 2.9
Assignee: Amarok Developers
URL:
Keywords:
: 329985 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-11-29 22:04 UTC by Myriam Schweingruber
Modified: 2017-12-02 13:51 UTC (History)
17 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Myriam Schweingruber 2013-11-29 22:04:30 UTC
Since the 2.8 release, cover fetching only works from last.fm

This is really unfortunate as last.fm is a bad resource for classical and vintage albums. Somehow it seems the cover search is disabled for all other resources (Google, Yahoo and Discogs)
Comment 1 Giuseppe Travaglini 2013-12-01 06:00:40 UTC
I found the same in openSUSE 13.1 with the version 2.8.0-4.2.1 (x86_64) from the openSUSE-Repository and with the version 2.8.0-19.5 (x86_64) from the Packman-Repository
Comment 2 e.cisotti 2013-12-02 11:18:17 UTC
Same problem on Kubuntu 13.10
Comment 3 Andreas Roth 2014-01-05 15:37:24 UTC
I've the same issue on Kubuntu 13.10. I ran amarok with --debug and got the following output (with Google selected as cover source):
amarok: BEGIN: virtual void Playlist::PrettyListView::contextMenuEvent(QContextMenuEvent*) 
amarok:   BEGIN: void Playlist::ViewCommon::trackMenu(QWidget*, const QModelIndex*, const QPoint&) 
amarok:     [CoverFetcher] "Adding interactive cover fetch for: 'Supernatural' from Google" 
amarok:     BEGIN: CoverFoundDialog::CoverFoundDialog(CoverFetchUnit::Ptr, const Metadata&, QWidget*) 
QPixmap::scaled: Pixmap is a null pixmap
QPixmap::scaled: Pixmap is a null pixmap
amarok:     END__: CoverFoundDialog::CoverFoundDialog(CoverFetchUnit::Ptr, const Metadata&, QWidget*) [Took: 0.009s] 
amarok:     [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 0)" 
amarok:   END__: void Playlist::ViewCommon::trackMenu(QWidget*, const QModelIndex*, const QPoint&) [Took: 1.5s] 
amarok: END__: virtual void Playlist::PrettyListView::contextMenuEvent(QContextMenuEvent*) [Took: 1.5s] 
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 0)" 

If i press continue in the dialog the following lines appear:
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 1)" 
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 2)" 
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 3)" 
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 4)" 
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 5)" 
amarok: [CoverFetcher] "Queueing cover fetch query: 'Supernatural Carlos Santana' (page 6)" 
It seems that somehow the query never completes and so no covers.
Comment 4 Daniel Hurtado 2014-01-11 11:55:12 UTC
*** This bug has been confirmed by popular vote. ***
Comment 5 Daniel Hurtado 2014-01-11 12:04:48 UTC
I have the same problem as reported. I tried Amarok under (K)Ubuntu 13.10 - cover fetching only worked from last.fm under KDE as well as under Unity. I would love cover fetching from Google to work again as the covers from last.fm are low resolution and often missing.

If I can help by providing any information please let me know.
Comment 6 Myriam Schweingruber 2014-01-15 08:38:23 UTC
*** Bug 329985 has been marked as a duplicate of this bug. ***
Comment 7 madgenius 2014-02-04 01:37:44 UTC
(In reply to comment #1)
> I found the same in openSUSE 13.1 with the version 2.8.0-4.2.1 (x86_64) from
> the openSUSE-Repository and with the version 2.8.0-19.5 (x86_64) from the
> Packman-Repository

Can confirm. Having the same issue.
Comment 8 Daniel Hurtado 2014-02-05 11:53:36 UTC
(In reply to comment #5)
> I have the same problem as reported. I tried Amarok under (K)Ubuntu 13.10 -
> cover fetching only worked from last.fm under KDE as well as under Unity. I
> would love cover fetching from Google to work again as the covers from
> last.fm are low resolution and often missing.
> 
> If I can help by providing any information please let me know.

I need to add that no other resources works apart from last.fm. So it's not only Google as I stated above, Yahoo and Discogs are concerned too. I use KDE 4.12 and Amarok 2.8 from the Kubuntu Backports-PPA (ppa:kubuntu-ppa/backports) on Kubuntu 13.10 (saucy).
Comment 9 robert marshall 2014-02-06 10:27:18 UTC
I *think* it - google retrieval - was working with 2.8 before I upgraded kubuntu to 13.10 (I did the 2.8 update via ppa)
Comment 10 Myriam Schweingruber 2014-02-06 13:57:42 UTC
*** Bug 329985 has been marked as a duplicate of this bug. ***
Comment 11 Zach 2014-03-21 02:41:51 UTC
Can confirm on Kubuntu 14.04, Amarok 2.8.0.  All other cover retrieval services (Google etc.) used to work.
Comment 12 Myriam Schweingruber 2014-03-21 08:58:40 UTC
Folks, really, there is no need to confirm over and over again: the source pages have changed, there is no way this can still work, and the code needs to be adapted. This happens quite frequently as the image providers don't want anyone to grab pictures.

So please, let's keep the noise down on a report that is confirmed, and please everybody stop confirming this :)
Comment 13 Dan Meltzer 2014-05-11 02:32:09 UTC
Yahoo Boss API is now pay to use.  It has been removed from current git.

The Discogs api we use (version 1) was disabled in January of 2012.  There is a new api detailed on their site that will need to be swapped to.  Shouldn't be too hard to do, and would be a good opportunity for someone new to get involved in Amarok development.  Check in on #amarok if you are interested/for more details

Not sure what's going on with the google one.  We just scrape the html version, and I suspect the google side has changed signifigantly since it was implemented a few years ago.  patches welcome!
Comment 14 Mark Fraser 2014-05-11 12:33:49 UTC
If it helps, the Google fetcher has been modified in Audex and that now works correctly.
Comment 15 Myriam Schweingruber 2015-02-07 10:08:17 UTC
Technically not a regression, as the change was in the external sources.