Sanity checks in add attribute dialog

qgis · Jul 24, 2014 · c255823 · c255823
1 parent 302b2ae
commit c255823
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/src/app/qgsaddattrdialog.cpp b/src/app/qgsaddattrdialog.cpp
@@ -113,6 +113,18 @@ void QgsAddAttrDialog::accept()
                           tr( "Invalid field name. This field name is reserved and cannot be used." ) );
     return;
   }
+  if ( mNameEdit->text().isEmpty() )
+  {
+    QMessageBox::warning( this, tr( "Warning" ),
+                          tr( "No name specified. Please specify a name to create a new field." ) );
+    return;
+  }
+  if ( mButtonVirtualField->isChecked() && mExpressionWidget->currentField().isEmpty() )
+  {
+    QMessageBox::warning( this, tr( "Warning" ),
+                          tr( "No expression specified. Please enter an expression that will be used to calculate the field values." ) );
+    return;
+  }
   QDialog::accept();
 }