Merge pull request #51 from mbrasser-jolla/master

[libcontacts] Name grouper fixes.
sailfishos · Oct 22, 2013 · 6aa4886 · 6aa4886
2 parents 25a83ac + 7faaae0
commit 6aa4886
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/src/seasidecache.cpp b/src/seasidecache.cpp
@@ -583,13 +583,9 @@ void SeasideCache::setNameGrouper(SeasideNameGrouper *grouper)
         new SeasideCache;
     instancePtr->m_nameGrouper.reset(grouper);
 
-    allContactNameGroups = getAllContactNameGroups();
-    QStringList groups = instancePtr->m_nameGrouper->allNameGroups();
-    for (int i = groups.count() - 1; i > -1; --i) {
-        const QString &group = groups.at(i);
-        if (!allContactNameGroups.contains(group))
-            allContactNameGroups.prepend(group);
-    }
+    allContactNameGroups = instancePtr->m_nameGrouper->allNameGroups();
+    if (!allContactNameGroups.contains(QLatin1String("#")))
+        allContactNameGroups << QLatin1String("#");
 }
 
 QString SeasideCache::nameGroup(const CacheItem *cacheItem)
@@ -607,7 +603,7 @@ QString SeasideCache::determineNameGroup(const CacheItem *cacheItem)
 
     if (!instancePtr->m_nameGrouper.isNull()) {
         QString group = instancePtr->m_nameGrouper->nameGroupForContact(cacheItem->contact, instancePtr->m_groupProperty);
-        if (!group.isNull()) {
+        if (!group.isNull() && allContactNameGroups.contains(group)) {
             return group;
         }
     }