Browse Source

Ammendment to previous commit

Felix Turowsky 1 year ago
parent
commit
c40b89872e
2 changed files with 7 additions and 3 deletions
  1. 4 1
      src/gui/dialogues/newflightdialog.cpp
  2. 3 2
      src/gui/dialogues/newflightdialog.h

+ 4 - 1
src/gui/dialogues/newflightdialog.cpp

@@ -396,12 +396,15 @@ void NewFlightDialog::informUserAboutMissingItems()
         }
     }
 
+    if(missing_items.isEmpty()) {
+        return;
+    }
+
     INFO(tr("Not all mandatory entries are valid.<br>"
             "The following item(s) are empty or invalid:"
             "<br><br><center><b>%1</b></center><br>"
             "Please go back and fill in the required data."
             ).arg(missing_items));
-    return;
 }
 
 /*!

+ 3 - 2
src/gui/dialogues/newflightdialog.h

@@ -25,13 +25,11 @@
 #include <QBitArray>
 
 #include "QtWidgets/qcalendarwidget.h"
-#include "src/database/database.h"
 #include "src/database/flightentry.h"
 #include "src/gui/dialogues/entryeditdialog.h"
 #include "src/gui/verification/userinput.h"
 #include "src/opl.h"
 #include "src/gui/verification/validationstate.h"
-#include "src/classes/date.h"
 
 namespace Ui {
 class NewFlightDialog;
@@ -177,6 +175,9 @@ private:
     bool userWantsToAddNewEntry(OPL::DbTable table);
 
 
+    /*!
+     * \brief informUserAboutMissingItems - collect information about missing mandatory items and display a messagebox to the user
+     */
     void informUserAboutMissingItems();
     bool pilotFunctionsInvalid();
     bool duplicateNamesPresent();