From ca58eba35596519c67bbfd9f39bdaa1701e5cf4b Mon Sep 17 00:00:00 2001
From: Tobias Herre <tobias.herre@wikimedia.de>
Date: Tue, 22 Sep 2020 13:29:10 +0000
Subject: [PATCH] Changed order of checks for empty fields

---
 kaform.js | 32 +++++++++++++++++---------------
 1 file changed, 17 insertions(+), 15 deletions(-)

diff --git a/kaform.js b/kaform.js
index 36dbcce..89c7b33 100644
--- a/kaform.js
+++ b/kaform.js
@@ -327,21 +327,14 @@ function downloadDocument1(){
 
 function downloadDocument(mail)
 {
-/*	if (!check_field("nickname", "Bitte gib deinen Nicknamen an!"))
-		return;
-*/	if (!check_field("realname", "Bitte gib deinen Realnamen an!"))
+	if (!check_field("projectid", "Bitte gib einen Projektnamen/Zweck der Reise an!"))
 		return;
 
-	if (!check_field("email", "Bitte gib deinen E-Mail-Adresse an!"))
-		return;
-	if (!check_field("projectid", "Bitte gib einen Projektnamen/Zweck der Reise an!"))
+	if (!check_field("realname", "Bitte gib deinen Realnamen an!"))
 		return;
 
-	if (countRows()==0){
-		document.getElementById("a_description").focus();
-		showAlert("Bitte f&uuml;ge mind. eine Auslage hinzu!");
+	if (!check_field("email", "Bitte gib deinen E-Mail-Adresse an!"))
 		return;
-	}
 
 	if (banktype == "bank-eu"){
 		iban = document.getElementById('iban');
@@ -369,11 +362,12 @@ function downloadDocument(mail)
 	}
 	
 
-	if (!document.getElementById("agree").checked){
-		document.getElementById("agree").focus();
-		showAlert("Bitte best&auml;tige, dass du alle Angaben nach bestem Wissen und Gewissen gemacht hast!");
+	if (countRows()==0){
+		document.getElementById("a_description").focus();
+		showAlert("Bitte f&uuml;ge mind. eine Auslage hinzu!");
 		return;
-	};
+	}
+
 
 	if (mail){
 		if (!isMailable()){
@@ -387,12 +381,20 @@ function downloadDocument(mail)
 	}
 	else{
 		document.getElementById('sendmail').value="no";
-		
 	}
 
 
 	document.getElementById('banktype').value=banktype;
 	document.getElementById("theform").submit(); 	
+
+	if (!document.getElementById("agree").checked){
+		document.getElementById("agree").focus();
+		showAlert("Bitte best&auml;tige, dass du alle Angaben nach bestem Wissen und Gewissen gemacht hast!");
+		return;
+	};
+
+
+	
 }