Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't connect action signals multiple times.

Browse Source 
Fixes #5249
This commit is contained in:
John Maguire 2016-02-11 17:07:00 +00:00
parent ce3ba02749
commit 7d1abbda55
1 changed files with 14 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/internet/core/internetservice.cpp
View File

@ -42,19 +42,26 @@ InternetService::InternetService(const QString& name, Application* app,
app_(app),
model_(model),
name_(name),
append_to_playlist_([]() {
return new QAction(
append_to_playlist_([&]() {
QAction* action = new QAction(
IconLoader::Load("media-playback-start", IconLoader::Base),
tr("Append to current playlist"), nullptr);
connect(action, SIGNAL(triggered()), this, SLOT(AppendToPlaylist()));
return action;
}),
replace_playlist_([]() {
return new QAction(
replace_playlist_([&]() {
QAction* action = new QAction(
IconLoader::Load("media-playback-start", IconLoader::Base),
tr("Replace current playlist"), nullptr);
connect(action, SIGNAL(triggered()), this, SLOT(ReplacePlaylist()));
return action;
}),
open_in_new_playlist_([]() {
return new QAction(IconLoader::Load("document-new", IconLoader::Base),
open_in_new_playlist_([&]() {
QAction* action =
new QAction(IconLoader::Load("document-new", IconLoader::Base),
tr("Open in new playlist"), nullptr);
connect(action, SIGNAL(triggered()), this, SLOT(OpenInNewPlaylist()));
return action;
}),
separator_([]() {
QAction* action = new QAction(nullptr);
@ -85,20 +92,14 @@ QList<QAction*> InternetService::GetPlaylistActions() {
}
QAction* InternetService::GetAppendToPlaylistAction() {
connect(append_to_playlist_.get(), SIGNAL(triggered()), this,
SLOT(AppendToPlaylist()));
return append_to_playlist_.get();
}
QAction* InternetService::GetReplacePlaylistAction() {
connect(replace_playlist_.get(), SIGNAL(triggered()), this,
SLOT(ReplacePlaylist()));
return replace_playlist_.get();
}
QAction* InternetService::GetOpenInNewPlaylistAction() {
connect(open_in_new_playlist_.get(), SIGNAL(triggered()), this,
SLOT(OpenInNewPlaylist()));
return open_in_new_playlist_.get();
}