Add some comments for checkParameters function

also set const back
qgis · Sep 18, 2019 · 55f89db · 55f89db
1 parent 5b33600
commit 55f89db
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/server/services/wms/qgswmsgetlegendgraphics.cpp b/src/server/services/wms/qgswmsgetlegendgraphics.cpp
@@ -171,7 +171,7 @@ namespace QgsWms
   QgsLayerTreeModel *legendModel( const QgsWmsRenderContext &context, QgsLayerTree &tree )
   {
 
-    QgsWmsParameters parameters = context.parameters();
+    const QgsWmsParameters parameters = context.parameters();
     std::unique_ptr<QgsLayerTreeModel> model( new QgsLayerTreeModel( &tree ) );
 
     if ( context.scaleDenominator() > 0 )

diff --git a/src/server/services/wms/qgswmsgetlegendgraphics.h b/src/server/services/wms/qgswmsgetlegendgraphics.h
@@ -32,6 +32,10 @@ namespace QgsWms
                                const QString &version, const QgsServerRequest &request,
                                QgsServerResponse &response );
 
+  /**
+   * checkParameters checks request \a parameters and sets HEIGHT and WIDTH to default values
+   * in case BBOX is specified for contextual legend and HEIGHT or WIDTH are not.
+   */
   void checkParameters( QgsWmsParameters &parameters );
 
   QgsLayerTreeModel *legendModel( const QgsWmsRenderContext &context, QgsLayerTree &tree );