made kapi saving faster

15 files changed, 132 insertions, 87 deletions

diff --git a/bin/kdepim/WhatsNew.txt b/bin/kdepim/WhatsNew.txt
index 2fd63e7..ff87423 100644
--- a/bin/kdepim/WhatsNew.txt
+++ b/bin/kdepim/WhatsNew.txt
@@ -5,8 +5,22 @@ Info about the changes in new versions of KDE-Pim/Pi
 KO/Pi:
 Fixed that tooltips were not updated after moving an item in agenda view.
+Fixed a bug in sorting start date for recurring events in list view.
 
 KA/Pi:
 All fields search does now actually search all the (possible) fields, 
 not only those listed in the contact list.
+Made is possible to inline a picture in a vcard on the Z.
+This was only possible on the desktop, now is it possible on the Z as well.
+Fixed of missing save settings after filter configuration.
+Made saving of addressbook much faster.
+
+QWhatsThis was not working on the Z ( only black rectangle was shown).
+This is Fixed.
+
+KDE-Sync:
+Now readonly KDE resources are synced as well.
+(They are not changed in KDE itself, of course).
+
+ 
 
 ********** VERSION 1.9.16 ************
diff --git a/kabc/addressee.cpp b/kabc/addressee.cpp
index 3ce733d..568dfc4 100644
--- a/kabc/addressee.cpp
+++ b/kabc/addressee.cpp
@@ -1684,4 +1684,5 @@ void Addressee::dump() const
 {
     return;
+#if 0 
   kdDebug(5700) << "Addressee {" << endl;
 
@@ -1747,4 +1748,5 @@ void Addressee::dump() const
 
   kdDebug(5700) << "}" << endl;
+#endif 
 }
 
diff --git a/kabc/picture.cpp b/kabc/picture.cpp
index 6a34b98..57aa297 100644
--- a/kabc/picture.cpp
+++ b/kabc/picture.cpp
@@ -54,14 +54,40 @@ Picture::~Picture()
 bool Picture::operator==( const Picture &p ) const
 {
-  if ( mIntern != p.mIntern ) return false;
-
+    //qDebug("compare PIC ");
+    if ( mUndefined && p.mUndefined ) {
+        //qDebug("compare PIC true 1 ");
+        return true;
+    }
+    if ( mUndefined || p.mUndefined ) {
+        //qDebug("compare PIC false 1");
+         return false;
+    }
+    // now we should deal with two defined pics!
+    if ( mIntern != p.mIntern ) {
+        //qDebug("compare PIC false 2");
+        return false;
+    }
   if ( mIntern ) {
-    if ( mData != p.mData )
+      //qDebug("mIntern ");
+      if ( mData.isNull() && p.mData.isNull() ) {
+          //qDebug("compare PIC true 2 ");
+          return true;
+      }
+      if ( mData.isNull() || p.mData.isNull() ){
+          //qDebug("compare PIC false 3-1");
+
+          return false;
+      }
+      if ( mData != p.mData ) {
+          //qDebug("compare PIC false 3");
       return false;
+      }
   } else {
-    if ( mUrl != p.mUrl )
+      if ( mUrl != p.mUrl ) {
+          //qDebug("compare PIC false 4");
       return false;
   }
-
+  }
+  //qDebug("compare PIC true ");
   return true;
 }
diff --git a/kabc/vcard/ContentLine.cpp b/kabc/vcard/ContentLine.cpp
index c368172..2f88cde 100644
--- a/kabc/vcard/ContentLine.cpp
+++ b/kabc/vcard/ContentLine.cpp
@@ -274,4 +274,31 @@ ContentLine::_parse()
 ContentLine::_assemble()
 {
+	//strRep_.truncate(0);
+	QString line;
+	if (!group_.isEmpty())
+		line = group_ + '.';
+	line += name_;
+	ParamListIterator it(paramList_);
+	for (; it.current(); ++it)
+		line += ";" + it.current()->asString();
+
+	if (value_ != 0)
+		line += ":" + value_->asString();
+
+	line = line.replace( QRegExp( "\n" ), "\\n" );
+
+	// Fold lines longer than 72 chars
+	const int maxLen = 72;
+	uint cursor = 0;
+    QString cut;
+	while( line.length() > ( cursor + 1 ) * maxLen ) {
+		cut += line.mid( cursor * maxLen, maxLen );
+		cut += "\r\n ";
+		++cursor;
+	}
+	cut += line.mid( cursor * maxLen );
+    strRep_ = cut.latin1();
+    //qDebug("ContentLine::_assemble()\n%s*****", strRep_.data());
+#if 0
 	vDebug("Assemble (argl) - my name is \"" + name_ + "\"");
 	strRep_.truncate(0);
@@ -308,5 +335,6 @@ ContentLine::_assemble()
 	}
 	strRep_ += line.mid( cursor * maxLen );
