some reworks

2 files changed, 37 insertions, 42 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
@@ -22,7 +22,4 @@ MailApplet::MailApplet( QWidget *parent )
 
-    m_config = new Config( "mail" );
-    m_config->setGroup( "Applet" );
-
-    //setFixedWidth( AppLnk::smallIconSize() );
     setFixedHeight( AppLnk::smallIconSize() );
+    setMinimumWidth(AppLnk::smallIconSize());
 
@@ -40,8 +37,4 @@ MailApplet::MailApplet( QWidget *parent )
 
-    if ( !m_config->readBoolEntry( "Disabled", false ) ) {
-        // delay 5 sec until the whole mail backend gets started .-)
         QTimer::singleShot( 5000, this, SLOT( startup() ) );
     }
-    repaint( true );
-}
 
@@ -51,4 +44,2 @@ MailApplet::~MailApplet() {
         delete m_statusMail;
-    if ( m_config )
-        delete m_config;
 }
@@ -74,12 +65,10 @@ void MailApplet::mouseReleaseEvent( QMouseEvent* e ) {
 
-void MailApplet::slotClicked() {
+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 );
     }
+
+    ledOnOff(false);
     if (m_statusMail)
@@ -103,2 +92,11 @@ 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() {
@@ -106,3 +104,6 @@ void MailApplet::slotCheck() {
     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 ) {
@@ -111,7 +112,12 @@ void MailApplet::slotCheck() {
     }
-
+    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;
@@ -125,15 +131,9 @@ void MailApplet::slotCheck() {
         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 );
+            if ( m_config.readBoolEntry( "PlaySound", false ) )
+                ODevice::inst()->playAlarmSound();
+            m_config.setGroup( "Status" );
+            m_config.writeEntry( "newMails", m_newMails );
         {
@@ -143,3 +143,3 @@ void MailApplet::slotCheck() {
         setText(QString::number( m_newMails ));
-//        repaint( true );
+        }
     } else {
@@ -148,11 +148,6 @@ void MailApplet::slotCheck() {
             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)" );
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
@@ -23,2 +23,3 @@ protected:
     void paintEvent( QPaintEvent* );
+    static void ledOnOff(bool);
 
@@ -29,3 +30,2 @@ protected slots:
 private:
-    Config *m_config;
     QTimer *m_intervalTimer;