- io_modem must keep copy of profile, otherwise it crashes (now it works -

  doh!)
- try to use key/value pairs for dialing where currently possible (init string
  and dial sequence)
- when editing telephone number, try to read in, including special chars
  stripping (-, /, whitespace) using QRegExp

6 files changed, 21 insertions, 9 deletions

diff --git a/noncore/apps/opie-console/dialdialog.cpp b/noncore/apps/opie-console/dialdialog.cpp
index 8bf32f9..0ace8f7 100644
--- a/noncore/apps/opie-console/dialdialog.cpp
+++ b/noncore/apps/opie-console/dialdialog.cpp
@@ -93,3 +93,11 @@ DialDialog::~DialDialog() {
 QString DialDialog::number() {
     return QString( "%1").arg( m_number );
+
 }
+
+void DialDialog::setNumber( int number )
+{
+    m_number = number;
+    LCD->display( m_number );
+}
+
diff --git a/noncore/apps/opie-console/dialdialog.h b/noncore/apps/opie-console/dialdialog.h
index 5c5b948..bec7b81 100644
--- a/noncore/apps/opie-console/dialdialog.h
+++ b/noncore/apps/opie-console/dialdialog.h
@@ -18,4 +18,6 @@ public:
     QString number();
 
+    void setNumber( int );
+
 private:
     float m_number;
diff --git a/noncore/apps/opie-console/dialer.cpp b/noncore/apps/opie-console/dialer.cpp
index 90e2b6c..89a0e8d 100644
--- a/noncore/apps/opie-console/dialer.cpp
+++ b/noncore/apps/opie-console/dialer.cpp
@@ -126,5 +126,6 @@ void Dialer::trydial(const QString& number)
 	{
 		switchState(state_init);
-		send("ATZ");
+		//send("ATZ");
+		send(m_profile.readEntry("InitString"));
 		QString response2 = receive();
 	}
@@ -150,5 +151,5 @@ void Dialer::trydial(const QString& number)
 		switchState(state_dialing);
 
-		send(QString("ATDT %1").arg(number));
+		send(QString("%1 %2").arg(m_profile.readEntry("DialPrefix1")).arg(number));
 		QString response5 = receive();
 	}
diff --git a/noncore/apps/opie-console/io_modem.cpp b/noncore/apps/opie-console/io_modem.cpp
index d93dc5a..22a3673 100644
--- a/noncore/apps/opie-console/io_modem.cpp
+++ b/noncore/apps/opie-console/io_modem.cpp
@@ -5,5 +5,6 @@
 
 IOModem::IOModem( const Profile &profile )
-	: IOSerial( profile ),  m_profile( profile ) {
+	: IOSerial( profile ) {
+	m_profile = profile;
 }
 
@@ -15,7 +16,7 @@ IOModem::~IOModem() {
 
 void IOModem::close() {
+	// maybe do a hangup here just in case...?
 
     IOSerial::close();
-
 }
 
@@ -24,8 +25,5 @@ bool IOModem::open() {
 	if(!ret) return false;
 
-	qWarning("IOModem::open continues...");
-
 	Dialer d(m_profile);
-	qWarning("dialer created");
 
 	int result = d.exec();
diff --git a/noncore/apps/opie-console/io_modem.h b/noncore/apps/opie-console/io_modem.h
index d681f66..8453b95 100644
--- a/noncore/apps/opie-console/io_modem.h
+++ b/noncore/apps/opie-console/io_modem.h
@@ -62,5 +62,5 @@ private:
     int  m_dialTime, m_delayRedial, m_numberTries, m_dtrDropTime,
         m_bpsDetect, m_dcdLines, m_multiLineUntag;
-	const Profile& m_profile;
+	Profile m_profile;
 
 private slots:
diff --git a/noncore/apps/opie-console/modemconfigwidget.cpp b/noncore/apps/opie-console/modemconfigwidget.cpp
index 0f0ce7c..878b0cf 100644
--- a/noncore/apps/opie-console/modemconfigwidget.cpp
+++ b/noncore/apps/opie-console/modemconfigwidget.cpp
@@ -5,5 +5,5 @@
 #include <qpushbutton.h>
 #include <qhbox.h>
-
+#include <qregexp.h>
 
 #include "modemconfigwidget.h"
@@ -188,4 +188,7 @@ void ModemConfigWidget::slotAT() {
 void ModemConfigWidget::slotDial() {
     DialDialog dial( this, "DialConfig", true );
+    if(!m_telNumber->text().isEmpty()) {
+        dial.setNumber(m_telNumber->text().replace(QRegExp("[\\-\\/\\ ]"), "").toInt());
+    }
     dial.showMaximized();
     if ( dial.exec() == QDialog::Accepted ) {