From 792f20b702d54b3c892d95c98a7ec94ba871cde7 Mon Sep 17 00:00:00 2001
From: radiskull <radiskull@9a2b23b2-43f4-4935-bed2-871a584d7cab>
Date: Mon, 12 Dec 2011 18:56:57 +0000
Subject: [PATCH] #83 gefixt und ein kleinen Bug bei der Anzeige von Fehlern
 behoben. (falscher Verweis auf ein Bild)

---
 ag/char_profil.php             |  6 +++---
 ag/include/attackenset.inc.php | 17 +++++++++++++++--
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/ag/char_profil.php b/ag/char_profil.php
index e86e050..aae8110 100644
--- a/ag/char_profil.php
+++ b/ag/char_profil.php
@@ -353,10 +353,10 @@ function eintragen($char_id,$name,$user_ida) {
 	//überprüfen, wie oft welche Attacke benutzt wird, sowie andere Abhängigkeiten
 	$fehler[0] = validateAttackSetTypes($attset_cf);
 	$fehler[1] = validateAttackSetTypes($attset_nf);
-
+	
 	//andere Sachen überprüfen
 	if($bild11 != $_POST['bild11']){
-		$fehler[3] = 'Die URL f&uuml;r das Bild ist ung&uuml;ltig!';
+		$fehler[2] = 'Die URL f&uuml;r das Bild ist ung&uuml;ltig!';
 	}
 
 	//Schauen ob ein Fehler vorhanden ist
@@ -376,7 +376,7 @@ function eintragen($char_id,$name,$user_ida) {
 		echo '<a href=index.php?as=char_profil&char_id='.$char_id.'><div align=center>weiter...</div></a>';
 	} else {
 		displayErrorMessage(NULL, $fehler[0].'<br>'.$fehler[1].'<br>'.$fehler[2], displayHistoryBackLink());
-		exit;
+		return;
 	}
 }
 
diff --git a/ag/include/attackenset.inc.php b/ag/include/attackenset.inc.php
index d369407..51107e3 100644
--- a/ag/include/attackenset.inc.php
+++ b/ag/include/attackenset.inc.php
@@ -97,8 +97,21 @@ function getAttackTypesFromAttackSet($attset) {
 }
 
 function validateAttackSetTypes($attset) {
-	if($attset == NULL) {
-		return 'Es wurden garkeine Attacken ausgewählt!';
+	//the attset is empty or wasn't even initialised
+	if($attset == NULL || count($attset) == 0) {
+		return NULL;
+	}
+	
+	//lets check for id's other then -1 (if every id is -1, it means the array is empty!)
+	$hit = false;
+	for($i = 1; $i < count($attset); $i++) {
+		if($attset[$i] != -1) {
+			$hit = true;
+		}
+	}
+	
+	if(!$hit) {
+		return NULL;
 	}
 
 	$types = getAttackTypesFromAttackSet($attset);