Skip to content

Commit

Permalink
Api cleanups for qgsfeature
Browse files Browse the repository at this point in the history
git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@9506 c8812cc2-4d05-0410-92ff-de0c093fc19c
  • Loading branch information
timlinux committed Oct 20, 2008
1 parent 7d8c918 commit e2ad459
Show file tree
Hide file tree
Showing 14 changed files with 53 additions and 53 deletions.
4 changes: 2 additions & 2 deletions python/core/qgsfeature.sip
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class QgsFeature
* Get the feature id for this feature
* @return Feature id
*/
int featureId() const;
int id() const;

/**
* Set the feature id for this feature
Expand Down Expand Up @@ -87,7 +87,7 @@ class QgsFeature
* Reset the dirtiness of the feature. (i.e. make clean)
* You would normally do this after it's saved to permanent storage (e.g. disk, an ACID-compliant database)
*/
void resetDirty();
void clean();

/**
* Get the geometry object associated with this feature
Expand Down
2 changes: 1 addition & 1 deletion src/app/qgsattributetable.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -501,7 +501,7 @@ void QgsAttributeTable::putFeatureInTable( int row, const QgsFeature& fet )
}

//id-field
int id = fet.featureId();
int id = fet.id();
QTableWidgetItem *twi = new QTableWidgetItem( QString::number( id ) );
twi->setTextAlignment( Qt::AlignRight | Qt::AlignVCenter );
setItem( row, 0, twi );
Expand Down
2 changes: 1 addition & 1 deletion src/app/qgsattributetabledisplay.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -379,7 +379,7 @@ void QgsAttributeTableDisplay::doSearch( QString searchString )
{
if ( searchTree->checkAgainst( mLayer->pendingFields(), f.attributeMap() ) )
{
mSearchIds << f.featureId();
mSearchIds << f.id();
}

// check if there were errors during evaluating
Expand Down
10 changes: 5 additions & 5 deletions src/app/qgsmaptoolidentify.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -317,12 +317,12 @@ void QgsMapToolIdentify::identifyVectorLayer( const QgsPoint& point )
featureCount++;

QTreeWidgetItem* featureNode = mResults->addNode( "foo" );
featureNode->setData( 0, Qt::UserRole, QVariant( f_it->featureId() ) ); // save feature id
lastFeatureId = f_it->featureId();
featureNode->setData( 0, Qt::UserRole, QVariant( f_it->id() ) ); // save feature id
lastFeatureId = f_it->id();
featureNode->setText( 0, fieldIndex );

if ( layer->isEditable() )
mResults->addEdit( featureNode, f_it->featureId() );
mResults->addEdit( featureNode, f_it->id() );

const QgsAttributeMap& attr = f_it->attributeMap();

Expand Down Expand Up @@ -476,7 +476,7 @@ void QgsMapToolIdentify::editFeature( int featureId )
{
for ( QgsFeatureList::iterator it = mFeatureList.begin(); it != mFeatureList.end(); it++ )
{
if ( it->featureId() == featureId )
if ( it->id() == featureId )
{
editFeature( *it );
break;
Expand All @@ -503,7 +503,7 @@ void QgsMapToolIdentify::editFeature( QgsFeature &f )
for ( QgsAttributeMap::const_iterator it = dst.begin(); it != dst.end(); it++ )
{
if ( !src.contains( it.key() ) || it.value() != src[it.key()] )
layer->changeAttributeValue( f.featureId(), it.key(), it.value() );
layer->changeAttributeValue( f.id(), it.key(), it.value() );
}
}
delete ad;
Expand Down
2 changes: 1 addition & 1 deletion src/app/qgsmaptoolmovefeature.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ void QgsMapToolMoveFeature::canvasPressEvent( QMouseEvent * e )
}

mStartPointMapCoords = toMapCoordinates( e->pos() );
mMovedFeature = cf.featureId(); //todo: take the closest feature, not the first one...
mMovedFeature = cf.id(); //todo: take the closest feature, not the first one...
mRubberBand = createRubberBand();
mRubberBand->setToGeometry( cf.geometry(), *vlayer );
mRubberBand->setColor( Qt::red );
Expand Down
4 changes: 2 additions & 2 deletions src/core/qgsfeature.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ QgsFeature::~QgsFeature()
* Get the feature id for this feature
* @return Feature id
*/
int QgsFeature::featureId() const
int QgsFeature::id() const
{
return mFid;
}
Expand Down Expand Up @@ -204,7 +204,7 @@ bool QgsFeature::isDirty() const
return mDirty;
}

