diff options
| author | Paolo Angelelli <paolo.angelelli@theqtcompany.com> | 2016-06-27 15:26:30 +0200 |
|---|---|---|
| committer | Paolo Angelelli <paolo.angelelli@theqtcompany.com> | 2016-06-28 10:06:53 +0000 |
| commit | e729fabed1b78729f65503ed3a7e0d5fddb13947 (patch) | |
| tree | 7f1a65c05bff044ab121b742fa0a52c35fa2734d /src | |
| parent | 17ab3e355ee6c16099139d601960b4212840e44c (diff) | |
| download | qtlocation-e729fabed1b78729f65503ed3a7e0d5fddb13947.tar.gz | |
Improving error reporting of HERE geoservice plugin
This patch makes the QGeoServiceProviderFactoryNokia spit clearer error
messages if the mandatory parameters are present but incorrect
Change-Id: I7d46409e7ea7d7517dd6c28f3679d60ebf619d5b
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/geoservices/nokia/qgeoserviceproviderplugin_nokia.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/plugins/geoservices/nokia/qgeoserviceproviderplugin_nokia.cpp b/src/plugins/geoservices/nokia/qgeoserviceproviderplugin_nokia.cpp index f6d05c36..4deac4ee 100644 --- a/src/plugins/geoservices/nokia/qgeoserviceproviderplugin_nokia.cpp +++ b/src/plugins/geoservices/nokia/qgeoserviceproviderplugin_nokia.cpp @@ -81,7 +81,11 @@ namespace token = parameters.value(QStringLiteral("here.token")).toString(); if (isValidParameter(appId) && isValidParameter(token)) - return; + return; + else if (!isValidParameter(appId)) + qWarning() << "Invalid here.app_id"; + else + qWarning() << "Invalid here.token"; if (parameters.contains(QStringLiteral("app_id")) || parameters.contains(QStringLiteral("token"))) qWarning() << QStringLiteral("Please prefix 'app_id' and 'token' with prefix 'here' (e.g.: 'here.app_id')"); |