sharpetronics/QGIS
1
0
Fork 0
mirror of https://github.com/qgis/QGIS.git synced 2025-02-28 00:17:30 -05:00
Code Issues Packages Projects Releases Wiki Activity

Tweak QgsProcessingUtils::combineLayerExtents for future proofing, remove deprecated usage

Browse Source
This commit is contained in:
Nyall Dawson 2019-04-02 16:20:08 +10:00
parent 82f2cb18b2
commit b6bc1ee2d4
15 changed files with 26 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
python/core/auto_generated/processing/qgsprocessingutils.sip.in
View File

@ -177,11 +177,11 @@ temporary layer store.
%End
static QgsRectangle combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs, const QgsCoordinateTransformContext &transformContext );
static QgsRectangle combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs, QgsProcessingContext &context );
%Docstring
Combines the extent of several map ``layers``. If specified, the target ``crs``
will be used to transform the layer's extent to the desired output reference system
using the specified ``transformContext``.
using the specified ``context``.
.. versionadded:: 3.10
%End

6
python/plugins/processing/algs/grass7/Grass7Algorithm.py
View File

@ -489,9 +489,9 @@ class Grass7Algorithm(QgsProcessingAlgorithm):
elif layer.type() == QgsMapLayerType.VectorLayer:
self.loadVectorLayer(layerName, layer, external=None, feedback=feedback)
self.postInputs()
self.postInputs(context)
def postInputs(self):
def postInputs(self, context):
"""
After layer imports, we need to update some internal parameters
"""
@ -500,7 +500,7 @@ class Grass7Algorithm(QgsProcessingAlgorithm):
# Build GRASS region
if self.region.isEmpty():
self.region = QgsProcessingUtils.combineLayerExtents(self.inputLayers)
self.region = QgsProcessingUtils.combineLayerExtents(self.inputLayers, context)
command = 'g.region n={} s={} e={} w={}'.format(
self.region.yMaximum(), self.region.yMinimum(),
self.region.xMaximum(), self.region.xMinimum()

2
python/plugins/processing/algs/grass7/ext/r_blend_combine.py
View File

@ -33,7 +33,7 @@ def processInputs(alg, parameters, context, feedback):
# Use v.in.ogr
for name in ['first', 'second']:
alg.loadRasterLayerFromParameter(name, parameters, context, False, None)
alg.postInputs()
alg.postInputs(context)
def processOutputs(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/r_blend_rgb.py
View File

@ -37,7 +37,7 @@ def processInputs(alg, parameters, context, feedback):
# Use v.in.ogr
for name in ['first', 'second']:
alg.loadRasterLayerFromParameter(name, parameters, context, False, None)
alg.postInputs()
alg.postInputs(context)
def processCommand(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/r_category.py
View File

@ -52,7 +52,7 @@ def processInputs(alg, parameters, context, feedback):
parameters, context,
False, None)
alg.loadRasterLayerFromParameter('map', parameters, context)
alg.postInputs()
alg.postInputs(context)
def processCommand(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/r_colors.py
View File

@ -54,7 +54,7 @@ def processInputs(alg, parameters, context, feedback):
if raster:
alg.loadRasterLayerFromParameter('raster', parameters, context, False, None)
alg.postInputs()
alg.postInputs(context)
def processCommand(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/r_colors_stddev.py
View File

@ -35,7 +35,7 @@ def processInputs(alg, parameters, context, feedback):
# We need to import all the bands and color tables of the input raster
alg.loadRasterLayerFromParameter('map', parameters, context, False, None)
alg.postInputs()
alg.postInputs(context)
def processCommand(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/r_null.py
View File

@ -42,7 +42,7 @@ def processInputs(alg, parameters, context, feedback):
# We need to import without r.external
alg.loadRasterLayerFromParameter('map', parameters, context, False)
alg.postInputs()
alg.postInputs(context)
def processCommand(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/r_rgb.py
View File

@ -32,7 +32,7 @@ def processInputs(alg, parameters, context, feedback):
# We need to import all the bands and color tables of the input raster
alg.loadRasterLayerFromParameter('input', parameters, context, False, None)
alg.postInputs()
alg.postInputs(context)
def processCommand(alg, parameters, context, feedback):

2
python/plugins/processing/algs/grass7/ext/v_sample.py
View File

@ -34,4 +34,4 @@ def processInputs(alg, parameters, context, feedback):
# and we can use r.external for the raster
alg.loadVectorLayerFromParameter('input', parameters, context, feedback, False)
alg.loadRasterLayerFromParameter('raster', parameters, context, True)
alg.postInputs()
alg.postInputs(context)

2
python/plugins/processing/algs/grass7/ext/v_to_3d.py
View File

@ -42,4 +42,4 @@ def processInputs(alg, parameters, context, feedback):
# We need to import the vector layer with v.in.ogr
alg.loadVectorLayerFromParameter('input', parameters, context, feedback, False)
alg.postInputs()
alg.postInputs(context)

2
python/plugins/processing/algs/qgis/RasterCalculator.py
View File

@ -127,7 +127,7 @@ class RasterCalculator(QgisAlgorithm):
bbox = transform.transformBoundingBox(bbox)
if bbox.isNull() and layers:
bbox = QgsProcessingUtils.combineLayerExtents(layers, crs)
bbox = QgsProcessingUtils.combineLayerExtents(layers, crs, context)
cellsize = self.parameterAsDouble(parameters, self.CELLSIZE, context)
if cellsize == 0 and not layers:

7
src/core/processing/qgsprocessingutils.cpp
View File

@ -627,7 +627,7 @@ void QgsProcessingUtils::createFeatureSinkPython( QgsFeatureSink **sink, QString
}
QgsRectangle QgsProcessingUtils::combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs, const QgsCoordinateTransformContext &transformContext )
QgsRectangle QgsProcessingUtils::combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs, QgsProcessingContext &context )
{
QgsRectangle extent;
for ( const QgsMapLayer *layer : layers )
@ -638,7 +638,7 @@ QgsRectangle QgsProcessingUtils::combineLayerExtents( const QList<QgsMapLayer *>
if ( crs.isValid() )
{
//transform layer extent to target CRS
QgsCoordinateTransform ct( layer->crs(), crs, transformContext );
QgsCoordinateTransform ct( layer->crs(), crs, context.transformContext() );
try
{
QgsRectangle reprojExtent = ct.transformBoundingBox( layer->extent() );
@ -662,7 +662,8 @@ QgsRectangle QgsProcessingUtils::combineLayerExtents( const QList<QgsMapLayer *>
// Deprecated
QgsRectangle QgsProcessingUtils::combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs )
{
return QgsProcessingUtils::combineLayerExtents( layers, crs, QgsCoordinateTransformContext( ) );
QgsProcessingContext context;
return QgsProcessingUtils::combineLayerExtents( layers, crs, context );
}
QVariant QgsProcessingUtils::generateIteratingDestination( const QVariant &input, const QVariant &id, QgsProcessingContext &context )

4
src/core/processing/qgsprocessingutils.h
View File

@ -217,10 +217,10 @@ class CORE_EXPORT QgsProcessingUtils
/**
* Combines the extent of several map \a layers. If specified, the target \a crs
* will be used to transform the layer's extent to the desired output reference system
* using the specified \a transformContext.
* using the specified \a context.
* \since QGIS 3.10
*/
static QgsRectangle combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs, const QgsCoordinateTransformContext &transformContext );
static QgsRectangle combineLayerExtents( const QList<QgsMapLayer *> &layers, const QgsCoordinateReferenceSystem &crs, QgsProcessingContext &context );
/**
* Combines the extent of several map \a layers. If specified, the target \a crs

9
tests/src/analysis/testqgsprocessing.cpp
View File

@ -6236,7 +6236,8 @@ void TestQgsProcessing::checkParamValues()
void TestQgsProcessing::combineLayerExtent()
{
QgsRectangle ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() );
QgsProcessingContext context;
QgsRectangle ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>(), context );
QVERIFY( ext.isNull() );
QString testDataDir = QStringLiteral( TEST_DATA_DIR ) + '/'; //defined in CmakeLists.txt
@ -6248,20 +6249,20 @@ void TestQgsProcessing::combineLayerExtent()
QFileInfo fi2( raster2 );
std::unique_ptr< QgsRasterLayer > r2( new QgsRasterLayer( fi2.filePath(), "R2" ) );
ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() << r1.get() );
ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() << r1.get(), context );
QGSCOMPARENEAR( ext.xMinimum(), 1535375.000000, 10 );
QGSCOMPARENEAR( ext.xMaximum(), 1535475, 10 );
QGSCOMPARENEAR( ext.yMinimum(), 5083255, 10 );
QGSCOMPARENEAR( ext.yMaximum(), 5083355, 10 );
ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() << r1.get() << r2.get() );
ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() << r1.get() << r2.get(), context );
QGSCOMPARENEAR( ext.xMinimum(), 781662, 10 );
QGSCOMPARENEAR( ext.xMaximum(), 1535475, 10 );
QGSCOMPARENEAR( ext.yMinimum(), 3339523, 10 );
QGSCOMPARENEAR( ext.yMaximum(), 5083355, 10 );
// with reprojection
ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() << r1.get() << r2.get(), QgsCoordinateReferenceSystem::fromEpsgId( 3785 ) );
ext = QgsProcessingUtils::combineLayerExtents( QList< QgsMapLayer *>() << r1.get() << r2.get(), QgsCoordinateReferenceSystem::fromEpsgId( 3785 ), context );
QGSCOMPARENEAR( ext.xMinimum(), 1995320, 10 );
QGSCOMPARENEAR( ext.xMaximum(), 2008833, 10 );
QGSCOMPARENEAR( ext.yMinimum(), 3523084, 10 );