Do not show in composer legend unused layers if filtering by map is enabled

wonder-sk · wonder-sk · commit 4ac94f227553 · 2014-10-24T16:25:50.000+07:00
This resolves #11457 and partially #11293
diff --git a/src/core/qgslegendrenderer.cpp b/src/core/qgslegendrenderer.cpp
@@ -208,6 +208,12 @@ QList<QgsLegendRenderer::Atom> QgsLegendRenderer::createAtomList( QgsLayerTreeGr
 
       QList<QgsLayerTreeModelLegendNode*> legendNodes = mLegendModel->layerLegendNodes( nodeLayer );
 
+      // workaround for the issue that "filtering by map" does not remove layer nodes that have no symbols present
+      // on the map. We explicitly skip such layers here. In future ideally that should be handled directly
+      // in the layer tree model
+      if ( legendNodes.isEmpty() && mLegendModel->legendFilterByMap() )
+        continue;
+
       QList<Atom> layerAtoms;
 
       for ( int j = 0; j < legendNodes.count(); j++ )
