[Server] Fix QgsServerProjectUtils readListEntry

rldhont · rldhont · commit 6feca53278c8 · 2017-05-26T15:48:03.000+02:00
diff --git a/src/server/qgsserverprojectutils.cpp b/src/server/qgsserverprojectutils.cpp
@@ -34,7 +34,20 @@ QString QgsServerProjectUtils::owsServiceAbstract( const QgsProject &project )
 
 QStringList QgsServerProjectUtils::owsServiceKeywords( const QgsProject &project )
 {
-  return project.readListEntry( QStringLiteral( "WMSKeywordList" ), QStringLiteral( "/" ) );
+  QStringList keywordList;
+  QStringList list = project.readListEntry( QStringLiteral( "WMSKeywordList" ), QStringLiteral( "/" ), QStringList() );
+  if ( !list.isEmpty() )
+  {
+    for ( int i = 0; i < list.size(); ++i )
+    {
+      QString keyword = list.at( i );
+      if ( !keyword.isEmpty() )
+      {
+        keywordList.append( keyword );
+      }
+    }
+  }
+  return keywordList;
 }
 
 QString QgsServerProjectUtils::owsServiceOnlineResource( const QgsProject &project )
@@ -141,7 +154,19 @@ QStringList QgsServerProjectUtils::wmsRestrictedComposers( const QgsProject &pro
 
 QStringList QgsServerProjectUtils::wmsOutputCrsList( const QgsProject &project )
 {
-  QStringList crsList = project.readListEntry( QStringLiteral( "WMSCrsList" ), QStringLiteral( "/" ), QStringList() );
+  QStringList crsList;
+  QStringList wmsCrsList = project.readListEntry( QStringLiteral( "WMSCrsList" ), QStringLiteral( "/" ), QStringList() );
+  if ( !wmsCrsList.isEmpty() )
+  {
+    for ( int i = 0; i < wmsCrsList.size(); ++i )
+    {
+      QString crs = wmsCrsList.at( i );
+      if ( !crs.isEmpty() )
+      {
+        crsList.append( crs );
+      }
+    }
+  }
   if ( crsList.isEmpty() )
   {
     QStringList valueList = project.readListEntry( QStringLiteral( "WMSEpsgList" ), QStringLiteral( "/" ), QStringList() );
diff --git a/tests/testdata/qgis_server/wcs_getcapabilities.txt b/tests/testdata/qgis_server/wcs_getcapabilities.txt
@@ -6,9 +6,6 @@ Content-Type: text/xml; charset=utf-8
   <name>WCS</name>
   <label>QGIS Server test</label>
   <description><![CDATA[Simple test app.]]></description>
-  <keywords>
-   <keyword></keyword>
-  </keywords>
   <responsibleParty>
    <individualName>Stéphane Brunner</individualName>
    <organisationName>QGIS</organisationName>
