Skip to content

Instantly share code, notes, and snippets.

@jef-n
Created April 22, 2015 09:28
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save jef-n/7656b837cef1641dbb18 to your computer and use it in GitHub Desktop.
Save jef-n/7656b837cef1641dbb18 to your computer and use it in GitHub Desktop.
diff --git a/src/providers/wms/qgswmsprovider.cpp b/src/providers/wms/qgswmsprovider.cpp
index 580c2da..8f9062d 100644
--- a/src/providers/wms/qgswmsprovider.cpp
+++ b/src/providers/wms/qgswmsprovider.cpp
@@ -123,7 +123,8 @@ QgsWmsProvider::QgsWmsProvider( QString const& uri, const QgsWmsCapabilities* ca
return;
}
- addLayers();
+ if( !addLayers() )
+ return;
// if there are already parsed capabilities, use them!
if ( capabilities )
@@ -265,15 +266,14 @@ QString QgsWmsProvider::getLegendGraphicUrl() const
return url.isEmpty() ? url : prepareUri( url );
}
-void QgsWmsProvider::addLayers()
+bool QgsWmsProvider::addLayers()
{
QgsDebugMsg( "Entering: layers:" + mSettings.mActiveSubLayers.join( ", " ) + ", styles:" + mSettings.mActiveSubStyles.join( ", " ) );
if ( mSettings.mActiveSubLayers.size() != mSettings.mActiveSubStyles.size() )
{
QgsMessageLog::logMessage( tr( "Number of layers and styles don't match" ), tr( "WMS" ) );
- mValid = false;
- return;
+ return false;
}
// Set the visibility of these new layers on by default
@@ -290,6 +290,8 @@ void QgsWmsProvider::addLayers()
mTileLayer = 0;
QgsDebugMsg( "Exiting." );
+
+ return true;
}
void QgsWmsProvider::setConnectionName( QString const &connName )
diff --git a/src/providers/wms/qgswmsprovider.h b/src/providers/wms/qgswmsprovider.h
index 7ea52da..51bce92 100644
--- a/src/providers/wms/qgswmsprovider.h
+++ b/src/providers/wms/qgswmsprovider.h
@@ -385,7 +385,7 @@ class QgsWmsProvider : public QgsRasterDataProvider
/**
* Add the list of WMS layer names to be rendered by this server
*/
- void addLayers();
+ bool addLayers();
/**
* Set the image projection (in WMS CRS format) used in the transfer from the WMS server
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment