Calmed down... 2nd try after merge

Use MetaFactory cuase it's so nice and avoids thousands of if() else if
stuff
Fix a bug in filetransfer where cancel after exit leaded to a crash cause
of bogus adresses in the QSocketNotifiers..

1 files changed, 39 insertions, 31 deletions

diff --git a/noncore/apps/opie-console/transferdialog.cpp b/noncore/apps/opie-console/transferdialog.cpp
index d639de6..ba06199 100644
--- a/noncore/apps/opie-console/transferdialog.cpp
+++ b/noncore/apps/opie-console/transferdialog.cpp
@@ -1,2 +1,10 @@
-#include "transferdialog.h"
+#include <qlayout.h>
+#include <qcombobox.h>
+#include <qlabel.h>
+#include <qlineedit.h>
+#include <qpushbutton.h>
+#include <qmessagebox.h>
+#include <qprogressbar.h>
+
+#include <opie/ofiledialog.h>
 
@@ -4,16 +12,17 @@
 #include "io_serial.h"
+#include "metafactory.h"
+#include "mainwindow.h"
+
+#include "transferdialog.h"
+
 
-#include "qlayout.h"
-#include "qcombobox.h"
-#include "qlabel.h"
-#include "qlineedit.h"
-#include "qpushbutton.h"
-#include "qmessagebox.h"
-#include "qprogressbar.h"
 
-#include "opie/ofiledialog.h"
 
-TransferDialog::TransferDialog(QWidget *parent, const char *name)
-: QDialog(/*parent, name*/NULL, NULL, true)
+
+
+
+TransferDialog::TransferDialog(MainWindow *parent, const char *name)
+: QDialog(/*parent, name*/0l, 0l, true), m_win(parent)
 {
+        m_lay = 0l;
         QVBoxLayout *vbox;
@@ -23,3 +32,2 @@ TransferDialog::TransferDialog(QWidget *parent, const char *name)
 
-        transfer = NULL;
 
@@ -34,5 +42,10 @@ TransferDialog::TransferDialog(QWidget *parent, const char *name)
         protocol = new QComboBox(this);
-        protocol->insertItem("XModem");
-        protocol->insertItem("YModem");
-        protocol->insertItem("ZModem");
+        /* use the fscking MetaFactory
+         * because we invented it for that fscking reason
+         * I'm really getting UPSET!!!!
+         */
+        QStringList list = m_win->factory()->fileTransferLayers();
+        for (QStringList::Iterator it =list.begin(); it != list.end(); ++it ) {
+            protocol->insertItem( (*it) );
+        }
 
@@ -78,3 +91,3 @@ void TransferDialog::slotFilename()
         QString f;
-        
+
         f = OFileDialog::getOpenFileName(0);
@@ -97,14 +110,9 @@ void TransferDialog::slotTransfer()
 
-        FileTransfer::Type transfermode = FileTransfer::SX;
-        if(protocol->currentText() == "YModem") transfermode == FileTransfer::SY;
-        if(protocol->currentText() == "ZModem") transfermode == FileTransfer::SZ;
-
-        // dummy profile
-        Profile profile("Dummy", "serial", "vt102", Profile::White, Profile::Black, Profile::VT102);
+        m_lay = m_win->factory()->newFileTransfer( protocol->currentText(),
+                                                   m_win->currentSession()->layer() );
+        m_lay->sendFile(filename->text());
 
-        transfer = new FileTransfer(transfermode, new IOSerial(profile));
-        transfer->sendFile(filename->text());
-        connect(transfer, SIGNAL(progress(const QString&, int, int, int, int, int)), SLOT(slotProgress(const QString&, int, int, int, int, int)));
-        connect(transfer, SIGNAL(error(int, const QString&)), SLOT(slotError(int, const QString&)));
-        connect(transfer, SIGNAL(sent()), SLOT(slotSent()));
+         connect(m_lay, SIGNAL(progress(const QString&, int, int, int, int, int)), SLOT(slotProgress(const QString&, int, int, int, int, int)));
+        connect(m_lay, SIGNAL(error(int, const QString&)), SLOT(slotError(int, const QString&)));
+        connect(m_lay, SIGNAL(sent()), SLOT(slotSent()));
 }
@@ -115,7 +123,7 @@ void TransferDialog::slotCancel()
 
-        if(transfer)
+        if(m_lay)
         {
-                transfer->cancel();
-                delete transfer;
-                transfer = NULL;
+                m_lay->cancel();
+                delete  m_lay;
+                m_lay = 0l;
                 QMessageBox::information(this,