[amarok] Fix the onlinedoc patch.
Kevin Kofler
kkofler at fedoraproject.org
Sun Jan 29 06:48:00 UTC 2012
commit 5c917b00cdfc0e294cc4c532b2feacf8bed1259c
Author: Kevin Kofler <Kevin at tigcc.ticalc.org>
Date: Sun Jan 29 07:47:52 2012 +0100
Fix the onlinedoc patch.
amarok-2.5.0-onlinedoc.patch | 50 +++++++++++++++++++++++++++++++----------
1 files changed, 38 insertions(+), 12 deletions(-)
---
diff --git a/amarok-2.5.0-onlinedoc.patch b/amarok-2.5.0-onlinedoc.patch
index d42db67..74dae36 100644
--- a/amarok-2.5.0-onlinedoc.patch
+++ b/amarok-2.5.0-onlinedoc.patch
@@ -1,6 +1,6 @@
diff -ur amarok-2.5.0/src/ActionClasses.cpp amarok-2.5.0-onlinedoc/src/ActionClasses.cpp
--- amarok-2.5.0/src/ActionClasses.cpp 2011-12-15 08:36:48.000000000 +0100
-+++ amarok-2.5.0-onlinedoc/src/ActionClasses.cpp 2012-01-28 08:01:44.000000000 +0100
++++ amarok-2.5.0-onlinedoc/src/ActionClasses.cpp 2012-01-29 07:43:56.000000000 +0100
@@ -31,10 +31,12 @@
#include "playlist/PlaylistActions.h"
#include "playlist/PlaylistModelStack.h"
@@ -14,7 +14,15 @@ diff -ur amarok-2.5.0/src/ActionClasses.cpp amarok-2.5.0-onlinedoc/src/ActionCla
#include <Osd.h>
#include <EqualizerDialog.h>
-@@ -152,6 +154,17 @@
+@@ -63,6 +65,7 @@
+ using namespace Amarok;
+
+ KHelpMenu *Menu::s_helpMenu = 0;
++OnlineHelpHandler *Menu::s_onlineHelpHandler = 0;
+
+ static void
+ safePlug( KActionCollection *ac, const char *name, QWidget *w )
+@@ -152,6 +155,20 @@
KMenu* menu = s_helpMenu->menu();
@@ -23,21 +31,24 @@ diff -ur amarok-2.5.0/src/ActionClasses.cpp amarok-2.5.0-onlinedoc/src/ActionCla
+ QAction *contentsAction = s_helpMenu->action( KHelpMenu::menuHelpContents );
+ // disconnect the default slot which fires up KHelpCenter
+ disconnect( contentsAction, SIGNAL(triggered(bool)), s_helpMenu, SLOT(appHelpActivated()) );
-+ // ensure no duplicate connection
-+ disconnect( contentsAction, SIGNAL(triggered(bool)), this, SLOT(onlineHelpActivated()) );
++ if ( s_onlineHelpHandler == 0 )
++ s_onlineHelpHandler = new OnlineHelpHandler( parent );
++ else
++ // ensure no duplicate connection
++ disconnect( contentsAction, SIGNAL(triggered(bool)), s_onlineHelpHandler, SLOT(onlineHelpActivated()) );
+ // connect our custom slot which fires up a browser on UserBase instead
-+ connect( contentsAction, SIGNAL(triggered(bool)), this, SLOT(onlineHelpActivated()) );
++ connect( contentsAction, SIGNAL(triggered(bool)), s_onlineHelpHandler, SLOT(onlineHelpActivated()) );
+ }
+
// NOTE: We hide "Report Bug..." because we need to add it on our own to name the dialog
// so it can be blacklisted from LikeBack.
s_helpMenu->action( KHelpMenu::menuReportBug )->setVisible( false );
-@@ -164,6 +177,12 @@
+@@ -164,6 +181,12 @@
return menu;
}
+void
-+Menu::onlineHelpActivated()
++OnlineHelpHandler::onlineHelpActivated()
+{
+ KToolInvocation::invokeBrowser( "http://userbase.kde.org/Amarok/Manual" );
+}
@@ -47,14 +58,29 @@ diff -ur amarok-2.5.0/src/ActionClasses.cpp amarok-2.5.0-onlinedoc/src/ActionCla
//////////////////////////////////////////////////////////////////////////////////////////
diff -ur amarok-2.5.0/src/ActionClasses.h amarok-2.5.0-onlinedoc/src/ActionClasses.h
--- amarok-2.5.0/src/ActionClasses.h 2011-12-15 08:36:48.000000000 +0100
-+++ amarok-2.5.0-onlinedoc/src/ActionClasses.h 2012-01-28 07:55:40.000000000 +0100
-@@ -42,6 +42,9 @@
- static Menu *instance();
- static KMenu *helpMenu( QWidget *parent = 0 );
++++ amarok-2.5.0-onlinedoc/src/ActionClasses.h 2012-01-29 07:34:57.000000000 +0100
+@@ -34,6 +34,16 @@
-+ private slots:
+ namespace Amarok
+ {
++ class OnlineHelpHandler : public QObject
++ {
++ Q_OBJECT
++ public:
++ OnlineHelpHandler( QObject* parent );
++
++ public slots:
+ void onlineHelpActivated();
++ }
+
+ class Menu : public KMenu
+ {
+ Q_OBJECT
+@@ -45,6 +55,7 @@
private:
static Menu *s_instance;
static KHelpMenu *s_helpMenu;
++ static OnlineHelpHandler *s_onlineHelpHandler;
+ };
+
+ class MenuAction : public KAction
More information about the scm-commits
mailing list