Skip to content

Commit e290aa5

Browse files
committed
Cleaning GetCapabilities response
1 parent 2dc76fd commit e290aa5

File tree

1 file changed

+22
-23
lines changed

1 file changed

+22
-23
lines changed

src/server/services/wms/qgswmsgetcapabilities.cpp

+22-23
Original file line numberDiff line numberDiff line change
@@ -756,40 +756,39 @@ namespace QgsWms
756756

757757
QDomElement layerParentElem = doc.createElement( QStringLiteral( "Layer" ) );
758758

759-
// Root Layer title
760-
QDomElement layerParentTitleElem = doc.createElement( QStringLiteral( "Title" ) );
761-
QDomText layerParentTitleText = doc.createTextNode( project->title() );
762-
layerParentTitleElem.appendChild( layerParentTitleText );
763-
layerParentElem.appendChild( layerParentTitleElem );
764-
765-
// Root Layer abstract
766-
QDomElement layerParentAbstElem = doc.createElement( QStringLiteral( "Abstract" ) );
767-
QDomText layerParentAbstText = doc.createTextNode( project->title() );
768-
layerParentAbstElem.appendChild( layerParentAbstText );
769-
layerParentElem.appendChild( layerParentAbstElem );
759+
if ( !project->title().isEmpty() )
760+
{
761+
// Root Layer title
762+
QDomElement layerParentTitleElem = doc.createElement( QStringLiteral( "Title" ) );
763+
QDomText layerParentTitleText = doc.createTextNode( project->title() );
764+
layerParentTitleElem.appendChild( layerParentTitleText );
765+
layerParentElem.appendChild( layerParentTitleElem );
766+
767+
// Root Layer abstract
768+
QDomElement layerParentAbstElem = doc.createElement( QStringLiteral( "Abstract" ) );
769+
QDomText layerParentAbstText = doc.createTextNode( project->title() );
770+
layerParentAbstElem.appendChild( layerParentAbstText );
771+
layerParentElem.appendChild( layerParentAbstElem );
772+
}
770773

771774
// Root Layer name
772-
QDomElement layerParentNameElem = doc.createElement( QStringLiteral( "Name" ) );
773-
QString rootName = QgsServerProjectUtils::wmsRootName( *project );
774-
if ( rootName.isEmpty() )
775+
QString rootLayerName = QgsServerProjectUtils::wmsRootName( *project );
776+
if ( rootLayerName.isEmpty() && !project->title().isEmpty() )
775777
{
776-
QDomText layerParentNameText = doc.createTextNode( project->title() );
777-
layerParentNameElem.appendChild( layerParentNameText );
778+
rootLayerName = project->title();
778779
}
779-
else
780+
781+
if ( !rootLayerName.isEmpty() )
780782
{
781-
QDomText layerParentNameText = doc.createTextNode( rootName );
783+
QDomElement layerParentNameElem = doc.createElement( QStringLiteral( "Name" ) );
784+
QDomText layerParentNameText = doc.createTextNode( rootLayerName );
782785
layerParentNameElem.appendChild( layerParentNameText );
786+
layerParentElem.appendChild( layerParentNameElem );
783787
}
784-
layerParentElem.appendChild( layerParentNameElem );
785788

786789
// Keyword list
787790
addKeywordListElement( project, doc, layerParentElem );
788791

789-
// Metadata (empty but needed for OGC tests RECOMMENDATIONS)
790-
QDomElement metaUrlElem = doc.createElement( QStringLiteral( "MetadataURL" ) );
791-
layerParentElem.appendChild( metaUrlElem );
792-
793792
// Root Layer tree name
794793
if ( projectSettings )
795794
{

0 commit comments

Comments
 (0)