-    //qDebug("ContentLine::_assemble()\n%s*****", strRep_.data());
+    qDebug("ContentLine::_assemble()\n%s*****", strRep_.data());
+#endif
 }
 
diff --git a/kabc/vcard/VCardv.cpp b/kabc/vcard/VCardv.cpp
index bc80707..49bfe43 100644
--- a/kabc/vcard/VCardv.cpp
+++ b/kabc/vcard/VCardv.cpp
@@ -24,4 +24,7 @@
 #include <qcstring.h>
 #include <qstrlist.h>
+#include <qstringlist.h>
+#include <qstring.h>
+#include <qvaluelist.h>
 
 #include <VCardEntity.h>
@@ -97,28 +100,27 @@ VCard::~VCard()
 VCard::_parse()
 {
-	vDebug("parse() called");
-	QStrList l;
 
-	RTokenise(strRep_, "\r\n", l);
+	QStringList l;
+	QStrList sl;
 
-	if (l.count() < 3) { // Invalid VCARD !
-		vDebug("Invalid vcard");
+	RTokenise(strRep_, "\r\n", sl);
+
+	if (sl.count() < 3) { // Invalid VCARD !
+        //qDebug("invalid vcard ");
 		return;
 	}
-
+    l = QStringList::fromStrList( sl );
 	// Get the first line
-	QCString beginLine = QCString(l.at(0)).stripWhiteSpace();
-
-	vDebug("Begin line == \"" + beginLine + "\"");
+	QString beginLine = l[0].stripWhiteSpace();
 
 	// Remove extra blank lines
-	while (QCString(l.last()).isEmpty())
+	while (l.last().isEmpty())
 		l.remove(l.last());
 
 	// Now we know this is the last line
-	QCString endLine = l.last();
+	QString endLine = l.last();
 
 	// Trash the first and last lines as we have seen them.
-	l.remove(0u);
+	l.remove(l.first());
 	l.remove(l.last());
 
@@ -133,6 +135,6 @@ VCard::_parse()
 	}
 
-	QCString firstPart(beginLine.left(split));
-	QCString valuePart(beginLine.mid(split + 1));
+	QString firstPart(beginLine.left(split));
+	QString valuePart(beginLine.mid(split + 1));
 
 	split = firstPart.find('.');
@@ -143,11 +145,11 @@ VCard::_parse()
 	}
 
