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

[processing] Also hide empty providers in toolbox

Browse Source 
Initially empty providers were shown for discoverability,
but discussion with @nirvn and @elpaso have convinced me this
is a bad approach.
This commit is contained in:
Nyall Dawson 2018-09-17 10:20:16 +10:00
parent 1d8dbda6a1
commit c76410bdbd
2 changed files with 8 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/gui/processing/qgsprocessingtoolboxmodel.cpp
View File

@ -742,8 +742,7 @@ bool QgsProcessingToolboxProxyModel::filterAcceptsRow( int sourceRow, const QMod
}
bool hasChildren = false;
// groups are shown only if they have visible children
// but providers are shown if they have visible children, OR the filter string is empty
// groups/providers are shown only if they have visible children
int count = sourceModel()->rowCount( sourceIndex );
for ( int i = 0; i < count; ++i )
{
@ -754,15 +753,7 @@ bool QgsProcessingToolboxProxyModel::filterAcceptsRow( int sourceRow, const QMod
}
}
if ( QgsProcessingProvider *provider = mModel->providerForIndex( sourceIndex ) )
{
return ( hasChildren || mFilterString.trimmed().isEmpty() ) && provider->isActive();
}
else
{
// group
return hasChildren; // || isRecentNode;
}
return hasChildren;
}
bool QgsProcessingToolboxProxyModel::lessThan( const QModelIndex &left, const QModelIndex &right ) const

19
tests/src/gui/testqgsprocessingmodel.cpp
View File

@ -425,17 +425,13 @@ void TestQgsProcessingModel::testProxyModel()
QModelIndex alg2Index = model.index( 0, 0, group1Index );
QCOMPARE( model.data( alg2Index, Qt::DisplayRole ).toString(), QStringLiteral( "a2" ) );
// empty providers/groups should not be shown
model.setFilters( QgsProcessingToolboxProxyModel::FilterModeler );
group1Index = model.index( 0, 0, QModelIndex() );
QCOMPARE( model.rowCount(), 3 );
QCOMPARE( model.rowCount(), 1 );
QCOMPARE( model.rowCount( group1Index ), 1 );
QCOMPARE( model.data( group1Index, Qt::DisplayRole ).toString(), QStringLiteral( "group1" ) );
QCOMPARE( model.data( model.index( 0, 0, group1Index ), Qt::DisplayRole ).toString(), QStringLiteral( "a2" ) );
// no filter string, so empty providers should be shown
QCOMPARE( model.data( model.index( 1, 0, QModelIndex() ), Qt::DisplayRole ).toString(), QStringLiteral( "provider1" ) );
QCOMPARE( model.rowCount( model.index( 1, 0, QModelIndex() ) ), 0 );
QCOMPARE( model.data( model.index( 2, 0, QModelIndex() ), Qt::DisplayRole ).toString(), QStringLiteral( "provider2" ) );
QCOMPARE( model.rowCount( model.index( 2, 0, QModelIndex() ) ), 0 );
model.setFilters( QgsProcessingToolboxProxyModel::FilterToolbox );
group2Index = model.index( 0, 0, QModelIndex() );
QCOMPARE( model.rowCount(), 3 );
@ -590,23 +586,20 @@ void TestQgsProcessingModel::testView()
QModelIndex alg1Index = view.model()->index( 0, 0, group2Index );
QCOMPARE( view.algorithmForIndex( alg1Index )->id(), QStringLiteral( "p2:a1" ) );
// empty providers/groups should not be shown
view.setFilters( QgsProcessingToolboxProxyModel::FilterModeler );
QCOMPARE( view.model()->rowCount(), 2 );
QCOMPARE( view.model()->data( view.model()->index( 0, 0, QModelIndex() ), Qt::DisplayRole ).toString(), QStringLiteral( "provider1" ) );
QCOMPARE( view.model()->rowCount( view.model()->index( 0, 0, QModelIndex() ) ), 0 );
provider2Index = view.model()->index( 1, 0, QModelIndex() );
QCOMPARE( view.model()->rowCount(), 1 );
provider2Index = view.model()->index( 0, 0, QModelIndex() );
QCOMPARE( view.model()->data( provider2Index, Qt::DisplayRole ).toString(), QStringLiteral( "provider2" ) );
QCOMPARE( view.model()->rowCount( provider2Index ), 1 );
group2Index = view.model()->index( 0, 0, provider2Index );
QCOMPARE( view.model()->rowCount( group2Index ), 1 );
QCOMPARE( view.algorithmForIndex( view.model()->index( 0, 0, group2Index ) )->id(), QStringLiteral( "p2:a1" ) );
view.setFilters( QgsProcessingToolboxProxyModel::FilterToolbox );
QCOMPARE( view.model()->rowCount(), 2 );
QCOMPARE( view.model()->rowCount(), 1 );
provider1Index = view.model()->index( 0, 0, QModelIndex() );
QCOMPARE( view.model()->data( provider1Index, Qt::DisplayRole ).toString(), QStringLiteral( "provider1" ) );
QCOMPARE( view.model()->rowCount( provider1Index ), 1 );
QCOMPARE( view.model()->data( view.model()->index( 1, 0, QModelIndex() ), Qt::DisplayRole ).toString(), QStringLiteral( "provider2" ) );
QCOMPARE( view.model()->rowCount( view.model()->index( 1, 0, QModelIndex() ) ), 0 );
group2Index = view.model()->index( 0, 0, provider1Index );
QCOMPARE( view.model()->rowCount( group2Index ), 1 );
QCOMPARE( view.algorithmForIndex( view.model()->index( 0, 0, group2Index ) )->id(), QStringLiteral( "p1:a2" ) );