Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Rename conversionFactor to scaleFactor
  • Loading branch information
m-kuhn committed Sep 10, 2018
1 parent 57677f6 commit cf82c13
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 12 deletions.
4 changes: 2 additions & 2 deletions python/core/auto_generated/qgscoordinatetransform.sip.in
Expand Up @@ -358,11 +358,11 @@ been modified in order to ensure that outdated CRS transforms are not created.
.. versionadded:: 3.0
%End

double conversionFactor( const QgsRectangle &referenceExtent ) const;
double scaleFactor( const QgsRectangle &referenceExtent ) const;
%Docstring
Computes an *estimated* conversion factor between source and destination units:

sourceUnits * conversionFactor = destinationUnits
sourceUnits * scaleFactor = destinationUnits

:param referenceExtent: A reference extent based on which to perform the computation

Expand Down
2 changes: 1 addition & 1 deletion src/core/qgscoordinatetransform.cpp
Expand Up @@ -799,7 +799,7 @@ void QgsCoordinateTransform::invalidateCache()
sCacheLock.unlock();
}

double QgsCoordinateTransform::conversionFactor( const QgsRectangle &ReferenceExtent ) const
double QgsCoordinateTransform::scaleFactor( const QgsRectangle &ReferenceExtent ) const
{
QgsPointXY source1( ReferenceExtent.xMinimum(), ReferenceExtent.yMinimum() );
QgsPointXY source2( ReferenceExtent.xMaximum(), ReferenceExtent.yMaximum() );
Expand Down
4 changes: 2 additions & 2 deletions src/core/qgscoordinatetransform.h
Expand Up @@ -408,13 +408,13 @@ class CORE_EXPORT QgsCoordinateTransform
/**
* Computes an *estimated* conversion factor between source and destination units:
*
* sourceUnits * conversionFactor = destinationUnits
* sourceUnits * scaleFactor = destinationUnits
*
* \param referenceExtent A reference extent based on which to perform the computation
*
* \since QGIS 3.4
*/
double conversionFactor( const QgsRectangle &referenceExtent ) const;
double scaleFactor( const QgsRectangle &referenceExtent ) const;

private:

Expand Down
2 changes: 1 addition & 1 deletion src/core/qgsmapsettings.cpp
Expand Up @@ -396,7 +396,7 @@ QgsCoordinateTransform QgsMapSettings::layerTransform( const QgsMapLayer *layer

double QgsMapSettings::layerToMapUnits( const QgsMapLayer *layer, const QgsRectangle &referenceExtent ) const
{
return layerTransform( layer ).conversionFactor( referenceExtent );
return layerTransform( layer ).scaleFactor( referenceExtent );
}


Expand Down
12 changes: 6 additions & 6 deletions tests/src/core/testqgscoordinatetransform.cpp
Expand Up @@ -36,8 +36,8 @@ class TestQgsCoordinateTransform: public QObject
void isValid();
void isShortCircuited();
void contextShared();
void conversionFactor();
void conversionFactor_data();
void scaleFactor();
void scaleFactor_data();

private:

Expand Down Expand Up @@ -215,7 +215,7 @@ void TestQgsCoordinateTransform::contextShared()
QCOMPARE( copy2.sourceDestinationDatumTransforms(), expected );
}

void TestQgsCoordinateTransform::conversionFactor()
void TestQgsCoordinateTransform::scaleFactor()
{
QFETCH( QgsCoordinateReferenceSystem, sourceCrs );
QFETCH( QgsCoordinateReferenceSystem, destCrs );
Expand All @@ -224,11 +224,11 @@ void TestQgsCoordinateTransform::conversionFactor()

QgsCoordinateTransform ct( sourceCrs, destCrs, QgsProject::instance() );

// qDebug() << QString::number(ct.conversionFactor( rect ), 'g', 17) ;
QVERIFY( qgsDoubleNear( ct.conversionFactor( rect ), factor ) );
// qDebug() << QString::number(ct.scaleFactor( rect ), 'g', 17) ;
QVERIFY( qgsDoubleNear( ct.scaleFactor( rect ), factor ) );
}

void TestQgsCoordinateTransform::conversionFactor_data()
void TestQgsCoordinateTransform::scaleFactor_data()
{
QTest::addColumn<QgsCoordinateReferenceSystem>( "sourceCrs" );
QTest::addColumn<QgsCoordinateReferenceSystem>( "destCrs" );
Expand Down

0 comments on commit cf82c13

Please sign in to comment.