-	if (qstrnicmp(firstPart, "BEGIN", 5) != 0) { // No BEGIN !
-		vDebug("No BEGIN");
+	if (firstPart.left(5) != "BEGIN" ) { // No BEGIN !
+		qDebug("no BEGIN in vcard ");
 		return;
 	}
 
-	if (qstrnicmp(valuePart, "VCARD", 5) != 0) { // Not a vcard !
-		vDebug("No VCARD");
+	if (valuePart.left(5) != "VCARD") { // Not a vcard !
+		qDebug("not a VCARD  ");
 		return;
 	}
@@ -160,40 +162,28 @@ VCard::_parse()
 	// Handle folded lines.
 
-	QStrList refolded;
+	QStringList refolded;
 
-	QStrListIterator it(l);
 
-	QCString cur;
+    QStringList::Iterator it =  l.begin(); 
 
-	for (; it.current(); ++it) {
-
-		cur = it.current();
+	QString cur;
 
+	for (; it != l.end(); ++it) {
+		cur = (*it);
 		++it;
-
-		while (
-			it.current()		&&
-			it.current()[0] == ' '	&&
-			strlen(it.current()) != 1)
-		{
-			cur += it.current() + 1;
+		while ( it!= l.end() && (*it).at(0) == ' '	&& (*it).length()!= 1) {
+			cur += (*it) ;
 			++it;
 		}
-
 		--it;
-
 		refolded.append(cur);
 	}
-
-	QStrListIterator it2(refolded);
-
-	for (; it2.current(); ++it2) {
-	  vDebug("New contentline using \"" + QCString(it2.current()) + "\"");
-		ContentLine * cl = new ContentLine(it2.current());
-
+    QStringList::Iterator it2 = refolded.begin();
+	for (; it2 != refolded.end(); ++it2) {
+		ContentLine * cl = new ContentLine(QCString((*it2).latin1()));
 		cl->parse();
 		if (cl->value() == 0)
 		{
-		    qDebug("Content line could not be parsed. Discarded: %s", it2.current());
+		    qDebug("Content line could not be parsed. Discarded: %s", (*it2).latin1());
 		    delete cl;
 		}
@@ -205,4 +195,8 @@ VCard::_parse()
 	// LAST LINE
 
+
+        // LR: sorry, but the remaining code in this method makes no sense
+
+#if 0
 	split = endLine.find(':');
 
@@ -225,4 +219,5 @@ VCard::_parse()
 	if (qstricmp(valuePart, "VCARD") != 0) // Not a vcard !
 		return;
+#endif
 }
 
diff --git a/kaddressbook/details/detailsviewcontainer.cpp b/kaddressbook/details/detailsviewcontainer.cpp
index ceffc77..229cce0 100644
--- a/kaddressbook/details/detailsviewcontainer.cpp
+++ b/kaddressbook/details/detailsviewcontainer.cpp
@@ -130,20 +130,11 @@ void ViewContainer::slotStyleSelected( int index )
   mCurrentLook->setAddressee( addr );
 }
-void ViewContainer::refreshView()
-{
-    if ( mCurrentLook  ) {
-        mCurrentLook->setAddressee( mCurrentAddressee );
-    }
-}
 
 void ViewContainer::setAddressee( const KABC::Addressee& addressee )
 {
   if ( mCurrentLook != 0 ) {
-    if ( addressee == mCurrentAddressee )
-      return;
-    else {
       mCurrentAddressee = addressee;
       mCurrentLook->setAddressee( mCurrentAddressee );
-    }
+    
   }
 }
diff --git a/kaddressbook/details/detailsviewcontainer.h b/kaddressbook/details/detailsviewcontainer.h
index b561d12..667f0cb 100644
--- a/kaddressbook/details/detailsviewcontainer.h
+++ b/kaddressbook/details/detailsviewcontainer.h
@@ -46,5 +46,4 @@ class ViewContainer : public QWidget
      */
     KABBasicLook *currentLook();
-    void refreshView();
     /**
       Return the contact currently displayed.
diff --git a/kaddressbook/imagewidget.cpp b/kaddressbook/imagewidget.cpp
index 49d456b..48370e3 100644
--- a/kaddressbook/imagewidget.cpp
+++ b/kaddressbook/imagewidget.cpp
@@ -132,8 +132,4 @@ ImageWidget::ImageWidget( QWidget *parent, const char *name )
 #endif //KAB_EMBEDDED  
 
-#ifndef DESKTOP_VERSION
-    mUsePhotoUrl->setChecked( true );
-    mUseLogoUrl->setChecked( true );
-#endif
 }
 
@@ -261,10 +257,9 @@ void ImageWidget::updateGUI()
   KURLRequester *ptr = (KURLRequester*)sender();
 
-#ifdef DESKTOP_VERSION
+
   if ( ptr == mPhotoUrl )
     mUsePhotoUrl->setEnabled( true );
   else if ( ptr == mLogoUrl )
     mUseLogoUrl->setEnabled( true );
-#endif
 }
 
diff --git a/kaddressbook/kabcore.cpp b/kaddressbook/kabcore.cpp
index f6bdda4..e14e579 100644
--- a/kaddressbook/kabcore.cpp
+++ b/kaddressbook/kabcore.cpp
@@ -1127,7 +1127,5 @@ void KABCore::setModifiedWOrefresh()
     mModified = true;
     mActionSave->setEnabled( mModified );
-#ifdef DESKTOP_VERSION
-  mDetails->refreshView();
-#endif
+
 
 }
@@ -1141,5 +1139,4 @@ void KABCore::setModified( bool modified )
 
   mViewManager->refreshView();
-  mDetails->refreshView();
 
 }
@@ -1167,5 +1164,5 @@ void KABCore::contactModified( const KABC::Addressee &addr )
   UndoStack::instance()->push( command );
   RedoStack::instance()->clear();
-
+  mDetails->setAddressee( addr );
   setModified( true );
 }
@@ -1634,5 +1631,4 @@ void KABCore::addressBookChanged()
   }
   mViewManager->refreshView();
-//  mDetails->refreshView();
 
 
@@ -2290,5 +2286,4 @@ void KABCore::manageCategories( )
         setModified( true );
         mViewManager->refreshView();
-        mDetails->refreshView();
         message( i18n("Removing categories done!"));
     }
diff --git a/kaddressbook/kabcore.h b/kaddressbook/kabcore.h
index c7c12ff..85ffbdb 100644
--- a/kaddressbook/kabcore.h
+++ b/kaddressbook/kabcore.h
@@ -96,8 +96,5 @@ class KABCore : public QWidget, public KSyncInterface
     void restoreSettings();
 
-    /**
-      Saves the global settings.
-     */
-    void saveSettings();
+  
 
     /**
@@ -359,5 +356,7 @@ class KABCore : public QWidget, public KSyncInterface
     void setDetailsVisible( bool visible );
     void setDetailsToState();
-    //    void slotSyncMenu( int );
+   
+    void saveSettings();
+
   private slots:
     void updateToolBar();
diff --git a/kaddressbook/viewmanager.cpp b/kaddressbook/viewmanager.cpp
index 33bef5a..59bddd9 100644
--- a/kaddressbook/viewmanager.cpp
+++ b/kaddressbook/viewmanager.cpp
@@ -606,4 +606,5 @@ void ViewManager::configureFilters()
       cw = 0; 
   mActionSelectFilter->setComboWidth( cw );
+  saveSettings();
 }
 
diff --git a/kde2file/abdump/main.cpp b/kde2file/abdump/main.cpp
index 1ee64f5..9ad78e5 100644
--- a/kde2file/abdump/main.cpp
+++ b/kde2file/abdump/main.cpp
@@ -105,6 +105,4 @@ int main( int argc, char *argv[] )
             if ( (*it).isEmpty() || ! (*it).resource() )
                 continue;
-            if ( (*it).resource()->readOnly()   )
-                continue;
             KABC::Addressee a = ( *it );
             QString vcard = converter.createVCard( a );
diff --git a/kde2file/caldump/main.cpp b/kde2file/caldump/main.cpp
index be1735b..08ccafb 100644
--- a/kde2file/caldump/main.cpp
+++ b/kde2file/caldump/main.cpp
@@ -130,5 +130,5 @@ int main( int argc, char *argv[] )
         for( it = allInc.begin(); it != allInc.end(); ++it ) {
             ResourceCalendar * re = calendarResource->resource( (*it) );
-            if ( re && !re->readOnly() ) {
+            if ( re && /*!re->readOnly() now readonly syncing possible */) {
                 ++num;
                 Incidence* cl = (*it)->clone();
diff --git a/korganizer/kolistview.cpp b/korganizer/kolistview.cpp
index 3d4acb7..f8bfc8b 100644
--- a/korganizer/kolistview.cpp
+++ b/korganizer/kolistview.cpp
@@ -78,10 +78,12 @@ bool ListItemVisitor::visit(Event *e)
     bool ok = false;
     QString start, end;
+    QDate ds, de;
     if ( e->doesRecur() ) {
-        QDate d = e->getNextOccurence( QDateTime( mDate, QTime(0,0,0)), &ok ).date();
+        ds = e->getNextOccurence( QDateTime( mDate, QTime(0,0,0)), &ok ).date();
         if ( ok ) {
             int days = e->dtStart().date().daysTo(e->dtEnd().date() );
-            start = KGlobal::locale()->formatDate(d,true);
-            end = KGlobal::locale()->formatDate(d.addDays( days),true);
+            start = KGlobal::locale()->formatDate(ds,true);
+            de = ds.addDays( days);
+            end = KGlobal::locale()->formatDate(de,true);
         }
 
@@ -90,4 +92,6 @@ bool ListItemVisitor::visit(Event *e)
         start =e->dtStartDateStr();
         end = e->dtEndDateStr();
+        ds = e->dtStart().date();
+        de = e->dtEnd().date();
     }
     mItem->setText(0,e->summary());
@@ -104,12 +108,10 @@ bool ListItemVisitor::visit(Event *e)
 
     QString key;
-    QDate d = e->dtStart().date();
     QTime t = e->doesFloat() ? QTime(0,0) : e->dtStart().time();
-    key.sprintf("%04d%02d%02d%02d%02d",d.year(),d.month(),d.day(),t.hour(),t.minute());
+    key.sprintf("%04d%02d%02d%02d%02d",ds.year(),ds.month(),ds.day(),t.hour(),t.minute());
     mItem->setSortKey(1,key);
 
-    d = e->dtEnd().date();
     t = e->doesFloat() ? QTime(0,0) : e->dtEnd().time();
-    key.sprintf("%04d%02d%02d%02d%02d",d.year(),d.month(),d.day(),t.hour(),t.minute());
+    key.sprintf("%04d%02d%02d%02d%02d",de.year(),de.month(),de.day(),t.hour(),t.minute());
     mItem->setSortKey(3,key);
 
diff --git a/version b/version
index 65eb8d1..d8563ed 100644
--- a/version
+++ b/version
@@ -1 +1 @@
-version = "1.9.16";
+version = "1.9.17";