More fixes for remote selection

2 files changed, 25 insertions, 0 deletions

diff --git a/noncore/tools/remote/remotetab.cpp b/noncore/tools/remote/remotetab.cpp
index 45995fd..c89d8c2 100644
--- a/noncore/tools/remote/remotetab.cpp
+++ b/noncore/tools/remote/remotetab.cpp
@@ -22,64 +22,72 @@ Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 RemoteTab::RemoteTab(QWidget *parent, const char *name):QWidget(parent,name)
 {
         QVBoxLayout *layout = new QVBoxLayout(this);
 
         topGroup = new TopGroup(this);
         //topGroup->setMaximumHeight(22);
         layout->addWidget(topGroup, 0, 0);
         printf("%d %d", topGroup->width(), topGroup->height());
 
         layout->addSpacing(1);
 
         dvdGroup = new DVDGroup(this);
         //dvdGroup->setMaximumHeight(68);
         layout->addWidget(dvdGroup, 0, 0);
 
         layout->addSpacing(1);
 
         vcrGroup = new VCRGroup(this);
         layout->addWidget(vcrGroup, 0, 0);
         //vcrGroup->setMaximumHeight(45);
 
         layout->addSpacing(1);
 
         channelGroup = new ChannelGroup(this);
         //channelGroup->setMaximumHeight(91);
         layout->addWidget(channelGroup, 0, 0);
 
         this->setMaximumWidth(240);
 }
 
 int RemoteTab::sendIR()
 {
+        QString curr_remote = topGroup->getRemotesText();
+        if(curr_remote != "")
+                cfg->setGroup(curr_remote);
+        else {
+                QMessageBox::warning(this, tr("Error"), tr("Please select or create\na remote layout"), QMessageBox::Ok, QMessageBox::NoButton);
+                return 0;
+        }
+        
         const QObject *button = sender();
         QString string = cfg->readEntry(button->name());
         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;
         }
 }
 
 void RemoteTab::setConfig(Config *newCfg)
 {
         cfg = newCfg;
         cfg->setGroup("Remotes");
         topGroup->updateRemotes(cfg);
         
         QString curr_remote = topGroup->getRemotesText();
         if(curr_remote != "")
                 remoteSelected(curr_remote);
 }
 
 void RemoteTab::remoteSelected(const QString &string)
 {
         printf("1%s\n", string.latin1() );
         cfg->setGroup(string);
         const QObject *obj;
 
         const QObjectList *objList = topGroup->children();
         for(obj = ((QObjectList *)objList)->first(); obj != 0; obj=((QObjectList *)objList)->next())
@@ -128,33 +136,37 @@ void RemoteTab::remoteSelected(const QString &string)
                         }
                         else
                         {
                                 cfg->setGroup("Default");
                                 ((QPushButton *)obj)->setText(cfg->readEntry((QString)obj->name()+"Label") );
                                 cfg->setGroup(string);
                         }
                 }
         }
 
         objList = channelGroup->children();
         for(obj = ((QObjectList *)objList)->first(); obj != 0; obj=((QObjectList *)objList)->next())
         {
                 if(obj->inherits("QPushButton"))
                 {
                         if(cfg->hasKey((QString)obj->name()+"Label"))
                         {
                                 ((QPushButton *)obj)->setText(cfg->readEntry((QString)obj->name()+"Label") );
                         }
                         else
                         {
                                 cfg->setGroup("Default");
                                 ((QPushButton *)obj)->setText(cfg->readEntry((QString)obj->name()+"Label") );
                                 cfg->setGroup(string);
                         }
                 }
         }
 }
 
 void RemoteTab::updateRemotesList()
 {
         topGroup->updateRemotes(cfg);
+        
+        QString curr_remote = topGroup->getRemotesText();
+        if(curr_remote != "")
+                remoteSelected(curr_remote);
 }
diff --git a/noncore/tools/remote/topgroup.cpp b/noncore/tools/remote/topgroup.cpp
index 7f2366b..d4eb6e2 100644
--- a/noncore/tools/remote/topgroup.cpp
+++ b/noncore/tools/remote/topgroup.cpp
@@ -18,41 +18,54 @@ Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 
 TopGroup::TopGroup(QWidget *parent, const char *name):QWidget(parent,name)
 {
         QHBoxLayout *layout = new QHBoxLayout(this, 0, -1, 0);
 
         QPushButton *power = new QPushButton("Power",this,"power");
         layout->addWidget(power);
         connect(power, SIGNAL(pressed()), this->parentWidget(), SLOT(sendIR()) );
         //power->setGeometry(5, 5,40, 20);
 
         layout->addSpacing(5);
 
         QPushButton *source = new QPushButton("Source",this,"source");
         layout->addWidget(source);
         connect(source, SIGNAL(pressed()), this->parentWidget(), SLOT(sendIR()) );
                 //source->setGeometry(50,5,40,20);
 
         remotes = new QComboBox(false, this, "remotes");
         connect(remotes, SIGNAL(activated(const QString&)), this->parentWidget(), SLOT(remoteSelected(const QString&)) );
         remotes->insertItem("Select Remote");
         //remotes->setGeometry(135,5,95,20);
 
         QLabel *remoteLabel = new QLabel(remotes, "Remote: ",this,"remoteLabel");
         //remoteLabel->setGeometry(90,5,40,20);
         remoteLabel->setAlignment(AlignRight | AlignVCenter);
 
         layout->addWidget(remoteLabel);
         layout->addWidget(remotes);
 }
 
 void TopGroup::updateRemotes(Config *cfg)
 {
+        // Save currently selected item if any
+        QString curr_remote = remotes->currentText();
+        
         remotes->clear();
         cfg->setGroup("Remotes");
         remotes->insertStringList(cfg->readListEntry("remoteList", ',') );
+        
+        // Select previously selected item
+        if(curr_remote != "") {
+                for(int i=0;i<remotes->count();i++) {
+                        if(remotes->text(i) == curr_remote) {
+                                remotes->setCurrentItem(i);
+                                break;
+                        }
+                }
+        }
 }
 
 QString TopGroup::getRemotesText()
 {
         return remotes->currentText();
 }