Clear both IRDA and BT lists at startup.

Enable IRDA and bluetooth at startup only.
Move index to the beginning after send.
Added status line change.
Added checks for NULL pointers and empty list of bluetooth devices.

1 files changed, 41 insertions, 33 deletions

diff --git a/core/obex/obexsend.cpp b/core/obex/obexsend.cpp
index bca6784..45754e3 100644
--- a/core/obex/obexsend.cpp
+++ b/core/obex/obexsend.cpp
@@ -70,26 +70,48 @@ void SendWidget::initUI() {
  * in send we'll first set everything up
  * and then wait for a list of devices.
  */
 void SendWidget::send( const QString& file, const QString& desc ) {
     m_file = file;
     m_irDa.clear();
+    m_bt.clear();
     m_start = 0;
 
     fileToSend->setText(desc.isEmpty() ? file : desc );
+
+    if ( !QCopChannel::isRegistered("QPE/IrDaApplet") )
+    {
+        irdaStatus->setText(tr("not enabled."));
+    }
+    else
+    {
+        QCopEnvelope e1("QPE/IrDaApplet", "enableIrda()");
+        irdaStatus->setText(tr("ready"));
+        sendButton->setEnabled( true );
+    }
+    if ( !QCopChannel::isRegistered("QPE/Bluetooth") )
+    {
+        btStatus->setText(tr("not enabled."));
+    }
+    else
+    {
+        QCopEnvelope e1("QPE/Bluetooth", "enableBluetooth()");
+        btStatus->setText(tr("ready."));
+        sendButton->setEnabled( true );
+    }
     read_receivers();
 }
 
 int SendWidget::addReceiver(const QString& str, const char *icon)
 {
     QListViewItem * item = new QListViewItem( receiverList, 0 );
     item->setText( 0, str );
     item->setPixmap( 1, OResource::loadPixmap( icon ) );
 
-    int id=receivers.count();
-    receivers[id]=item;
+    int id = receivers.count();
+    receivers[id] = item;
     return id;
 }
 
 bool SendWidget::receiverSelected(int id)
 {
     return (bool)(receivers[id]->pixmap(2) != NULL);
@@ -151,15 +173,17 @@ void SendWidget::slotIrTry(unsigned int trI) {
 }
 void SendWidget::slotStartIrda() {
     if ( !m_irDa.count() ) 
         return;
     if ( m_irDaIt == m_irDa.end() || !receiverSelected(m_irDaIt.key())) {
         irdaStatus->setText(tr("complete."));
+        m_irDaIt = m_irDa.begin();
         return;
     }
     setReceiverStatus( m_irDaIt.key(), tr("Start sending") );
+    irdaStatus->setText(tr("sending."));
     m_obex->send( m_file, tr("noaddress") );
 }
 
 void SendWidget::dispatchBt( const QCString& str, const QByteArray& ar ) {
     if ( str == "devices(QStringMap)" ) {
         QDataStream stream( ar, IO_ReadOnly );
@@ -179,25 +203,29 @@ void SendWidget::slotBtSent( bool b) {
 }
 void SendWidget::slotBtTry(unsigned int trI) {
     setReceiverStatus( m_btIt.key(), tr("Try %1").arg( QString::number( trI ) ) );
 }
 void SendWidget::slotStartBt() {
     // skip past unselected receivers
+    if ( !m_bt.count() ) 
+        return;
     while((m_btIt != m_bt.end()) && !receiverSelected(m_btIt.key()))
         ++m_btIt;
     if (m_btIt == m_bt.end() ) {
         btStatus->setText(tr("complete."));
+        m_btIt = m_bt.begin();
         return;
     }
     setReceiverStatus( m_btIt.key(), tr("Start sending") );
+    btStatus->setText(tr("sending."));
     m_btobex->send( m_file, m_btIt.data().second() );
 }
 
 void SendWidget::send_to_receivers() {
-    slotStartIrda();
     slotStartBt();
+    slotStartIrda();
 }
 
 /**
  * Read receivers saved by bluetooth manager
  */
 void SendWidget::read_receivers()
@@ -206,31 +234,17 @@ void SendWidget::read_receivers()
     DeviceHandler handler;
     QValueList<RemoteDevice>::ConstIterator it;
 
     receiverList->clear();
     receivers.clear();
     sendButton->setDisabled( true );
+    btStatus->setText(tr("load."));
+    m_bt.clear();
 
-    if ( !QCopChannel::isRegistered("QPE/IrDaApplet") )
+    if ( QCopChannel::isRegistered("QPE/Bluetooth") )
     {
-        irdaStatus->setText(tr("not enabled."));
-    }
-    else
-    {
-        QCopEnvelope e1("QPE/IrDaApplet", "enableIrda()");
-        irdaStatus->setText(tr("ready"));
-        sendButton->setEnabled( true );
-    }
-    if ( !QCopChannel::isRegistered("QPE/Bluetooth") )
-    {
-        btStatus->setText(tr("not enabled."));
-    }
-    else
-    {
-        QCopEnvelope e1("QPE/Bluetooth", "enableBluetooth()");
-
         devices = handler.load();
         for( it = devices.begin(); it != devices.end() ; ++it )
         {
             int id = addReceiver((*it).name(), "obex/bt.png");
             m_bt.insert(id, Pair((*it).name(), (*it).mac()));
         }
@@ -240,43 +254,37 @@ void SendWidget::read_receivers()
     }
 }
 
 
 void SendWidget::scan_for_receivers()
 {
+    sendButton->setDisabled( true );
     receiverList->clear();
     receivers.clear();
-    sendButton->setDisabled( true );
+    m_irDa.clear();
+    m_bt.clear();
 
-    if ( !QCopChannel::isRegistered("QPE/IrDaApplet") )
-    {
-        irdaStatus->setText(tr("not enabled."));
-    }
-    else
+    if ( QCopChannel::isRegistered("QPE/IrDaApplet") )
     {
-        QCopEnvelope e1("QPE/IrDaApplet", "enableIrda()");
         irdaStatus->setText(tr("searching..."));
         sendButton->setEnabled( true );
         QCopEnvelope e2("QPE/IrDaApplet", "listDevices()");
     }
 
-    if ( !QCopChannel::isRegistered("QPE/Bluetooth") )
+    if ( QCopChannel::isRegistered("QPE/Bluetooth") )
     {
-        btStatus->setText(tr("not enabled."));
-    }
-    else
-    {
-        QCopEnvelope e1("QPE/Bluetooth", "enableBluetooth()");
         btStatus->setText(tr("searching..."));
         sendButton->setEnabled( true );
         QCopEnvelope e3("QPE/Bluetooth", "listDevices()");
     }
 }
 
 void SendWidget::toggle_receiver(QListViewItem* item)
 {
+    if (!item)
+        return;
     // toggle the state of an individual receiver.
     if (item->pixmap(2))
         item->setPixmap(2, QPixmap());
     else
         item->setPixmap(2, OResource::loadPixmap("obex/check.png"));
 }