[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/kdepim/enterprise/kdepim/libkdepim
From:       Thomas McGuire <mcguire () kde ! org>
Date:       2010-05-28 14:19:02
Message-ID: 20100528141902.CB964AC8C1 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1131606 by tmcguire:

Fix parsing of dist list entries without a UID, which is the
case for all dist list entries on Kolab dist lists.

Now I can see dist list members for dist lists saved on
Kolab again.

How did that ever work??

MERGE: trunk? Needs to be checked


 M  +3 -5      distributionlist.cpp  


--- branches/kdepim/enterprise/kdepim/libkdepim/distributionlist.cpp #1131605:1131606
@@ -37,15 +37,13 @@
     if ( (*it).isEmpty() )
       continue;
     // parse "uid,email"
-    QStringList helpList = QStringList::split( ',', (*it) );
+    QStringList helpList = QStringList::split( ',', (*it), true );
     Q_ASSERT( !helpList.isEmpty() );
     if ( helpList.isEmpty() )
       continue;
-    const QString uid = helpList.first();
-    QString email;
     Q_ASSERT( helpList.count() < 3 ); // 1 or 2 items, but not more
-    if ( helpList.count() == 2 )
-      email = helpList.last();
+    const QString uid = helpList.first();
+    const QString email = helpList.last();
     res.append( qMakePair( uid, email ) );
   }
   return res;
[prev in list] [next in list] [prev in thread] [next in thread]