From 53477598891164156691abd944dc4dc1255fcd76 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20de=20Paula=20Rodrigues=20Neto=20Assis?=
 <espinafre@gmail.com>
Date: Mon, 25 May 2020 07:49:00 -0300
Subject: [PATCH] Code review.

---
 src/providers/postgres/qgspostgresprovider.cpp | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/providers/postgres/qgspostgresprovider.cpp b/src/providers/postgres/qgspostgresprovider.cpp
index 31e500ed2dab..bf2f84d92a79 100644
--- a/src/providers/postgres/qgspostgresprovider.cpp
+++ b/src/providers/postgres/qgspostgresprovider.cpp
@@ -2343,7 +2343,7 @@ bool QgsPostgresProvider::addFeatures( QgsFeatureList &flist, Flags flags )
 
       QString fieldname = mAttributeFields.at( idx ).name();
 
-      if ( mGeneratedValues.contains( idx ) && !mGeneratedValues.value( idx, QString() ).isEmpty() )
+      if ( !mGeneratedValues.value( idx, QString() ).isEmpty() )
       {
         QgsDebugMsg( QStringLiteral( "Skipping field %1 (idx %2) which is GENERATED." ).arg( fieldname, idx ) );
         continue;
@@ -2375,12 +2375,7 @@ bool QgsPostgresProvider::addFeatures( QgsFeatureList &flist, Flags flags )
 
       if ( i == flist.size() )
       {
-        // generated values override everything.
-        if ( !mGeneratedValues[idx].isEmpty() )
-        {
-          values += delim + defVal;
-        }
-        else if ( qgsVariantEqual( v, defVal ) )
+        if ( qgsVariantEqual( v, defVal ) )
         {
           if ( defVal.isNull() )
           {