- more factorization

4 files changed, 52 insertions, 14 deletions

diff --git a/noncore/apps/opie-console/metafactory.cpp b/noncore/apps/opie-console/metafactory.cpp
index 04f1cbe..ea7f739 100644
--- a/noncore/apps/opie-console/metafactory.cpp
+++ b/noncore/apps/opie-console/metafactory.cpp
@@ -44,6 +44,7 @@ QStringList MetaFactory::fileTransferLayers()const {
     }
     return list;
 }
+
 IOLayer* MetaFactory::newIOLayer( const QString& str,const Profile& prof ) {
     IOLayer* lay = 0l;
 
@@ -59,6 +60,16 @@ IOLayer* MetaFactory::newIOLayer( const QString& str,const Profile& prof ) {
     return lay;
 }
 
+QWidget *MetaFactory::newConfigWidget ( const QString& str, QWidget* parent) {
+    QWidget *w = NULL;
+    configWidget c;
+
+    c = m_confFact[str];
+    if(c) w = c(parent);
+
+    return w;
+}
+
 QString MetaFactory::name( const QString& str ) {
     return m_namemap[str];
 }
diff --git a/noncore/apps/opie-console/metafactory.h b/noncore/apps/opie-console/metafactory.h
index 7fa53c6..2245231 100644
--- a/noncore/apps/opie-console/metafactory.h
+++ b/noncore/apps/opie-console/metafactory.h
@@ -35,6 +35,7 @@ public:
     QStringList configWidgets()const;
     QStringList fileTransferLayers()const;
     IOLayer* newIOLayer( const QString&,const Profile& );
+    QWidget *newConfigWidget ( const QString&, QWidget* );
 
     QString name( const QString& );
 
diff --git a/noncore/apps/opie-console/profileeditordialog.cpp b/noncore/apps/opie-console/profileeditordialog.cpp
index 859abdd..374844e 100644
--- a/noncore/apps/opie-console/profileeditordialog.cpp
+++ b/noncore/apps/opie-console/profileeditordialog.cpp
@@ -19,30 +19,50 @@ static QWidget *factory_serial(QWidget *parent)
         device_frame->setFrameStyle(QFrame::Panel | QFrame::Sunken);
 
         QLabel *frame_device = new QLabel(QObject::tr("Device"), device_frame);
-        QLabel *frame_number = new QLabel(QObject::tr("Phone number"), device_frame);
-        frame_number->hide();
 
         QLineEdit *frame_device_line = new QLineEdit("/dev/ttyS0", device_frame);
-        QLineEdit *frame_number_line = new QLineEdit(device_frame);
-        frame_number_line->hide();
 
         QVBoxLayout *vbox_frame = new QVBoxLayout(device_frame, 2);
         vbox_frame->add(frame_device);
         vbox_frame->add(frame_device_line);
-        vbox_frame->add(frame_number);
-        vbox_frame->add(frame_number_line);
 
         return device_frame;
 }
 
 static QWidget *factory_irda(QWidget *parent)
 {
-        return NULL;
+        QFrame *device_frame = new QFrame(parent);
+        device_frame->setFrameStyle(QFrame::Panel | QFrame::Sunken);
+
+        QLabel *frame_device = new QLabel(QObject::tr("Device"), device_frame);
+
+        QLineEdit *frame_device_line = new QLineEdit("/dev/ircomm0", device_frame);
+
+        QVBoxLayout *vbox_frame = new QVBoxLayout(device_frame, 2);
+        vbox_frame->add(frame_device);
+        vbox_frame->add(frame_device_line);
+
+        return device_frame;
 }
 
 static QWidget *factory_modem(QWidget *parent)
 {
-        return NULL;
+        QFrame *device_frame = new QFrame(parent);
+        device_frame->setFrameStyle(QFrame::Panel | QFrame::Sunken);
+
+        QLabel *frame_device = new QLabel(QObject::tr("Device"), device_frame);
+        QLabel *frame_number = new QLabel(QObject::tr("Phone number"), device_frame);
+
+        QLineEdit *frame_device_line = new QLineEdit("/dev/ttyS0", device_frame);
+        QLineEdit *frame_number_line = new QLineEdit(device_frame);
+
+        QVBoxLayout *vbox_frame = new QVBoxLayout(device_frame, 2);
+        vbox_frame->add(frame_device);
+        vbox_frame->add(frame_device_line);
+        vbox_frame->add(frame_number);
+        vbox_frame->add(frame_number_line);
+
+        return device_frame;
 }
 
 ProfileEditorDialog::ProfileEditorDialog( MetaFactory* fact,
@@ -86,7 +106,11 @@ ProfileEditorDialog::ProfileEditorDialog( MetaFactory* fact )
                 device_box->insertItem(m_fact->name((*it)));
 
         // connection tab, factory part
-        QFrame *device_frame = static_cast<QFrame*>(factory_serial(tabconn));
+        plugin_base = new QWidget(tabconn);
+        plugin_layout = new QHBoxLayout(plugin_base, 0);
+
+        plugin_plugin = m_fact->newConfigWidget("serial", plugin_base);
+        plugin_layout->add(plugin_plugin);
 
         // connection tab, general part
 
@@ -160,7 +184,7 @@ ProfileEditorDialog::ProfileEditorDialog( MetaFactory* fact )
         QVBoxLayout *vbox = new QVBoxLayout(tabconn, 2);
         vbox->add(device);
         vbox->add(device_box);
-        vbox->add(device_frame);
+        vbox->add(plugin_base);
         vbox->add(speed);
         vbox->add(speed_box);
         vbox->add(flow);
@@ -214,12 +238,12 @@ ProfileEditorDialog::~ProfileEditorDialog() {
 
 void ProfileEditorDialog::slotDevice(int id)
 {
-        MetaFactory::configWidget c;
+        delete plugin_plugin;
 
-        //c = m_fact->;
-        c = factory_serial;
+        plugin_plugin = m_fact->newConfigWidget(prof_type(), plugin_base);
+        plugin_layout->add(plugin_plugin);
 
-        QFrame *device_frame = static_cast<QFrame*>(c(NULL));
+        plugin_plugin->show();
 }
 
 void ProfileEditorDialog::slotOk()
diff --git a/noncore/apps/opie-console/profileeditordialog.h b/noncore/apps/opie-console/profileeditordialog.h
index dfe4490..00f3c99 100644
--- a/noncore/apps/opie-console/profileeditordialog.h
+++ b/noncore/apps/opie-console/profileeditordialog.h
@@ -52,6 +52,8 @@ private:
         QLineEdit *name_line;
         //QLineEdit *frame_device_line, *frame_number_line;
         QComboBox *terminal_box, *speed_box, *device_box;
+        QWidget *plugin_base, *plugin_plugin;
+        QHBoxLayout *plugin_layout;
 };
 
 #endif