public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Kevin Kofler <Kevin@tigcc.ticalc.org>
To: git-commits@fedoraproject.org
Subject: [rpms/amarok] epel10: Fix the onlinedoc patch.
Date: Mon, 06 Jul 2026 18:04:58 GMT	[thread overview]
Message-ID: <178336109866.1.645592256467038548.rpms-amarok-5c917b00cdfc@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/amarok
Branch : epel10
Commit : 5c917b00cdfc0e294cc4c532b2feacf8bed1259c
Author : Kevin Kofler <Kevin@tigcc.ticalc.org>
Date   : 2012-01-29T07:47:52+01:00
Stats  : +38/-12 in 1 file(s)
URL    : https://src.fedoraproject.org/rpms/amarok/c/5c917b00cdfc0e294cc4c532b2feacf8bed1259c?branch=epel10

Log:
Fix the onlinedoc patch.

---
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

                 reply	other threads:[~2026-07-06 18:04 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178336109866.1.645592256467038548.rpms-amarok-5c917b00cdfc@fedoraproject.org \
    --to=kevin@tigcc.ticalc.org \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox