[virtual layers] warn if unique identifier is not a valid field name

qgis · Nov 19, 2020 · 11b577e · 11b577e
1 parent 6797118
commit 11b577e
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/src/providers/virtual/qgsvirtuallayersourceselect.cpp b/src/providers/virtual/qgsvirtuallayersourceselect.cpp
@@ -218,7 +218,18 @@ void QgsVirtualLayerSourceSelect::testQuery()
     std::unique_ptr<QgsVectorLayer> vl( new QgsVectorLayer( def.toString(), QStringLiteral( "test" ), QStringLiteral( "virtual" ), options ) );
     if ( vl->isValid() )
     {
-      QMessageBox::information( nullptr, tr( "Virtual layer test" ), tr( "No error" ) );
+      const QStringList fieldNames = vl->fields().names();
+      if ( !mUIDField->text().isEmpty() && !vl->fields().names().contains( mUIDField->text() ) )
+      {
+        QStringList bulletedFieldNames;
+        for ( const QString &fieldName : fieldNames )
+        {
+          bulletedFieldNames.append( QStringLiteral( "<li>%1" ).arg( fieldName ) );
+        }
+        QMessageBox::warning( nullptr, tr( "Virtual layer test " ), tr( "The unique identifier field <b>%1</b> was not found in list of fields:<ul>%2</ul>" ).arg( mUIDField->text(), bulletedFieldNames.join( ' ' ) ) );
+      }
+      else
+        QMessageBox::information( nullptr, tr( "Virtual layer test" ), tr( "No error" ) );
     }
     else
     {