Skip to content

Commit

Permalink
Rename faceIsActive() to isFaceActive()
Browse files Browse the repository at this point in the history
  • Loading branch information
wonder-sk committed Aug 20, 2018
1 parent 419ec1f commit 26631b3
Show file tree
Hide file tree
Showing 9 changed files with 14 additions and 14 deletions.
2 changes: 1 addition & 1 deletion python/core/auto_generated/mesh/qgsmeshdataprovider.sip.in
Expand Up @@ -361,7 +361,7 @@ Returns vector/scalar value associated with the index from the dataset
See QgsMeshDatasetMetadata.isVector() to check if the returned value is vector or scalar
%End

virtual bool faceIsActive( QgsMeshDatasetIndex index, int faceIndex ) const = 0;
virtual bool isFaceActive( QgsMeshDatasetIndex index, int faceIndex ) const = 0;
%Docstring
Returns whether the face is active for particular dataset

Expand Down
2 changes: 1 addition & 1 deletion src/core/mesh/qgsmeshdataprovider.h
Expand Up @@ -348,7 +348,7 @@ class CORE_EXPORT QgsMeshDatasetSourceInterface SIP_ABSTRACT
* | F1 | F2 | F3 |
* V3 ---- V4 ---- V6-----V8
*/
virtual bool faceIsActive( QgsMeshDatasetIndex index, int faceIndex ) const = 0;
virtual bool isFaceActive( QgsMeshDatasetIndex index, int faceIndex ) const = 0;
};


