Skip to content

Commit

Permalink
[mssql] Ignore unset attributes when changing attribute values, addin…
Browse files Browse the repository at this point in the history
…g features
  • Loading branch information
nyalldawson committed Feb 7, 2025
1 parent 5c045df commit 011c141
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions src/providers/mssql/qgsmssqlprovider.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1202,6 +1202,9 @@ bool QgsMssqlProvider::addFeatures( QgsFeatureList &flist, Flags flags )
if ( fld.name().isEmpty() )
continue; // invalid

if ( attrs.at( i ).userType() == qMetaTypeId< QgsUnsetAttributeValue >() )
continue;

if ( mDefaultValues.contains( i ) && mDefaultValues.value( i ) == attrs.at( i ).toString() )
continue; // skip fields having default values

Expand Down Expand Up @@ -1300,6 +1303,9 @@ bool QgsMssqlProvider::addFeatures( QgsFeatureList &flist, Flags flags )
if ( fld.name().isEmpty() )
continue; // invalid

if ( attrs.at( i ).userType() == qMetaTypeId< QgsUnsetAttributeValue >() )
continue;

if ( mDefaultValues.contains( i ) && mDefaultValues.value( i ) == attrs.at( i ).toString() )
continue; // skip fields having default values

Expand Down Expand Up @@ -1554,6 +1560,9 @@ bool QgsMssqlProvider::changeAttributeValues( const QgsChangedAttributesMap &att
if ( fld.name().isEmpty() )
continue; // invalid

if ( it2.value().userType() == qMetaTypeId< QgsUnsetAttributeValue >() )
continue;

if ( mComputedColumns.contains( fld.name() ) )
continue; // skip computed columns because they are done server side.

Expand Down Expand Up @@ -1593,6 +1602,9 @@ bool QgsMssqlProvider::changeAttributeValues( const QgsChangedAttributesMap &att
if ( fld.name().isEmpty() )
continue; // invalid

if ( it2.value().userType() == qMetaTypeId< QgsUnsetAttributeValue >() )
continue;

if ( mComputedColumns.contains( fld.name() ) )
continue; // skip computed columns because they are done server side.

Expand Down

0 comments on commit 011c141

Please sign in to comment.