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

Make size a property of QgsProcessingModelComponent

Browse Source 
API only for now, cannot be set by users
This commit is contained in:
Nyall Dawson 2020-03-02 12:04:25 +10:00
parent 6c98448a7f
commit 55a582d068
4 changed files with 66 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
python/core/auto_generated/processing/models/qgsprocessingmodelcomponent.sip.in
View File

@ -51,6 +51,24 @@ Returns the position of the model component within the graphical modeler.
Sets the ``position`` of the model component within the graphical modeler.
.. seealso:: :py:func:`position`
%End
QSizeF size() const;
%Docstring
Returns the size of the model component within the graphical modeler.
.. seealso:: :py:func:`setSize`
.. versionadded:: 3.14
%End
void setSize( QSizeF size );
%Docstring
Sets the ``size`` of the model component within the graphical modeler.
.. seealso:: :py:func:`size`
.. versionadded:: 3.14
%End
virtual QgsProcessingModelComponent *clone() = 0 /Factory/;

14
src/core/processing/models/qgsprocessingmodelcomponent.cpp
View File

@ -43,11 +43,23 @@ void QgsProcessingModelComponent::setPosition( QPointF position )
mPosition = position;
}
QSizeF QgsProcessingModelComponent::size() const
{
return mSize;
}
void QgsProcessingModelComponent::setSize( QSizeF size )
{
mSize = size;
}
void QgsProcessingModelComponent::saveCommonProperties( QVariantMap &map ) const
{
map.insert( QStringLiteral( "component_pos_x" ), mPosition.x() );
map.insert( QStringLiteral( "component_pos_y" ), mPosition.y() );
map.insert( QStringLiteral( "component_description" ), mDescription );
map.insert( QStringLiteral( "component_width" ), mSize.width() );
map.insert( QStringLiteral( "component_height" ), mSize.height() );
}
void QgsProcessingModelComponent::restoreCommonProperties( const QVariantMap &map )
@ -57,6 +69,8 @@ void QgsProcessingModelComponent::restoreCommonProperties( const QVariantMap &ma
pos.setY( map.value( QStringLiteral( "component_pos_y" ) ).toDouble() );
mPosition = pos;
mDescription = map.value( QStringLiteral( "component_description" ) ).toString();
mSize.setWidth( map.value( QStringLiteral( "component_width" ), QString::number( DEFAULT_COMPONENT_WIDTH ) ).toDouble() );
mSize.setHeight( map.value( QStringLiteral( "component_height" ), QString::number( DEFAULT_COMPONENT_HEIGHT ) ).toDouble() );
}
///@endcond

20
src/core/processing/models/qgsprocessingmodelcomponent.h
View File

