some reworks

2 files changed, 45 insertions, 50 deletions

diff --git a/noncore/net/mail/taskbarapplet/mailapplet.cpp b/noncore/net/mail/taskbarapplet/mailapplet.cpp
index ce1e7ac..b777ecf 100644
--- a/noncore/net/mail/taskbarapplet/mailapplet.cpp
+++ b/noncore/net/mail/taskbarapplet/mailapplet.cpp
@@ -21,9 +21,6 @@ MailApplet::MailApplet( QWidget *parent )
 : QLabel( parent ) {
 
-    m_config = new Config( "mail" );
-    m_config->setGroup( "Applet" );
-
-    //setFixedWidth( AppLnk::smallIconSize() );
-    setFixedHeight( AppLnk::smallIconSize() );
+    setFixedHeight(AppLnk::smallIconSize());
+    setMinimumWidth(AppLnk::smallIconSize());
 
     hide();
@@ -39,9 +36,5 @@ MailApplet::MailApplet( QWidget *parent )
     sigaction(SIGPIPE,&blocking_action,&temp_action);
 
-    if ( !m_config->readBoolEntry( "Disabled", false ) ) {
-        // delay 5 sec until the whole mail backend gets started .-)
-        QTimer::singleShot( 5000, this, SLOT( startup() ) );
-    }
-    repaint( true );
+    QTimer::singleShot( 5000, this, SLOT( startup() ) );
 }
 
@@ -50,6 +43,4 @@ MailApplet::~MailApplet() {
     if ( m_statusMail )
         delete m_statusMail;
-    if ( m_config )
-        delete m_config;
 }
 
@@ -73,14 +64,12 @@ void MailApplet::mouseReleaseEvent( QMouseEvent* e ) {
 }
 
-void MailApplet::slotClicked() {
-    QCopEnvelope e( "QPE/System", "execute(QString)" );
-    e << QString( "opiemail" );
-
-    ODevice *device = ODevice::inst();
-    if ( !device-> ledList().isEmpty() ) {
-        OLed led = ( device->ledList().contains( Led_Mail ) ) ? Led_Mail : device->ledList()[0];
-
-        device->setLedState( led, Led_Off );
+void MailApplet::slotClicked()
+{
+    {
+        QCopEnvelope e( "QPE/System", "execute(QString)" );
+        e << QString( "opiemail" );
     }
+
+    ledOnOff(false);
     if (m_statusMail)
         m_statusMail->reset_status();
@@ -102,17 +91,34 @@ void MailApplet::startup()
 }
 
+void MailApplet::ledOnOff(bool how)
+{
+    ODevice *device = ODevice::inst();
+    if ( !device->ledList().isEmpty() ) {
+        OLed led = ( device->ledList().contains( Led_Mail ) ) ? Led_Mail : device->ledList()[0];
+        device->setLedState( led,  (how?(device->ledStateList( led ).contains( Led_BlinkSlow )?Led_BlinkSlow:Led_On):Led_Off) );
+    }
+}
+
 void MailApplet::slotCheck() {
     // Check wether the check interval has been changed.
     odebug << "MailApplet::slotCheck()" << oendl;
-    int newIntervalMs = m_config->readNumEntry( "CheckEvery", 5 ) * 60000;
+    Config m_config( "mail" );
+    m_config.setGroup( "Applet" );
+
+    int newIntervalMs = m_config.readNumEntry( "CheckEvery", 5 ) * 60000;
     if ( newIntervalMs != m_intervalMs ) {
         m_intervalTimer->changeInterval( newIntervalMs );
         m_intervalMs = newIntervalMs;
     }
-
+    if ( m_config.readBoolEntry( "Disabled", false ) ) {
+        hide();
+        ledOnOff(false);
+        odebug << "MailApplet::slotCheck() - disabled" << oendl;
+        return;
+    }
     if (m_statusMail == 0) {
+        odebug << "MailApplet::slotCheck() - no mailhandle" << oendl;
         return;
     }
-
     folderStat stat;
     m_statusMail->check_current_stat( stat );
@@ -124,36 +130,25 @@ void MailApplet::slotCheck() {
             show();
         if (newMailsOld != m_newMails) {
-            ODevice *device = ODevice::inst();
-            if ( m_config->readBoolEntry( "BlinkLed", true ) ) {
-                if ( !device->ledList().isEmpty() ) {
-                    OLed led = ( device->ledList().contains( Led_Mail ) ) ? Led_Mail : device->ledList()[0];
-                    device->setLedState( led, device->ledStateList( led ).contains( Led_BlinkSlow ) ? Led_BlinkSlow : Led_On );
-                }
+            if ( m_config.readBoolEntry( "BlinkLed", true ) ) {
+                ledOnOff(true);
             }
-            if ( m_config->readBoolEntry( "PlaySound", false ) )
-                device->playAlarmSound();
-        }
-        Config cfg( "mail" );
-        cfg.setGroup( "Status" );
-        cfg.writeEntry( "newMails", m_newMails );
-        {
-            QCopEnvelope env( "QPE/Pim", "newMails(int)" );
-            env <<  m_newMails;
+            if ( m_config.readBoolEntry( "PlaySound", false ) )
+                ODevice::inst()->playAlarmSound();
+            m_config.setGroup( "Status" );
+            m_config.writeEntry( "newMails", m_newMails );
+            {
+                QCopEnvelope env( "QPE/Pim", "newMails(int)" );
+                env <<  m_newMails;
+            }
+            setText(QString::number( m_newMails ));
         }
-        setText(QString::number( m_newMails ));
-//        repaint( true );
     } else {
         ODevice *device = ODevice::inst();
         if ( !isHidden() )
             hide();
-        if ( !device->ledList().isEmpty() ) {
-            OLed led = ( device->ledList().contains( Led_Mail ) ) ? Led_Mail : device->ledList()[0];
-            device->setLedState( led, Led_Off );
-        }
-
         if ( newMailsOld != m_newMails ) {
-            Config cfg( "mail" );
-            cfg.setGroup( "Status" );
-            cfg.writeEntry( "newMails", m_newMails );
+            ledOnOff(false);
+            m_config.setGroup( "Status" );
+            m_config.writeEntry( "newMails", m_newMails );
             QCopEnvelope env( "QPE/Pim", "newMails(int)" );
             env <<  m_newMails;
diff --git a/noncore/net/mail/taskbarapplet/mailapplet.h b/noncore/net/mail/taskbarapplet/mailapplet.h
index f28f274..f46bf19 100644
--- a/noncore/net/mail/taskbarapplet/mailapplet.h
+++ b/noncore/net/mail/taskbarapplet/mailapplet.h
@@ -22,4 +22,5 @@ protected:
     void mouseReleaseEvent( QMouseEvent* );
     void paintEvent( QPaintEvent* );
+    static void ledOnOff(bool);
 
 protected slots:
@@ -28,5 +29,4 @@ protected slots:
     void slotClicked();
 private:
-    Config *m_config;
     QTimer *m_intervalTimer;
     int m_intervalMs;