void QgsFeature::resetDirty()
void QgsFeature::clean()
{
mDirty = false;
}
4 changes: 2 additions & 2 deletions src/core/qgsfeature.h
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ class CORE_EXPORT QgsFeature
* Get the feature id for this feature
* @return Feature id
*/
int featureId() const;
int id() const;

/**
* Set the feature id for this feature
Expand Down Expand Up @@ -128,7 +128,7 @@ class CORE_EXPORT QgsFeature
* Reset the dirtiness of the feature. (i.e. make clean)
* You would normally do this after it's saved to permanent storage (e.g. disk, an ACID-compliant database)
*/
void resetDirty();
void clean();

/**
* Get the geometry object associated with this feature
Expand Down
2 changes: 1 addition & 1 deletion src/core/qgsvectordataprovider.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ bool QgsVectorDataProvider::featureAtId( int featureId,

while ( nextFeature( feature ) )
{
if ( feature.featureId() == featureId )
if ( feature.id() == featureId )
return TRUE;
}

Expand Down
2 changes: 1 addition & 1 deletion src/core/qgsvectorfilewriter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,7 @@ QgsVectorFileWriter::writeAsShapefile( QgsVectorLayer* layer,
// write all features
while ( provider->nextFeature( fet ) )
{
if ( onlySelected && !ids.contains( fet.featureId() ) )
if ( onlySelected && !ids.contains( fet.id() ) )
continue;

if ( shallTransform )
Expand Down
64 changes: 32 additions & 32 deletions src/core/qgsvectorlayer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ void QgsVectorLayer::drawLabels( QPainter * p, const QgsRect& viewExtent, const
{
if ( mRenderer->willRenderFeature( &fet ) )
{
bool sel = mSelectedFeatureIds.contains( fet.featureId() );
bool sel = mSelectedFeatureIds.contains( fet.id() );
mLabel->renderLabel( p, viewExtent, ct, theMapToPixelTransform, fet, sel, 0, scale );
}
featureCount++;
Expand Down Expand Up @@ -742,13 +742,13 @@ bool QgsVectorLayer::draw( QgsRenderContext& rendererContext )
if ( mEditable )
{
// Cache this for the use of (e.g.) modifying the feature's uncommitted geometry.
mCachedGeometries[fet.featureId()] = *fet.geometry();
mCachedGeometries[fet.id()] = *fet.geometry();
}

// check if feature is selected
// only show selections of the current layer
// TODO: create a mechanism to let layer know whether it's current layer or not [MD]
bool sel = mSelectedFeatureIds.contains( fet.featureId() );
bool sel = mSelectedFeatureIds.contains( fet.id() );

//QgsDebugMsg(QString("markerScale before renderFeature(): %1").arg(markerScaleFactor));
// markerScalerFactore reflects the wanted scaling of the marker
Expand Down Expand Up @@ -843,7 +843,7 @@ void QgsVectorLayer::select( QgsRect & rect, bool lock )
QgsFeature f;
while ( nextFeature( f ) )
{
select( f.featureId(), false ); // don't emit signal (not to redraw it everytime)
select( f.id(), false ); // don't emit signal (not to redraw it everytime)
}

emit selectionChanged(); // now emit signal to redraw layer
Expand All @@ -861,7 +861,7 @@ void QgsVectorLayer::invertSelection()
QgsFeature fet;
while ( nextFeature( fet ) )
{
select( fet.featureId(), false ); // don't emit signal
select( fet.id(), false ); // don't emit signal
}

for ( QgsFeatureIds::iterator iter = tmp.begin(); iter != tmp.end(); ++iter )
Expand Down Expand Up @@ -994,7 +994,7 @@ QgsRect QgsVectorLayer::boundingBoxOfSelected()
QgsFeature fet;
while ( nextFeature( fet ) )
{
if ( mSelectedFeatureIds.contains( fet.featureId() ) )
if ( mSelectedFeatureIds.contains( fet.id() ) )
{
if ( fet.geometry() )
{
Expand Down Expand Up @@ -1128,9 +1128,9 @@ void QgsVectorLayer::setSubsetString( QString subset )

void QgsVectorLayer::updateFeatureAttributes( QgsFeature &f )
{
if ( mChangedAttributeValues.contains( f.featureId() ) )
if ( mChangedAttributeValues.contains( f.id() ) )
{
const QgsAttributeMap &map = mChangedAttributeValues[f.featureId()];
const QgsAttributeMap &map = mChangedAttributeValues[f.id()];
for ( QgsAttributeMap::const_iterator it = map.begin(); it != map.end(); it++ )
f.changeAttribute( it.key(), it.value() );
}
Expand All @@ -1141,8 +1141,8 @@ void QgsVectorLayer::updateFeatureAttributes( QgsFeature &f )

void QgsVectorLayer::updateFeatureGeometry( QgsFeature &f )
{
if ( mChangedGeometries.contains( f.featureId() ) )
f.setGeometry( mChangedGeometries[f.featureId()] );
if ( mChangedGeometries.contains( f.id() ) )
f.setGeometry( mChangedGeometries[f.id()] );
}


Expand Down Expand Up @@ -1234,7 +1234,7 @@ bool QgsVectorLayer::nextFeature( QgsFeature &f )

for ( QgsFeatureList::iterator it = mAddedFeatures.begin(); it != mAddedFeatures.end(); it++ )
{
if ( fid != it->featureId() )
if ( fid != it->id() )
{
found = true;
f.setAttributeMap( it->attributeMap() );
Expand All @@ -1243,7 +1243,7 @@ bool QgsVectorLayer::nextFeature( QgsFeature &f )
}

if ( !found )
QgsLogger::warning( QString( "No attributes for the added feature %1 found" ).arg( f.featureId() ) );
QgsLogger::warning( QString( "No attributes for the added feature %1 found" ).arg( f.id() ) );
}
else
{
Expand All @@ -1265,7 +1265,7 @@ bool QgsVectorLayer::nextFeature( QgsFeature &f )

for ( ; mFetchAddedFeaturesIt != mAddedFeatures.end(); mFetchAddedFeaturesIt++ )
{
int fid = mFetchAddedFeaturesIt->featureId();
int fid = mFetchAddedFeaturesIt->id();

if ( mFetchConsidered.contains( fid ) )
// must have changed geometry outside rectangle
Expand Down Expand Up @@ -1297,7 +1297,7 @@ bool QgsVectorLayer::nextFeature( QgsFeature &f )

while ( dataProvider()->nextFeature( f ) )
{
if ( mFetchConsidered.contains( f.featureId() ) )
if ( mFetchConsidered.contains( f.id() ) )
continue;

if ( mEditable )
Expand Down Expand Up @@ -1333,7 +1333,7 @@ int QgsVectorLayer::featureAtId( int featureId, QgsFeature& f, bool fetchGeometr

for ( QgsFeatureList::iterator it = mAddedFeatures.begin(); it != mAddedFeatures.end(); it++ )
{
if ( featureId != it->featureId() )
if ( featureId != it->id() )
{
found = true;
f.setAttributeMap( it->attributeMap() );
Expand All @@ -1342,7 +1342,7 @@ int QgsVectorLayer::featureAtId( int featureId, QgsFeature& f, bool fetchGeometr
}

if ( !found )
QgsLogger::warning( QString( "No attributes for the added feature %1 found" ).arg( f.featureId() ) );
QgsLogger::warning( QString( "No attributes for the added feature %1 found" ).arg( f.id() ) );
}
else
{
Expand All @@ -1359,9 +1359,9 @@ int QgsVectorLayer::featureAtId( int featureId, QgsFeature& f, bool fetchGeometr
//added features
for ( QgsFeatureList::iterator iter = mAddedFeatures.begin(); iter != mAddedFeatures.end(); ++iter )
{
if ( iter->featureId() == featureId )
if ( iter->id() == featureId )
{
f.setFeatureId( iter->featureId() );
f.setFeatureId( iter->id() );

if ( fetchGeometries )
f.setGeometry( *iter->geometry() );
Expand Down Expand Up @@ -1574,7 +1574,7 @@ int QgsVectorLayer::addRing( const QList<QgsPoint>& ring )
addRingReturnCode = f.geometry()->addRing( ring );
if ( addRingReturnCode == 0 )
{
mChangedGeometries.insert( f.featureId(), *f.geometry() );
mChangedGeometries.insert( f.id(), *f.geometry() );
setModified( true, true );
break;
}
Expand Down Expand Up @@ -1610,7 +1610,7 @@ int QgsVectorLayer::addIsland( const QList<QgsPoint>& ring )
//look if id of selected feature belongs to an added feature
for ( QgsFeatureList::iterator addedIt = mAddedFeatures.begin(); addedIt != mAddedFeatures.end(); ++addedIt )
{
if ( addedIt->featureId() == selectedFeatureId )
if ( addedIt->id() == selectedFeatureId )
{
return addedIt->geometry()->addIsland( ring );
}
Expand Down Expand Up @@ -1644,7 +1644,7 @@ int QgsVectorLayer::translateFeature( int featureId, double dx, double dy )
//look if id of selected feature belongs to an added feature
for ( QgsFeatureList::iterator addedIt = mAddedFeatures.begin(); addedIt != mAddedFeatures.end(); ++addedIt )
{
if ( addedIt->featureId() == featureId )
if ( addedIt->id() == featureId )
{
return addedIt->geometry()->translate( dx, dy );
}
Expand Down Expand Up @@ -1729,7 +1729,7 @@ int QgsVectorLayer::splitFeatures( const QList<QgsPoint>& splitLine, bool topolo
if ( splitFunctionReturn == 0 )
{
//change this geometry
mChangedGeometries.insert( select_it->featureId(), *( select_it->geometry() ) );
mChangedGeometries.insert( select_it->id(), *( select_it->geometry() ) );

//insert new features
for ( int i = 0; i < newGeometries.size(); ++i )
Expand Down Expand Up @@ -2504,7 +2504,7 @@ bool QgsVectorLayer::changeAttributeValue( int fid, int field, QVariant value, b
{
// updated added feature
int i;
for ( i = 0; i < mAddedFeatures.size() && mAddedFeatures[i].featureId() != fid; i++ )
for ( i = 0; i < mAddedFeatures.size() && mAddedFeatures[i].id() != fid; i++ )
;

if ( i == mAddedFeatures.size() )
Expand Down Expand Up @@ -2783,21 +2783,21 @@ bool QgsVectorLayer::commitChanges()
{
for ( QgsFeatureList::iterator iter = mAddedFeatures.begin(); iter != mAddedFeatures.end(); ++iter )
{
if ( mDeletedFeatureIds.contains( iter->featureId() ) )
if ( mDeletedFeatureIds.contains( iter->id() ) )
{
mDeletedFeatureIds.remove( iter->featureId() );
mDeletedFeatureIds.remove( iter->id() );

if ( mChangedGeometries.contains( iter->featureId() ) )
mChangedGeometries.remove( iter->featureId() );
if ( mChangedGeometries.contains( iter->id() ) )
mChangedGeometries.remove( iter->id() );

mAddedFeatures.erase( iter );

continue;
}

if ( mChangedGeometries.contains( iter->featureId() ) )
if ( mChangedGeometries.contains( iter->id() ) )
{
iter->setGeometry( mChangedGeometries.take( iter->featureId() ) );
iter->setGeometry( mChangedGeometries.take( iter->id() ) );
}
}

Expand Down Expand Up @@ -2973,7 +2973,7 @@ QgsFeatureList QgsVectorLayer::selectedFeatures()
// Check this selected item against the uncommitted added features
for ( QgsFeatureList::iterator iter = mAddedFeatures.begin(); iter != mAddedFeatures.end(); ++iter )
{
if ( *it == iter->featureId() )
if ( *it == iter->id() )
{
feat = QgsFeature( *iter );
selectionIsAddedFeature = TRUE;
Expand Down Expand Up @@ -3024,7 +3024,7 @@ bool QgsVectorLayer::addFeatures( QgsFeatureList features, bool makeSelected )

if ( makeSelected )
{
mSelectedFeatureIds.insert( iter->featureId() );
mSelectedFeatureIds.insert( iter->id() );
}
}

Expand Down Expand Up @@ -3146,7 +3146,7 @@ int QgsVectorLayer::snapWithContext( const QgsPoint& startPoint, double snapping
QgsFeature f;
while ( nextFeature( f ) )
{
snapToGeometry( startPoint, f.featureId(), f.geometry(), sqrSnappingTolerance, snappingResults, snap_to );
snapToGeometry( startPoint, f.id(), f.geometry(), sqrSnappingTolerance, snappingResults, snap_to );
++n;
}

Expand Down
2 changes: 1 addition & 1 deletion src/core/spatialindex/qgsspatialindex.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ bool QgsSpatialIndex::featureInfo( QgsFeature& f, Tools::Geometry::Region& r, lo
if ( !g )
return false;

id = f.featureId();
id = f.id();
r = rectToRegion( g->boundingBox() );
return true;
}
Expand Down
2 changes: 1 addition & 1 deletion src/plugins/grass/qgsgrassmodule.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2967,7 +2967,7 @@ void QgsGrassModuleSelection::updateSelection()
int i = 0;
while ( provider->nextFeature( feature ) )
{
if ( !selected.contains( feature.featureId() ) )
if ( !selected.contains( feature.id() ) )
continue;

QgsAttributeMap attr = feature.attributeMap();
Expand Down
Loading

0 comments on commit e2ad459

Please sign in to comment.