mirror of
https://github.com/clementine-player/Clementine
synced 2024-12-15 10:48:33 +01:00
Remove a couple more broken lyric providers and sort the list of lyrics properly. Fixes issue #881
This commit is contained in:
parent
d6cb1d3c71
commit
548430853c
@ -97,11 +97,8 @@ void SongInfoView::ReloadSettings() {
|
|||||||
<< ProviderByName("lyricsreg.com")
|
<< ProviderByName("lyricsreg.com")
|
||||||
<< ProviderByName("lyricsmania.com")
|
<< ProviderByName("lyricsmania.com")
|
||||||
<< ProviderByName("metrolyrics.com")
|
<< ProviderByName("metrolyrics.com")
|
||||||
<< ProviderByName("seeklyrics.com")
|
|
||||||
<< ProviderByName("azlyrics.com")
|
<< ProviderByName("azlyrics.com")
|
||||||
<< ProviderByName("mp3lyrics.org")
|
|
||||||
<< ProviderByName("songlyrics.com")
|
<< ProviderByName("songlyrics.com")
|
||||||
<< ProviderByName("lyricsmode.com")
|
|
||||||
<< ProviderByName("elyrics.net")
|
<< ProviderByName("elyrics.net")
|
||||||
<< ProviderByName("lyricsdownload.com")
|
<< ProviderByName("lyricsdownload.com")
|
||||||
<< ProviderByName("lyrics.com")
|
<< ProviderByName("lyrics.com")
|
||||||
@ -121,10 +118,10 @@ void SongInfoView::ReloadSettings() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Enable all the providers in the list and rank them
|
// Enable all the providers in the list and rank them
|
||||||
int relevance = 0;
|
int relevance = 100;
|
||||||
foreach (SongInfoProvider* provider, ordered_providers) {
|
foreach (SongInfoProvider* provider, ordered_providers) {
|
||||||
provider->set_enabled(true);
|
provider->set_enabled(true);
|
||||||
qobject_cast<UltimateLyricsProvider*>(provider)->set_relevance(relevance++);
|
qobject_cast<UltimateLyricsProvider*>(provider)->set_relevance(relevance--);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Any lyric providers we don't have in ordered_providers are considered disabled
|
// Any lyric providers we don't have in ordered_providers are considered disabled
|
||||||
|
@ -119,6 +119,7 @@ void UltimateLyricsProvider::LyricsFetched(quint64 id, QNetworkReply* reply) {
|
|||||||
data.id_ = "ultimatelyrics/" + name_;
|
data.id_ = "ultimatelyrics/" + name_;
|
||||||
data.title_ = tr("Lyrics from %1").arg(name_);
|
data.title_ = tr("Lyrics from %1").arg(name_);
|
||||||
data.type_ = CollapsibleInfoPane::Data::Type_Lyrics;
|
data.type_ = CollapsibleInfoPane::Data::Type_Lyrics;
|
||||||
|
data.relevance_ = relevance();
|
||||||
|
|
||||||
AutoSizedTextEdit* editor = new AutoSizedTextEdit;
|
AutoSizedTextEdit* editor = new AutoSizedTextEdit;
|
||||||
editor->setHtml(lyrics);
|
editor->setHtml(lyrics);
|
||||||
|
Loading…
Reference in New Issue
Block a user