Fix khelpcenter service group entry appearing when no child service items have documentation

This relates to Bug 1968
Fix khelpcenter services showing tree expansion hints

khelpcenter/navigator.cpp

@@ -225,8 +225,9 @@ void Navigator::insertParentAppDocs( const TQString &name, NavigatorItem *topIte
   KServiceGroup::List::ConstIterator end = entries.end();
   for ( ; it != end; ++it ) {
     TQString desktopFile = ( *it )->entryPath();
-    if ( TQDir::isRelativePath( desktopFile ) )
+    if ( TQDir::isRelativePath( desktopFile ) ) {
         desktopFile = locate( "apps", desktopFile );
+    }
     createItemFromDesktopFile( topItem, desktopFile );
   }
 }

khelpcenter/navigatorappitem.cpp

@@ -78,20 +78,21 @@ void NavigatorAppItem::setOpen(bool open)
   TQListViewItem::setOpen(open); 
 }
 
-void NavigatorAppItem::populate( bool recursive )
+bool NavigatorAppItem::populate( bool recursive )
 {
-  if ( mPopulated ) return;
+  bool entriesAdded = false;
+
+  if ( mPopulated ) return false;
 
   KServiceGroup::Ptr root = KServiceGroup::group(mRelpath);
   if ( !root ) {
     kdWarning() << "No Service groups\n";
-    return;
+    return false;
   }
   KServiceGroup::List list = root->entries();
 
 
-  for ( KServiceGroup::List::ConstIterator it = list.begin();
+  for ( KServiceGroup::List::ConstIterator it = list.begin(); it != list.end(); ++it )
-        it != list.end(); ++it )
   {
     KSycocaEntry * e = *it;
     KService::Ptr s;
@@ -108,20 +109,41 @@ void NavigatorAppItem::populate( bool recursive )
           DocEntry *entry = new DocEntry( s->name(), url, s->icon() );
           item = new NavigatorItem( entry, this );
           item->setAutoDeleteDocEntry( true );
-          item->setExpandable( true );
+          item->setExpandable( false );
+          entriesAdded = true;
         }
         break;
       }
       case KST_KServiceGroup:
       {
         g = static_cast<KServiceGroup*>(e);
-        if ( ( g->childCount() == 0 ) || g->name().startsWith( "." ) )
+        if ( ( g->childCount() == 0 ) || g->name().startsWith( "." ) ) {
           continue;
+        }
+        KServiceGroup::List entryList = g->entries(false, true, false, false);
+        if (entryList.count() > 0) {
+          int entryCount = 0;
+          for( KServiceGroup::List::ConstIterator it2 = entryList.begin(); it2 != entryList.end(); it2++)
+          {
+            KSycocaEntry *p = (*it2);
+            if (p->isType(KST_KService))
+            {
+              KService *s = static_cast<KService *>(p);
+              url = documentationURL( s );
+              if ( !url.isEmpty() ){
+                entryCount++;
+              }
+            }
+          }
+          if (entryCount > 0) {
             DocEntry *entry = new DocEntry( g->caption(), "", g->icon() );
             NavigatorAppItem *appItem;
             appItem = new NavigatorAppItem( entry, this, g->relPath() );
             appItem->setAutoDeleteDocEntry( true );
             if ( recursive ) appItem->populate( recursive );
+            entriesAdded = true;
+          }
+        }
         break;
       }
       default:
@@ -130,6 +152,8 @@ void NavigatorAppItem::populate( bool recursive )
   }
   sortChildItems( 0, true /* ascending */ );
   mPopulated = true;
+
+  return entriesAdded;
 }
 
 TQString NavigatorAppItem::documentationURL( KService *s )

khelpcenter/navigatorappitem.h

@@ -42,7 +42,7 @@ class NavigatorAppItem : public NavigatorItem
     void setRelpath( const TQString & );
 
     virtual void setOpen(bool);
-    void populate( bool recursive = false );
+    bool populate( bool recursive = false );
 
   protected:
     TQString documentationURL( KService *s );