Stop Qt complaining that showLegendLayerFileGroups(bool) has an incom…

…patible argument list to triggered() and therefore not connecting the signal to the slot. The "Show file groups" option in the legend right-click menu should now work better (even properly?).

git-svn-id: http://svn.osgeo.org/qgis/trunk@4887 c8812cc2-4d05-0410-92ff-de0c093fc19c

src/legend/qgslegend.cpp

@@ -412,7 +412,7 @@ void QgsLegend::handleRightClickEvent(QTreeWidgetItem* item, const QPoint& posit
   theMenu.addAction(QIcon(QPixmap(iconsPath+QString("/folder_new.png"))), tr("&Add group"), this, SLOT(addGroup()));
   theMenu.addAction(QIcon(QPixmap(iconsPath+QString("/mActionExpandTree.png"))), tr("&Expand all"), this, SLOT(expandAll()));
   theMenu.addAction(QIcon(QPixmap(iconsPath+QString("/mActionCollapseTree.png"))), tr("&Collapse all"), this, SLOT(collapseAll()));
-  QAction* showFileGroupsAction = theMenu.addAction(tr("Show file groups"), this, SLOT(showLegendLayerFileGroups(bool)));
+  QAction* showFileGroupsAction = theMenu.addAction(tr("Show file groups"), this, SLOT(showLegendLayerFileGroups()));
   showFileGroupsAction->setCheckable(true);
   showFileGroupsAction->blockSignals(true);
   showFileGroupsAction->setChecked(mShowLegendLayerFiles);
@@ -1325,18 +1325,20 @@ void QgsLegend::makeToTopLevelItem()
     }
 }
 
-void QgsLegend::showLegendLayerFileGroups(bool show)
+void QgsLegend::showLegendLayerFileGroups()
 {
+  // Toggle the boolean associated with the checkbox
+  mShowLegendLayerFiles = !mShowLegendLayerFiles;
+
   QgsLegendLayerFileGroup* theFileGroup = 0;
   QTreeWidgetItem* theItem = firstItem();
   do
     {
       theFileGroup = dynamic_cast<QgsLegendLayerFileGroup*>(theItem);
       if(theFileGroup)
 	{
-	  setItemHidden(theFileGroup, !show);
+	  setItemHidden(theFileGroup, !mShowLegendLayerFiles);
 	}
     }
   while(theItem = nextItem(theItem));
-  mShowLegendLayerFiles = show;
 }

src/legend/qgslegend.h

@@ -240,7 +240,7 @@ this item may be moved back to the original position with resetToInitialPosition
   /**Removes the current item and inserts it as a toplevel item at the end of the legend*/
   void makeToTopLevelItem();
   /**Show/ Hide the legend layer file groups*/
-  void showLegendLayerFileGroups(bool show);
+  void showLegendLayerFileGroups();
 private:
 
   /**Pointer to QGisApp, needed for signal/slot reasons*/