Consider number of layers in a project can be higher than 100 for caching

Author: mhugent

src/mapserver/qgsconfigcache.cpp

@@ -101,6 +101,12 @@ QgsWMSConfigParser* QgsConfigCache::wmsConfiguration( const QString& filePath, c
     p = new QgsWMSProjectParser( doc, filePath );
   }
 
+  int numberOfLayers = p->nLayers();
+  if ( numberOfLayers > 100 )
+  {
+    mWMSConfigCache.setMaxCost( numberOfLayers );
+  }
+
   mWMSConfigCache.insert( filePath, p );
   mFileSystemWatcher.addPath( filePath );
   return p;

src/mapserver/qgssldconfigparser.cpp

@@ -702,6 +702,20 @@ QList< QPair< QString, QgsLayerCoordinateTransform > > QgsSLDConfigParser::layer
   return QList< QPair< QString, QgsLayerCoordinateTransform > >();
 }
 
+int QgsSLDConfigParser::nLayers() const
+{
+  if ( mXMLDoc )
+  {
+    QDomNode sldNode = mXMLDoc->documentElement();
+    if ( !sldNode.isNull() )
+    {
+      QDomNodeList layerNodeList = sldNode.toElement().elementsByTagName( "UserLayer" );
+      return layerNodeList.size();
+    }
+  }
+  return 0;
+}
+
 QList<QDomElement> QgsSLDConfigParser::findNamedLayerElements( const QString& layerName ) const
 {
   QList<QDomElement> resultList;

src/mapserver/qgssldconfigparser.h

@@ -114,6 +114,8 @@ class QgsSLDConfigParser: public QgsWMSConfigParser
 
     QList< QPair< QString, QgsLayerCoordinateTransform > > layerCoordinateTransforms() const;
 
+    int nLayers() const;
+
   private:
 
     /**SLD as dom document*/

src/mapserver/qgswmsconfigparser.h

@@ -105,6 +105,9 @@ class QgsWMSConfigParser
     virtual void addExternalGMLData( const QString& layerName, QDomDocument* gmlDoc ) = 0;
 
     virtual QList< QPair< QString, QgsLayerCoordinateTransform > > layerCoordinateTransforms() const = 0;
+
+    virtual int nLayers() const = 0;
+
 #if 0
     /**List of GML datasets passed outside SLD (e.g. in a SOAP request). Key of the map is the layer name*/
     QMap<QString, QDomDocument*> mExternalGMLDatasets;

src/mapserver/qgswmsprojectparser.cpp

@@ -1638,6 +1638,11 @@ void QgsWMSProjectParser::drawOverlays( QPainter* p, int dpi, int width, int hei
   }
 }
 
+int QgsWMSProjectParser::nLayers() const
+{
+  return mProjectParser.numberOfLayers();
+}
+
 QDomElement QgsWMSProjectParser::composerByName( const QString& composerName ) const
 {
   QDomElement composerElem;

src/mapserver/qgswmsprojectparser.h

@@ -97,6 +97,8 @@ class QgsWMSProjectParser: public QgsWMSConfigParser
     /**Draw text annotation items from the QGIS projectfile*/
     void drawOverlays( QPainter* p, int dpi, int width, int height ) const;
 
+    int nLayers() const;
+
   private:
     QgsServerProjectParser mProjectParser;