Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[mssql] For consistency with other providers deleteFeatures should re…
…turn true to an empty list
  • Loading branch information
nyalldawson committed Oct 25, 2018
1 parent cd64d31 commit fb0b8fd
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/providers/mssql/qgsmssqlprovider.cpp
Expand Up @@ -1385,6 +1385,9 @@ bool QgsMssqlProvider::deleteFeatures( const QgsFeatureIds &id )
if ( mFidColName.isEmpty() )
return false;

if ( id.empty() )
return true; // for consistency providers return true to an empty list

QString featureIds;
for ( QgsFeatureIds::const_iterator it = id.begin(); it != id.end(); ++it )
{
Expand Down

0 comments on commit fb0b8fd

Please sign in to comment.