summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTimothy Pearson <kb9vqf@pearsoncomputing.net>2013-04-06 01:52:59 -0500
committerTimothy Pearson <kb9vqf@pearsoncomputing.net>2013-04-06 01:52:59 -0500
commit6fd6724b2f4c1a0bed961eeb267381c7d54746c0 (patch)
treea35d6ed6a59bc81a8d1d30f170cdc5f8d6eee8ad
parent02d43b793cdfc6e9a5ef09397619654e4d69bbc0 (diff)
downloadtdebase-6fd6724b2f4c1a0bed961eeb267381c7d54746c0.tar.gz
tdebase-6fd6724b2f4c1a0bed961eeb267381c7d54746c0.zip
Add warning messages on hot key command execution failure
-rw-r--r--khotkeys/shared/actions.cpp9
1 files changed, 7 insertions, 2 deletions
diff --git a/khotkeys/shared/actions.cpp b/khotkeys/shared/actions.cpp
index 3174b1dcf..c8590789a 100644
--- a/khotkeys/shared/actions.cpp
+++ b/khotkeys/shared/actions.cpp
@@ -29,6 +29,7 @@
#include <tdeaccel.h>
#include <kservice.h>
#include <kprocess.h>
+#include <tdemessagebox.h>
#include "windows.h"
#include "action_data.h"
@@ -153,7 +154,10 @@ void Command_url_action::execute()
KService::Ptr service = KService::serviceByDesktopName( cmd );
if( service != NULL )
{
- KRun::run( *service, KURL::List());
+ if (!KRun::run( *service, KURL::List()))
+ {
+ KMessageBox::sorry(0, "<qt>" + i18n("KHotKeys was unable to execute") + " '" + cmd + "'<p>" + i18n("Please verify existence of the service") + "</qt>", i18n("Unable to launch service!"));
+ }
break;
}
}
@@ -166,11 +170,12 @@ void Command_url_action::execute()
if( !KRun::runCommand(
cmd + ( uri.hasArgsAndOptions() ? uri.argsAndOptions() : "" ),
cmd, uri.iconName())) {
- // CHECKME ?
+ KMessageBox::sorry(0, "<qt>" + i18n("KHotKeys was unable to execute") + " '" + cmd + "'<p>" + i18n("Please verify existence and permissions of the executable file") + "</qt>", i18n("Unable to launch program"));
}
break;
}
default: // error
+ KMessageBox::sorry(0, "<qt>" + i18n("KHotKeys was unable to execute") + " '" + cmd + "'<p>" + i18n("Please verify existence and permissions of the executable file") + "</qt>", i18n("Unable to launch program"));
return;
}
timeout.start( 1000, true ); // 1sec timeout