[Server] WMS GetPrint group layers: use const list

qgis · May 1, 2020 · fe078c8 · fe078c8
1 parent 815b18f
commit fe078c8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/src/server/services/wms/qgswmsrendercontext.cpp b/src/server/services/wms/qgswmsrendercontext.cpp
@@ -289,15 +289,7 @@ bool QgsWmsRenderContext::isValidLayer( const QString &nickname ) const
 
 QList<QgsMapLayer *> QgsWmsRenderContext::layersFromGroup( const QString &nickname ) const
 {
-  QList<QgsMapLayer *> layers;
-  if ( mLayerGroups.contains( nickname ) )
-  {
-    for ( QgsMapLayer *layer : mLayerGroups[nickname] )
-    {
-      layers.append( layer );
-    }
-  }
-  return layers;
+  return mLayerGroups.value( nickname );
 }
 
 bool QgsWmsRenderContext::isValidGroup( const QString &name ) const

diff --git a/src/server/services/wms/qgswmsrenderer.cpp b/src/server/services/wms/qgswmsrenderer.cpp
@@ -620,7 +620,8 @@ namespace QgsWms
             {
               QList<QgsMapLayer *> layersFromGroup;
 
-              for ( QgsMapLayer *layerFromGroup : mContext.layersFromGroup( layer.mNickname ) )
+              const QList<QgsMapLayer *> cLayersFromGroup = mContext.layersFromGroup( layer.mNickname );
+              for ( QgsMapLayer *layerFromGroup : cLayersFromGroup )
               {
 
                 if ( ! layerFromGroup )