Skip to content

Commit

Permalink
More user of userFriendlyIdentifier
Browse files Browse the repository at this point in the history
  • Loading branch information
nyalldawson committed Dec 20, 2019
1 parent fdf211a commit 403e79e
Show file tree
Hide file tree
Showing 10 changed files with 14 additions and 18 deletions.
2 changes: 1 addition & 1 deletion src/app/qgsrasterlayerproperties.cpp
Expand Up @@ -268,7 +268,7 @@ QgsRasterLayerProperties::QgsRasterLayerProperties( QgsMapLayer *lyr, QgsMapCanv
metadataFrame->setLayout( layout );

QgsDebugMsg( "Setting crs to " + mRasterLayer->crs().toWkt() );
QgsDebugMsg( "Setting crs to " + mRasterLayer->crs().authid() + " - " + mRasterLayer->crs().description() );
QgsDebugMsg( "Setting crs to " + mRasterLayer->crs().userFriendlyIdentifier() );
mCrsSelector->setCrs( mRasterLayer->crs() );

// Set text for pyramid info box
Expand Down
4 changes: 2 additions & 2 deletions src/app/qgstemplateprojectsmodel.cpp
Expand Up @@ -51,8 +51,8 @@ QgsTemplateProjectsModel::QgsTemplateProjectsModel( QObject *parent )
QStandardItem *emptyProjectItem = new QStandardItem();

emptyProjectItem->setData( tr( "New Empty Project" ), QgsProjectListItemDelegate::TitleRole );
connect( QgsProject::instance(), &QgsProject::crsChanged, this, [emptyProjectItem]() { emptyProjectItem->setData( QgsProject::instance()->crs().description(), QgsProjectListItemDelegate::CrsRole ); } );
emptyProjectItem->setData( QgsProject::instance()->crs().description(), QgsProjectListItemDelegate::CrsRole );
connect( QgsProject::instance(), &QgsProject::crsChanged, this, [emptyProjectItem]() { emptyProjectItem->setData( QgsProject::instance()->crs().userFriendlyIdentifier(), QgsProjectListItemDelegate::CrsRole ); } );
emptyProjectItem->setData( QgsProject::instance()->crs().userFriendlyIdentifier(), QgsProjectListItemDelegate::CrsRole );
emptyProjectItem->setFlags( Qt::ItemFlag::ItemIsSelectable | Qt::ItemFlag::ItemIsEnabled ) ;
QSize previewSize( 250, 177 );
QImage image( previewSize, QImage::Format_ARGB32 );
Expand Down
6 changes: 2 additions & 4 deletions src/core/metadata/qgslayermetadataformatter.cpp
Expand Up @@ -118,8 +118,7 @@ QString QgsLayerMetadataFormatter::extentSectionHtml( const bool showSpatialExte
myMetadata += QStringLiteral( "<tr><td class=\"highlight\">" ) + tr( "CRS" ) + QStringLiteral( "</td><td>" );
if ( mMetadata.crs().isValid() )
{
myMetadata += mMetadata.crs().authid() + QStringLiteral( " - " );
myMetadata += mMetadata.crs().description() + QStringLiteral( " - " );
myMetadata += mMetadata.crs().userFriendlyIdentifier() + QStringLiteral( " - " );
if ( mMetadata.crs().isGeographic() )
myMetadata += tr( "Geographic" );
else
Expand All @@ -135,8 +134,7 @@ QString QgsLayerMetadataFormatter::extentSectionHtml( const bool showSpatialExte
{
myMetadata += QLatin1String( "<br />\n" );
}
myMetadata += QStringLiteral( "<strong>" ) + tr( "CRS" ) + QStringLiteral( ": </strong>" ) + spatialExtent.extentCrs.authid() + QStringLiteral( " - " );
myMetadata += spatialExtent.extentCrs.description() + QStringLiteral( " - " );
myMetadata += QStringLiteral( "<strong>" ) + tr( "CRS" ) + QStringLiteral( ": </strong>" ) + spatialExtent.extentCrs.userFriendlyIdentifier() + QStringLiteral( " - " );
if ( spatialExtent.extentCrs.isGeographic() )
myMetadata += tr( "Geographic" );
else
Expand Down
3 changes: 1 addition & 2 deletions src/core/qgsvectorlayer.cpp
Expand Up @@ -4758,8 +4758,7 @@ QString QgsVectorLayer::htmlMetadata() const
myMetadata += QStringLiteral( "<tr><td class=\"highlight\">" ) + tr( "CRS" ) + QStringLiteral( "</td><td>" );
if ( crs().isValid() )
{
myMetadata += crs().authid() + QStringLiteral( " - " );
myMetadata += crs().description() + QStringLiteral( " - " );
myMetadata += crs().userFriendlyIdentifier() + QStringLiteral( " - " );
if ( crs().isGeographic() )
myMetadata += tr( "Geographic" );
else
Expand Down
3 changes: 1 addition & 2 deletions src/core/raster/qgsrasterlayer.cpp
Expand Up @@ -323,8 +323,7 @@ QString QgsRasterLayer::htmlMetadata() const
myMetadata += QStringLiteral( "<tr><td class=\"highlight\">" ) % tr( "CRS" ) + QStringLiteral( "</td><td>" );
if ( crs().isValid() )
{
myMetadata += crs().authid() % QStringLiteral( " - " ) %
crs().description() + QStringLiteral( " - " );
myMetadata += crs().userFriendlyIdentifier() % QStringLiteral( " - " );
if ( crs().isGeographic() )
myMetadata += tr( "Geographic" );
else
Expand Down
2 changes: 1 addition & 1 deletion src/gui/qgsmetadatawidget.cpp
Expand Up @@ -304,7 +304,7 @@ void QgsMetadataWidget::crsChanged()
{
if ( ( mCrs.isValid() ) && ( mLayer ) )
{
lblCurrentCrs->setText( tr( "CRS: %1 - %2" ).arg( mCrs.authid(), mCrs.description() ) );
lblCurrentCrs->setText( tr( "CRS: %1" ).arg( mCrs.userFriendlyIdentifier() ) );
spatialExtentSelector->setEnabled( true );
spatialExtentSelector->setOutputCrs( mCrs );

Expand Down
2 changes: 1 addition & 1 deletion src/providers/wcs/qgswcscapabilities.cpp
Expand Up @@ -1030,7 +1030,7 @@ bool QgsWcsCapabilities::parseDescribeCoverageDom11( QByteArray const &xml, QgsW
box = QgsRectangle( low[0], low[1], high[0], high[1] );
}
coverage->boundingBoxes.insert( authid, box );
QgsDebugMsg( "crs: " + crs.authid() + ' ' + crs.description() + QString( " axisInverted = %1" ).arg( crs.hasAxisInverted() ) );
QgsDebugMsg( "crs: " + crs.userFriendlyIdentifier() + QString( " axisInverted = %1" ).arg( crs.hasAxisInverted() ) );
QgsDebugMsg( "BoundingBox: " + authid + " : " + box.toString() );
}
}
Expand Down
4 changes: 2 additions & 2 deletions src/providers/wcs/qgswcsprovider.cpp
Expand Up @@ -565,7 +565,7 @@ bool QgsWcsProvider::readBlock( int bandNo, QgsRectangle const &viewExtent, int
{
QgsDebugMsg( QStringLiteral( "Cached does not have CRS" ) );
}
QgsDebugMsg( "Cache CRS: " + cacheCrs.authid() + ' ' + cacheCrs.description() );
QgsDebugMsg( "Cache CRS: " + cacheCrs.userFriendlyIdentifier() );

QgsRectangle cacheExtent = QgsGdalProviderBase::extent( mCachedGdalDataset.get() );
QgsDebugMsg( "viewExtent = " + viewExtent.toString() );
Expand Down Expand Up @@ -1174,7 +1174,7 @@ bool QgsWcsProvider::calculateExtent() const
{
QgsDebugMsg( QStringLiteral( "Cached does not have CRS" ) );
}
QgsDebugMsg( "Cache CRS: " + cacheCrs.authid() + ' ' + cacheCrs.description() );
QgsDebugMsg( "Cache CRS: " + cacheCrs.userFriendlyIdentifier() );

// We can only verify extent if CRS is set
// If dataset comes rotated, GDAL probably cuts latitude extend, disable
Expand Down
2 changes: 1 addition & 1 deletion src/providers/wms/qgswmsprovider.cpp
Expand Up @@ -2951,7 +2951,7 @@ QgsRasterIdentifyResult QgsWmsProvider::identify( const QgsPointXY &point, QgsRa
// for results -> verify CRS and reprojects if necessary
QMap<QgsFeatureId, QgsFeature * > features = gml.featuresMap();
QgsCoordinateReferenceSystem featuresCrs = gml.crs();
QgsDebugMsg( QStringLiteral( "%1 features read, crs: %2 %3" ).arg( features.size() ).arg( featuresCrs.authid(), featuresCrs.description() ) );
QgsDebugMsg( QStringLiteral( "%1 features read, crs: %2" ).arg( features.size() ).arg( featuresCrs.userFriendlyIdentifier() ) );
QgsCoordinateTransform coordinateTransform;
if ( featuresCrs.isValid() && featuresCrs != crs() )
{
Expand Down
4 changes: 2 additions & 2 deletions src/providers/wms/qgswmssourceselect.cpp
Expand Up @@ -1109,8 +1109,8 @@ QString QgsWMSSourceSelect::descriptionForAuthId( const QString &authId )
return mCrsNames[ authId ];

QgsCoordinateReferenceSystem qgisSrs = QgsCoordinateReferenceSystem::fromOgcWmsCrs( authId );
mCrsNames.insert( authId, qgisSrs.description() );
return qgisSrs.description();
mCrsNames.insert( authId, qgisSrs.userFriendlyIdentifier() );
return qgisSrs.userFriendlyIdentifier();
}

void QgsWMSSourceSelect::addDefaultServers()
Expand Down

0 comments on commit 403e79e

Please sign in to comment.