Reorder list of messages in validate_form to correspond order on the form

diff --git a/survey/surveycollector.cgi b/survey/surveycollector.cgi
index 23be6ff3f6632c64bb28357a0e5f325865bdcd4f..6ab3f3345d9377228d315b04ba141f3f68b129f0 100755 (executable)
--- a/survey/surveycollector.cgi
+++ b/survey/surveycollector.cgi
@@ -7,21 +7,21 @@ import os
 
 def validate_form(form):
     messages = []
-    if not 'pers_time' in form or form['pers_time'].value == 'none':
-        messages.append("Please indicate how much time you spend in your personal computing environment.")
-    if not 'man_time' in form or form['man_time'].value == 'none':
-        messages.append("Please indicate how much time you spend in a managed computing environment.")
-    if not 'virt_time' in form or form['virt_time'].value == 'none':
-        messages.append("Please indicate how often you use virtual machines.")
     if not 'bg_datamod' in form or not len(form.getlist('bg_datamod')):
         messages.append("Please indicate want kind of data you are working with.")
     if 'bg_datamod' in form and 'other' in form.getlist('bg_datamod') and not form['bg_datamod_other'].value:
         messages.append("You selected 'Other data modality' but did not specific which one.")
+    if not 'pers_time' in form or form['pers_time'].value == 'none':
+        messages.append("Please indicate how much time you spend in your personal computing environment.")
     if 'pers_maint_time' in form and form['pers_maint_time'].value:
         try:
             t = float(form['pers_maint_time'].value)
         except:
             messages.append("The value you entered as maintenance effort per month needs to be a (floating point) number. For example: 1.2 or 5")
+    if not 'man_time' in form or form['man_time'].value == 'none':
+        messages.append("Please indicate how much time you spend in a managed computing environment.")
+    if not 'virt_time' in form or form['virt_time'].value == 'none':
+        messages.append("Please indicate how often you use virtual machines.")
     return messages