Correctly select first remote upon loading configuration; show a warning message if user presses a button that has not been configured

1 files changed, 15 insertions, 4 deletions

diff --git a/noncore/tools/remote/remotetab.cpp b/noncore/tools/remote/remotetab.cpp
index 84038ca..45995fd 100644
--- a/noncore/tools/remote/remotetab.cpp
+++ b/noncore/tools/remote/remotetab.cpp
@@ -16,2 +16,4 @@ Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 
+#include <qmessagebox.h>
+
 #include "remotetab.h"
@@ -53,6 +55,11 @@ int RemoteTab::sendIR()
         QString string = cfg->readEntry(button->name());
-        string+='\n';
-        
-        LircHandler lh;
-        return lh.sendIR(string.latin1());
+        if(string != "") {
+                string+='\n';
+                LircHandler lh;
+                return lh.sendIR(string.latin1());
+        }
+        else {
+                QMessageBox::warning(this, tr("Error"), tr("This button has not\nbeen configured"), QMessageBox::Ok, QMessageBox::NoButton);
+                return 0;
+        }
 }
@@ -64,2 +71,6 @@ void RemoteTab::setConfig(Config *newCfg)
         topGroup->updateRemotes(cfg);
+        
+        QString curr_remote = topGroup->getRemotesText();
+        if(curr_remote != "")
+                remoteSelected(curr_remote);
 }