Fixed incorrect update of KNote "last modified" field caused by the lauch of KNotes. This resolves issue #38.

Signed-off-by: Michele Calgaro <michele.calgaro@yahoo.it>

knotes/knote.cpp

@@ -515,13 +515,31 @@ TQColor KNote::bgColor() const
 
 void KNote::setColor( const TQColor& fg, const TQColor& bg )
 {
-    m_journal->setCustomProperty( "KNotes", "FgColor", fg.name() );
+    bool updateJournal = false;
-    m_journal->setCustomProperty( "KNotes", "BgColor", bg.name() );
+    TQString journalFg = m_journal->customProperty( "KNotes", "FgColor" );
+    if ( journalFg.isEmpty() || journalFg != fg.name() )
+    {
+        m_journal->setCustomProperty( "KNotes", "FgColor", fg.name() );
+        updateJournal = true;
+    }
+    TQString journalbg = m_journal->customProperty( "KNotes", "BgColor" );
+    if ( journalbg.isEmpty() || journalbg != bg.name() )
+    {
+        m_journal->setCustomProperty( "KNotes", "BgColor", bg.name() );
+        updateJournal = true;
+    }
+
     m_config->setFgColor( fg );
     m_config->setBgColor( bg );
 
-    m_journal->updated();  // because setCustomProperty() doesn't call it!!
+    if (updateJournal)
-    emit sigDataChanged(noteId());
+    {
+        // Only update the journal if new configuration was really used.
+        // This is necessary because setColor() is called also when loading a note from an .ics
+        // file and we do not want to inadvertantly update the last modified field each time.
+        m_journal->updated();  // because setCustomProperty() doesn't call it!!
+        emit sigDataChanged(noteId());
+    }
     m_config->writeConfig();
 
     TQPalette newpalette = palette();

kontact/plugins/knotes/knotes_part_p.h

@@ -71,7 +71,7 @@ class KNotesIconViewItem : public TDEIconViewItem
       TQPixmap icon = TDEGlobal::iconLoader()->loadIcon( "knotes", TDEIcon::Desktop );
       icon = effect.apply( icon, TDEIconEffect::Colorize, 1, color, false );
       setPixmap( icon );
-      setText( journal->summary() );
+      TDEIconViewItem::setText( journal->summary() );
     }
 
     KCal::Journal *journal()