@ -21,6 +21,7 @@
#include "qgis_core.h"
#include "qgis.h"
#include <QPointF>
#include <QSizeF>
///@cond NOT_STABLE
@ -59,6 +60,20 @@ class CORE_EXPORT QgsProcessingModelComponent
*/
void setPosition( QPointF position );
/**
* Returns the size of the model component within the graphical modeler.
* \see setSize()
* \since QGIS 3.14
*/
QSizeF size() const;
/**
* Sets the \a size of the model component within the graphical modeler.
* \see size()
* \since QGIS 3.14
*/
void setSize( QSizeF size );
/**
* Clones the component.
*
@ -91,11 +106,16 @@ class CORE_EXPORT QgsProcessingModelComponent
private:
static constexpr double DEFAULT_COMPONENT_WIDTH = 200;
static constexpr double DEFAULT_COMPONENT_HEIGHT = 30;
//! Position of component within model
QPointF mPosition;
QString mDescription;
QSizeF mSize = QSizeF( DEFAULT_COMPONENT_WIDTH, DEFAULT_COMPONENT_HEIGHT );
};
///@endcond

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

@ -7680,6 +7680,8 @@ void TestQgsProcessing::modelerAlgorithm()
QVERIFY( !child.isActive() );
child.setPosition( QPointF( 1, 2 ) );
QCOMPARE( child.position(), QPointF( 1, 2 ) );
child.setSize( QSizeF( 3, 4 ) );
QCOMPARE( child.size(), QSizeF( 3, 4 ) );
QVERIFY( child.parametersCollapsed() );
child.setParametersCollapsed( false );
QVERIFY( !child.parametersCollapsed() );
@ -7865,11 +7867,14 @@ void TestQgsProcessing::modelerAlgorithm()
QgsProcessingModelAlgorithm alg1a( "test", "testGroup" );
QgsProcessingModelParameter bool1;
bool1.setPosition( QPointF( 1, 2 ) );
bool1.setSize( QSizeF( 11, 12 ) );
alg1a.addModelParameter( new QgsProcessingParameterBoolean( "p1", "desc" ), bool1 );
QCOMPARE( alg1a.parameterDefinitions().count(), 1 );
QCOMPARE( alg1a.parameterDefinition( "p1" )->type(), QStringLiteral( "boolean" ) );
QCOMPARE( alg1a.parameterComponent( "p1" ).position().x(), 1.0 );
QCOMPARE( alg1a.parameterComponent( "p1" ).position().y(), 2.0 );
QCOMPARE( alg1a.parameterComponent( "p1" ).size().width(), 11.0 );
QCOMPARE( alg1a.parameterComponent( "p1" ).size().height(), 12.0 );
alg1a.updateModelParameter( new QgsProcessingParameterBoolean( "p1", "descx" ) );
QCOMPARE( alg1a.parameterDefinition( "p1" )->description(), QStringLiteral( "descx" ) );
alg1a.removeModelParameter( "bad" );
@ -8085,10 +8090,12 @@ void TestQgsProcessing::modelerAlgorithm()
alg5c1.setParametersCollapsed( true );
alg5c1.setDescription( "child 1" );
alg5c1.setPosition( QPointF( 1, 2 ) );
alg5c1.setSize( QSizeF( 11, 21 ) );
QMap<QString, QgsProcessingModelOutput> alg5c1outputs;
QgsProcessingModelOutput alg5c1out1;
alg5c1out1.setDescription( QStringLiteral( "my output" ) );
alg5c1out1.setPosition( QPointF( 3, 4 ) );
alg5c1out1.setSize( QSizeF( 31, 41 ) );
alg5c1outputs.insert( QStringLiteral( "a" ), alg5c1out1 );
alg5c1.setModelOutputs( alg5c1outputs );
alg5.addChildAlgorithm( alg5c1 );
@ -8104,6 +8111,7 @@ void TestQgsProcessing::modelerAlgorithm()
QgsProcessingModelParameter alg5pc1;
alg5pc1.setParameterName( QStringLiteral( "my_param" ) );
alg5pc1.setPosition( QPointF( 11, 12 ) );
alg5pc1.setSize( QSizeF( 21, 22 ) );
alg5.addModelParameter( new QgsProcessingParameterBoolean( QStringLiteral( "my_param" ) ), alg5pc1 );
QDomDocument doc = QDomDocument( "model" );
@ -8126,6 +8134,8 @@ void TestQgsProcessing::modelerAlgorithm()
QCOMPARE( alg6c1.description(), QStringLiteral( "child 1" ) );
QCOMPARE( alg6c1.position().x(), 1.0 );
QCOMPARE( alg6c1.position().y(), 2.0 );
QCOMPARE( alg6c1.size().width(), 11.0 );
QCOMPARE( alg6c1.size().height(), 21.0 );
QCOMPARE( alg6c1.parameterSources().count(), 5 );
QCOMPARE( alg6c1.parameterSources().value( "x" ).at( 0 ).source(), QgsProcessingModelChildParameterSource::ModelParameter );
QCOMPARE( alg6c1.parameterSources().value( "x" ).at( 0 ).parameterName(), QStringLiteral( "p1" ) );
@ -8155,7 +8165,8 @@ void TestQgsProcessing::modelerAlgorithm()
QCOMPARE( alg6c1.modelOutput( "a" ).description(), QStringLiteral( "my output" ) );
QCOMPARE( alg6c1.modelOutput( "a" ).position().x(), 3.0 );
QCOMPARE( alg6c1.modelOutput( "a" ).position().y(), 4.0 );
QCOMPARE( alg6c1.modelOutput( "a" ).size().width(), 31.0 );
QCOMPARE( alg6c1.modelOutput( "a" ).size().height(), 41.0 );
QgsProcessingModelChildAlgorithm alg6c2 = alg6.childAlgorithm( "cx2" );
QCOMPARE( alg6c2.childId(), QStringLiteral( "cx2" ) );
@ -8169,6 +8180,8 @@ void TestQgsProcessing::modelerAlgorithm()
QCOMPARE( alg6.parameterComponent( "my_param" ).parameterName(), QStringLiteral( "my_param" ) );
QCOMPARE( alg6.parameterComponent( "my_param" ).position().x(), 11.0 );
QCOMPARE( alg6.parameterComponent( "my_param" ).position().y(), 12.0 );
QCOMPARE( alg6.parameterComponent( "my_param" ).size().width(), 21.0 );
QCOMPARE( alg6.parameterComponent( "my_param" ).size().height(), 22.0 );
QCOMPARE( alg6.parameterDefinitions().count(), 1 );
QCOMPARE( alg6.parameterDefinitions().at( 0 )->type(), QStringLiteral( "boolean" ) );