Fixes #36

config/Windows/newbatch.xml

@@ -495,6 +495,20 @@
             var approval = findChildObject(this, 'approval');
             approval.checked = true;
             var target = findChildObject(this, 'target');
+            var checkSubmitEnable = function () {
+                if(roasted.currentIndex > 0) {
+                    if(timefield.text.length > 0) {
+                        if(duration.text.length > 0) {
+                            if(batch.tempData.length > 0) {
+                                if(green.text.length > 0) {
+                                    return true;
+                                }
+                            }
+                        }
+                    }
+                }
+                return false;
+            }
             submitbutton.clicked.connect(function() {
                 var proceed = false;
                 if(validateCapacity()) {
@@ -503,7 +517,11 @@
                     proceed = displayWarning("Suspicious Input", "Entered green coffee weight exceeds maximum batch size. Continue?");
                 }
                 if(proceed) {
-                    doSubmit();
+                    if(checkSubmitEnable()) {
+                        doSubmit();
+                    } else {
+                        displayError("Incomplete Input", "Some required information is not available. Please check inputs and try again.");
+                    }
                 }
             });
             var doSubmit = function() {

src/typica.w

@@ -937,12 +937,15 @@ message and return the selected action.
 
 @<Function prototypes for scripting@>=
 QScriptValue displayWarning(QScriptContext *context, QScriptEngine *engine);
+QScriptValue displayError(QScriptContext *context, QScriptEngine *engine);
 
 @ This function is exposed to the host environment.
 
 @<Set up the scripting engine@>=
 constructor = engine->newFunction(displayWarning);
 engine->globalObject().setProperty("displayWarning", constructor);
+constructor = engine->newFunction(displayError);
+engine->globalObject().setProperty("displayError", constructor);
 
 @ The function takes some arguments.
 
@@ -959,6 +962,13 @@ QScriptValue displayWarning(QScriptContext *context, QScriptEngine *)
     return QScriptValue(false);
 }
 
+QScriptValue displayError(QScriptContext *context, QScriptEngine *)
+{
+    QMessageBox::critical(NULL, argument<QString>(0, context),
+                          argument<QString>(1, context));
+    return QScriptValue();
+}
+
 @* Scripting QMainWindow.
 
 \noindent Rather than directly exposing |QMainWindow| to the scripting engine,