Expand Down
2 changes: 1 addition & 1 deletion src/core/mesh/qgsmeshlayer.cpp
Expand Up @@ -128,7 +128,7 @@ QgsMeshDatasetValue QgsMeshLayer::datasetValue( const QgsMeshDatasetIndex &index
if ( faceIndex >= 0 )
{
int nativeFaceIndex = mTriangularMesh->trianglesToNativeFaces().at( faceIndex );
if ( dataProvider()->faceIsActive( index, nativeFaceIndex ) )
if ( dataProvider()->isFaceActive( index, nativeFaceIndex ) )
{

if ( dataProvider()->datasetGroupMetadata( index ).dataType() == QgsMeshDatasetGroupMetadata::DataOnFaces )
Expand Down
2 changes: 1 addition & 1 deletion src/core/mesh/qgsmeshlayerrenderer.cpp
Expand Up @@ -114,7 +114,7 @@ void QgsMeshLayerRenderer::copyScalarDatasetValues( QgsMeshLayer *layer )
mScalarActiveFaceFlagValues.resize( mNativeMesh.faces.count() );
for ( int i = 0; i < mNativeMesh.faces.count(); ++i )
{
bool active = layer->dataProvider()->faceIsActive( datasetIndex, i );
bool active = layer->dataProvider()->isFaceActive( datasetIndex, i );
mScalarActiveFaceFlagValues[i] = active;
}

Expand Down
2 changes: 1 addition & 1 deletion src/core/mesh/qgsmeshmemorydataprovider.cpp
Expand Up @@ -407,7 +407,7 @@ QgsMeshDatasetValue QgsMeshMemoryDataProvider::datasetValue( QgsMeshDatasetIndex
}
}

bool QgsMeshMemoryDataProvider::faceIsActive( QgsMeshDatasetIndex index, int faceIndex ) const
bool QgsMeshMemoryDataProvider::isFaceActive( QgsMeshDatasetIndex index, int faceIndex ) const
{
Q_UNUSED( index );
Q_UNUSED( faceIndex );
Expand Down
2 changes: 1 addition & 1 deletion src/core/mesh/qgsmeshmemorydataprovider.h
Expand Up @@ -128,7 +128,7 @@ class QgsMeshMemoryDataProvider: public QgsMeshDataProvider
QgsMeshDatasetGroupMetadata datasetGroupMetadata( int groupIndex ) const override;
QgsMeshDatasetMetadata datasetMetadata( QgsMeshDatasetIndex index ) const override;
QgsMeshDatasetValue datasetValue( QgsMeshDatasetIndex index, int valueIndex ) const override;
bool faceIsActive( QgsMeshDatasetIndex index, int faceIndex ) const override;
bool isFaceActive( QgsMeshDatasetIndex index, int faceIndex ) const override;

//! Returns the memory provider key
static QString providerKey();
Expand Down
2 changes: 1 addition & 1 deletion src/providers/mdal/qgsmdalprovider.cpp
Expand Up @@ -216,7 +216,7 @@ QgsMeshDatasetValue QgsMdalProvider::datasetValue( QgsMeshDatasetIndex index, in
return val;
}

bool QgsMdalProvider::faceIsActive( QgsMeshDatasetIndex index, int faceIndex ) const
bool QgsMdalProvider::isFaceActive( QgsMeshDatasetIndex index, int faceIndex ) const
{
DatasetGroupH group = MDAL_M_datasetGroup( mMeshH, index.group() );
if ( !group )
Expand Down
2 changes: 1 addition & 1 deletion src/providers/mdal/qgsmdalprovider.h
Expand Up @@ -64,7 +64,7 @@ class QgsMdalProvider : public QgsMeshDataProvider
QgsMeshDatasetGroupMetadata datasetGroupMetadata( int groupIndex ) const override;
QgsMeshDatasetMetadata datasetMetadata( QgsMeshDatasetIndex index ) const override;
QgsMeshDatasetValue datasetValue( QgsMeshDatasetIndex index, int valueIndex ) const override;
bool faceIsActive( QgsMeshDatasetIndex index, int faceIndex ) const override;
bool isFaceActive( QgsMeshDatasetIndex index, int faceIndex ) const override;

private:
MeshH mMeshH;
Expand Down
12 changes: 6 additions & 6 deletions tests/src/core/testqgsmeshlayer.cpp
Expand Up @@ -180,7 +180,7 @@ void TestQgsMeshLayer::test_read_vertex_scalar_dataset()
QCOMPARE( QgsMeshDatasetValue( 2.0 + i ), dp->datasetValue( ds, 3 ) );
QCOMPARE( QgsMeshDatasetValue( 1.0 + i ), dp->datasetValue( ds, 4 ) );

QVERIFY( dp->faceIsActive( ds, 0 ) );
QVERIFY( dp->isFaceActive( ds, 0 ) );
}
}
}
Expand Down Expand Up @@ -220,7 +220,7 @@ void TestQgsMeshLayer::test_read_vertex_vector_dataset()
QCOMPARE( QgsMeshDatasetValue( 2 + i, 2 + i ), dp->datasetValue( ds, 3 ) );
QCOMPARE( QgsMeshDatasetValue( 1 + i, -2 + i ), dp->datasetValue( ds, 4 ) );

QVERIFY( dp->faceIsActive( ds, 0 ) );
QVERIFY( dp->isFaceActive( ds, 0 ) );
}
}
}
Expand Down Expand Up @@ -257,7 +257,7 @@ void TestQgsMeshLayer::test_read_face_scalar_dataset()
QCOMPARE( QgsMeshDatasetValue( 1 + i ), dp->datasetValue( ds, 0 ) );
QCOMPARE( QgsMeshDatasetValue( 2 + i ), dp->datasetValue( ds, 1 ) );

QVERIFY( dp->faceIsActive( ds, 0 ) );
QVERIFY( dp->isFaceActive( ds, 0 ) );
}
}
}
Expand Down Expand Up @@ -295,7 +295,7 @@ void TestQgsMeshLayer::test_read_face_vector_dataset()
QCOMPARE( QgsMeshDatasetValue( 1 + i, 1 + i ), dp->datasetValue( ds, 0 ) );
QCOMPARE( QgsMeshDatasetValue( 2 + i, 2 + i ), dp->datasetValue( ds, 1 ) );

QVERIFY( dp->faceIsActive( ds, 0 ) );
QVERIFY( dp->isFaceActive( ds, 0 ) );
}
}
}
Expand Down Expand Up @@ -331,8 +331,8 @@ void TestQgsMeshLayer::test_read_vertex_scalar_dataset_with_inactive_face()
QCOMPARE( QgsMeshDatasetValue( 1.0 + i ), dp->datasetValue( ds, 4 ) );

// We have 2 faces
QVERIFY( !dp->faceIsActive( ds, 0 ) );
QVERIFY( dp->faceIsActive( ds, 1 ) );
QVERIFY( !dp->isFaceActive( ds, 0 ) );
QVERIFY( dp->isFaceActive( ds, 1 ) );
}
}

Expand Down

0 comments on commit 26631b3

Please sign in to comment.