qcop channels for start, stop wlan interface works now...

... the qcop handling has to become more general, and this handlet in the interface itself.

4 files changed, 65 insertions, 20 deletions

diff --git a/noncore/settings/networksettings/TODO b/noncore/settings/networksettings/TODO
index 614501f..b28c5a0 100644
--- a/noncore/settings/networksettings/TODO
+++ b/noncore/settings/networksettings/TODO
@@ -1,4 +1,10 @@
+* make dcop handling portable
+* move dcop call for start, stop interfaces from wlan to interface
+
+
+Old TODO of Benmayer:
+
 Types:
 -Ethernet Connection (Done)
 -ISDN Connection
 -Modem COnnection (Started)
diff --git a/noncore/settings/networksettings/wlan/wlanimp2.cpp b/noncore/settings/networksettings/wlan/wlanimp2.cpp
index e7b842b..b988822 100644
--- a/noncore/settings/networksettings/wlan/wlanimp2.cpp
+++ b/noncore/settings/networksettings/wlan/wlanimp2.cpp
@@ -13,9 +13,9 @@
 #include <qcheckbox.h>
 #include <qtabwidget.h>
 #include <qcombobox.h>
 
-#ifdef QWS 
+#ifdef QWS
  #include <opie/oprocess.h>
 #else
  #define OProcess KProcess
  #include <kprocess.h>
@@ -25,9 +25,9 @@
 #define PREUP "/etc/netwrok/if-pre-up.d/wireless-tools"
 
 /**
  * Constructor, read in the wireless.opts file for parsing later.
- */ 
+ */
 WLANImp::WLANImp( QWidget* parent, const char* name, Interface *i, bool modal, WFlags fl) : WLAN(parent, name, modal, fl), currentProfile("*"), interface(i) {
   interfaces = new Interfaces;
   interfaceSetup = new InterfaceSetupImp(tabWidget, "InterfaceSetupImp", i, interfaces);
   tabWidget->insertTab(interfaceSetup, "TCP/IP");
@@ -39,14 +39,14 @@ WLANImp::WLANImp( QWidget* parent, const char* name, Interface *i, bool modal, W
   }
 }
 
 WLANImp::~WLANImp() {
-  delete interfaces;
+//FIXME:  delete interfaces;
 }
 
 /**
  * Change the profile for both wireless settings and network settings.
- */ 
+ */
 void WLANImp::setProfile(const QString &profile){
   interfaceSetup->setProfile(profile);
   parseOpts();
 }
