Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Prevent concurrent wfs iterators for now
  • Loading branch information
mhugent committed Jan 21, 2013
1 parent c5ed2cf commit 8e2262b
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 1 deletion.
14 changes: 13 additions & 1 deletion src/providers/wfs/qgswfsfeatureiterator.cpp
Expand Up @@ -12,6 +12,12 @@ QgsWFSFeatureIterator::QgsWFSFeatureIterator( QgsWFSProvider* provider, const Qg
return;
}

if ( mProvider->mActiveIterator )
{
mProvider->mActiveIterator->close();
}
mProvider->mActiveIterator = this;

switch ( request.filterType() )
{
case QgsFeatureRequest::FilterRect:
Expand Down Expand Up @@ -67,5 +73,11 @@ bool QgsWFSFeatureIterator::rewind()

bool QgsWFSFeatureIterator::close()
{
return false;
if ( !mProvider )
{
return false;
}
mProvider->mActiveIterator = 0;
mProvider = 0;
return true;
}
5 changes: 5 additions & 0 deletions src/providers/wfs/qgswfsprovider.cpp
Expand Up @@ -50,6 +50,7 @@ static const QString GML_NAMESPACE = "http://www.opengis.net/gml";
QgsWFSProvider::QgsWFSProvider( const QString& uri )
: QgsVectorDataProvider( uri ),
mNetworkRequestFinished( true ),
mActiveIterator( 0 ),
mRequestEncoding( QgsWFSProvider::GET ),
mUseIntersect( false ),
mWKBType( QGis::WKBUnknown ),
Expand Down Expand Up @@ -123,6 +124,10 @@ QgsWFSProvider::~QgsWFSProvider()
{
deleteData();
delete mSpatialIndex;
if ( mActiveIterator )
{
mActiveIterator->close();
}
}

void QgsWFSProvider::reloadData()
Expand Down

0 comments on commit 8e2262b

Please sign in to comment.