@@ -57,9 +57,9 @@ void WLANImp::parseOpts() {
 
   if (! interfaces->isInterfaceSet())
     return;
 
-  
+
   opt = interfaces->getInterfaceOption("wireless_essid", error);
   if(opt == "any" || opt == "off" || opt.isNull()){
     essid->setEditText("any");
   } else {
@@ -74,9 +74,9 @@ void WLANImp::parseOpts() {
   } else {
     // Managed/Infrastructure mode
     mode->setCurrentItem(1);
   }
-  
+
   opt = interfaces->getInterfaceOption("wireless_ap", error).simplifyWhiteSpace();
   if (! opt.isNull()) {
     specifyAp->setChecked(true);
     macEdit->setText(opt);
@@ -131,17 +131,17 @@ void WLANImp::parseKeyStr(QString keystr) {
 	  break;
         case 4:
           keyRadio3->setChecked(true);
 	  break;
-      }   
+      }
     } else {
       // key
       key = (*it);
     }
     if (! key.isNull()) {
       if (enc == -1)
         enc = 1;
-      QStringList::Iterator next = ++it;   
+      QStringList::Iterator next = ++it;
       if (it == keys.end()) {
         break;
       }
       if ((*(next)).left(1) == "[") {
@@ -186,10 +186,10 @@ void WLANImp::accept() {
         (keyRadio3->isChecked() && keyLineEdit3->text().isEmpty())) {
       QMessageBox::information(this, "Error", "Please enter a WEP key.", QMessageBox::Ok);
       return;
     }
-  }	
-  
+  }
+
   if (essid->currentText().isEmpty()) {
     QMessageBox::information(this, "Error", "Please select/enter an ESSID.", QMessageBox::Ok);
     return;
   }
@@ -197,9 +197,9 @@ void WLANImp::accept() {
   if (specifyAp->isChecked() && macEdit->text().isEmpty()) {
     QMessageBox::information(this, "Error", "Please enter the MAC address of the Access Point.", QMessageBox::Ok);
     return;
   }
-  
+
   // Try to save the interfaces settings.
   writeOpts();
 
   // Close out the dialog
diff --git a/noncore/settings/networksettings/wlan/wlanmodule.cpp b/noncore/settings/networksettings/wlan/wlanmodule.cpp
index 3c988d5..e6f082c 100644
--- a/noncore/settings/networksettings/wlan/wlanmodule.cpp
+++ b/noncore/settings/networksettings/wlan/wlanmodule.cpp
@@ -7,8 +7,11 @@
 #include <qlabel.h>
 #include <qprogressbar.h>
 #include <qtabwidget.h>
 
+
+
+
 /**
  * Constructor, find all of the possible interfaces
  */
 WLANModule::WLANModule() : Module() {
@@ -20,8 +23,9 @@ WLANModule::WLANModule() : Module() {
 WLANModule::~WLANModule(){
   Interface *i;
   for ( i=list.first(); i != 0; i=list.next() )
     delete i;
+
 }
 
 /**
  * Change the current profile
@@ -72,12 +76,9 @@ QWidget *WLANModule::information(Interface *i){
   WExtensions we(i->getInterfaceName());
   if(!we.doesHaveWirelessExtensions())
     return NULL;
 
-  WlanInfoImp *info = new WlanInfoImp(0, i->getInterfaceName(), Qt::WDestructiveClose);
-  InterfaceInformationImp *information = new InterfaceInformationImp(info->tabWidget, "InterfaceSetupImp", i);
-  info->tabWidget->insertTab(information, "TCP/IP");
-  return info;
+  return getInfo( i );
 }
 
 /**
  * Get all active (up or down) interfaces
@@ -127,22 +128,45 @@ void WLANModule::receive(const QCString &param, const QByteArray &arg)
 
     stream >> interface;
     stream >> action;
     qDebug("got interface %s and acion %s", interface.latin1(), action.latin1());
+    // find interfaces
+    Interface *ifa=0;
+    for ( Interface *i=list.first(); i != 0; i=list.next() ){
+        if (i->getInterfaceName() == interface){
+            qDebug("found interface %s",interface.latin1());
+            ifa = i;
+        }
+    }
+
+    if (ifa == 0){
+        qFatal("Did not find %s",interface.latin1());
+    }
 
     if (count == 2){
-        // those should call the interface
+        // those should call the interface directly
+        QWidget *info = getInfo( ifa );
+        info->showMaximized();
+
         if ( action.contains("start" ) ){
-            qDebug("starting %s not yet implemented",interface.latin1());
+            ifa->start();
         } else if ( action.contains("restart" ) ){
-            qDebug("restarting %s not yet implemented",interface.latin1());
+            ifa->restart();
         } else if ( action.contains("stop" ) ){
-            qDebug("stopping %s not yet implemented",interface.latin1());
+            ifa->stop();
+        }else if ( action.contains("refresh" ) ){
+            ifa->refresh();
         }
     }else if (count == 3){
         QString value;
         stream >> value;
         qDebug("setting %s of %s to %s", action.latin1(), interface.latin1(), value.latin1() );
+        if ( action.contains("ESSID") ){
+            qDebug("Setting ESSID not yet impl");
+        }else if (action.contains("Channel")){
+            qDebug("Setting Channel not yet impl");
+        }else
+            qDebug("wlan plugin has no clue");
     }
     // if (param.contains("QString,QString,QString")) {
 //         QDataStream stream(arg,IO_ReadOnly);
 //         QString arg1, arg2, arg3;
@@ -150,4 +174,14 @@ void WLANModule::receive(const QCString &param, const QByteArray &arg)
 //         qDebug("interface >%s< setting >%s< value >%s<",arg1.latin1(),arg2.latin1(),arg3.latin1());
 //     }
 }
 
+QWidget *WLANModule::getInfo( Interface *i)
+{
+    qDebug("WLANModule::getInfo start");
+    WlanInfoImp *info = new WlanInfoImp(0, i->getInterfaceName(), Qt::WDestructiveClose);
+    InterfaceInformationImp *information = new InterfaceInformationImp(info->tabWidget, "InterfaceSetupImp", i);
+    info->tabWidget->insertTab(information, "TCP/IP", 0);
+
+    qDebug("WLANModule::getInfo return");
+    return info;
+}
diff --git a/noncore/settings/networksettings/wlan/wlanmodule.h b/noncore/settings/networksettings/wlan/wlanmodule.h
index a617a90..0963137 100644
--- a/noncore/settings/networksettings/wlan/wlanmodule.h
+++ b/noncore/settings/networksettings/wlan/wlanmodule.h
@@ -2,8 +2,10 @@
 #define WLAN_MODULE_H
 
 #include "module.h"
 
+//class WlanInfoImp;
+
 class WLANModule : Module{
 
 signals:
   void updateInterface(Interface *i);
@@ -25,11 +27,14 @@ public:
   QString getPixmapName(Interface* i);
   virtual void receive(const QCString&, const QByteArray&);
 
 private:
+  QWidget *getInfo(Interface*);
+
   QList<Interface> list;
   QString profile;
-
+  //  WlanInfoImp *info;
+  //  Interface *iface;
 };
 
 extern "C"
 {