From 344dd2e6a239a681c5dc1f338b5aabd4c414ca19 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Thu, 7 Aug 2008 15:19:49 +0000
Subject: [PATCH]

---
 cis/ToDo_CIS.html                |   8 ++
 content/student/studentDBDML.php | 191 +++++++++++++++++--------------
 include/betriebsmittel.class.php |   2 +-
 3 files changed, 111 insertions(+), 90 deletions(-)

diff --git a/cis/ToDo_CIS.html b/cis/ToDo_CIS.html
index 6d0a5e0d4..5abd50357 100644
--- a/cis/ToDo_CIS.html
+++ b/cis/ToDo_CIS.html
@@ -68,6 +68,14 @@ kann selber die Auswertung abfragen.</span><br>
   </li>
 </ul>
 <br>
+<h2>Urlaubstool</h2>
+<ul>
+  <li><span style="font-weight: bold; color: rgb(51, 255, 51);"><font
+ color="#000000"><b>Doppelt verbuchte bzw &uuml;berlappende Urlaube
+sollten nicht mehrfach vom Urlaubsanspruch abgezogen werden.</b></font></span><br>
+  </li>
+</ul>
+<br>
 <h2>Sonstiges</h2>
 <ul>
   <li>HTML-Beschreibung fuer RDF Files unter
diff --git a/content/student/studentDBDML.php b/content/student/studentDBDML.php
index 540ca5e60..97be85f9b 100644
--- a/content/student/studentDBDML.php
+++ b/content/student/studentDBDML.php
@@ -1652,110 +1652,123 @@ if(!$error)
 		{
 			$bm = new betriebsmittel($conn, null, true);
 
-			//Nachschauen ob dieses Betriebsmittel schon existiert
-			if($bm->getBetriebsmittel($_POST['betriebsmitteltyp'],$_POST['nummerold']))
+			//Das speichern von Zutrittskarten ohne Nummern verhindern
+			if($_POST['betriebsmitteltyp']=='Zutrittskarte' && $_POST['nummer']=='')
 			{
-				if(count($bm->result)>0)
+				$error = true;
+				$return = false;
+				$errormsg = 'Eine Zutrittskarte muss eine Nummer haben. Um die Zuordnung zu dieser Karte zu loeschen entfernen Sie bitte den ganzen Datensatz';
+			}
+			else 
+			{
+				//Nachschauen ob dieses Betriebsmittel schon existiert
+				if($bm->getBetriebsmittel($_POST['betriebsmitteltyp'],$_POST['nummerold']))
 				{
-					//Wenn die Nummer gleich bleibt dann die alte ID verwenden da es
-					//unterschiedliche Schluessel gibt die die gleiche nummer haben ?!?
-					if($_POST['nummer']==$_POST['nummerold'])
+					if(count($bm->result)>0)
 					{
-						$betriebsmittel_id = $_POST['betriebsmittel_id'];
-					}
-					else 
-						$betriebsmittel_id = $bm->result[0]->betriebsmittel_id;
-					//Wenn ein Eintrag gefunden wurde, dann wird die Beschreibung aktualisiert
-					if($bm->load($betriebsmittel_id))
-					{
-						$bm->beschreibung = $_POST['beschreibung'];
-						$bm->nummer = $_POST['nummer'];
-						if(!$bm->save(false))
+						//Wenn die Nummer gleich bleibt dann die alte ID verwenden da es
+						//unterschiedliche Schluessel gibt die die gleiche nummer haben ?!?
+						if($_POST['nummer']==$_POST['nummerold'])
+						{
+							$betriebsmittel_id = $_POST['betriebsmittel_id'];
+						}
+						else 
+							$betriebsmittel_id = $bm->result[0]->betriebsmittel_id;
+						//Wenn ein Eintrag gefunden wurde, dann wird die Beschreibung aktualisiert
+						if($bm->load($betriebsmittel_id))
+						{
+							$bm->beschreibung = $_POST['beschreibung'];
+							$bm->nummer = $_POST['nummer'];
+							if(!$bm->save(false))
+							{
+								$return = false;
+								$error = true;
+								$errormsg = 'Fehler beim Speichern des Betriebsmittels';
+							}
+						}
+						else
 						{
 							$return = false;
 							$error = true;
-							$errormsg = 'Fehler beim Speichern des Betriebsmittels';
+							$errormsg = 'Gefundener Eintrag konnte nicht geladen werden!?!?';
 						}
 					}
 					else
 					{
-						$return = false;
-						$error = true;
-						$errormsg = 'Gefundener Eintrag konnte nicht geladen werden!?!?';
+						//Wenn kein Eintrag gefunden wurde, dann wird ein neuer Eintrag angelegt
+						$bm->betriebsmitteltyp = $_POST['betriebsmitteltyp'];
+						$bm->nummer = $_POST['nummer'];
+						$bm->beschreibung = $_POST['beschreibung'];
+						$bm->reservieren = false;
+						$bm->ort_kurzbz = null;
+						$bm->insertamum = date('Y-m-d H:i:s');
+						$bm->insertvon = $user;
+	
+						if($bm->save(true))
+						{
+							$betriebsmittel_id = $bm->betriebsmittel_id;
+						}
+						else
+						{
+							$error = true;
+							$return = false;
+							$errormsg = 'Fehler beim Anlegen des Betriebsmittels';
+						}
+					}
+	
+					//Zuordnung Betriebsmittel-Person anlegen
+					$bmp = new betriebsmittelperson($conn, null, null, true);
+					if($_POST['neu']!='true')
+					{
+						if($bmp->load($betriebsmittel_id, $_POST['person_id']))
+						{
+							$bmp->updateamum = date('Y-m-d H:i:s');
+							$bmp->updatevon = $user;
+							$bmp->new = false;
+						}
+						else
+						{
+							//$error = true;
+							//$return = false;
+							//$errormsg = 'Fehler beim Laden der Betriebmittelperson Zuordnung'.$betriebsmittel_id.'-'.$_POST['person_id'];
+							$bmp->insertamum = date('Y-m-d H:i:s');
+							$bmp->insertvon = $user;
+							$bmp->new = true;
+						}
+					}
+					else
+					{
+						$bmp->insertamum = date('Y-m-d H:i:s');
+						$bmp->insertvon = $user;
+						$bmp->new = true;
+					}
+	
+					if(!$error)
+					{
+						$bmp->person_id = $_POST['person_id'];
+						$bmp->betriebsmittel_id=$betriebsmittel_id;
+						$bmp->anmerkung = $_POST['anmerkung'];
+						$bmp->kaution = trim(str_replace(',','.',$_POST['kaution']));
+						$bmp->ausgegebenam = $_POST['ausgegebenam'];
+						$bmp->retouram = $_POST['retouram'];
+	
+						if($bmp->save())
+						{
+							$return = true;
+							$data = $betriebsmittel_id;
+						}
+						else
+						{
+							$return = false;
+							$errormsg = $bmp->errormsg;
+						}
 					}
 				}
 				else
 				{
-					//Wenn kein Eintrag gefunden wurde, dann wird ein neuer Eintrag angelegt
-					$bm->betriebsmitteltyp = $_POST['betriebsmitteltyp'];
-					$bm->nummer = $_POST['nummer'];
-					$bm->beschreibung = $_POST['beschreibung'];
-					$bm->reservieren = false;
-					$bm->ort_kurzbz = null;
-					$bm->insertamum = date('Y-m-d H:i:s');
-					$bm->insertvon = $user;
-
-					if($bm->save(true))
-					{
-						$betriebsmittel_id = $bm->betriebsmittel_id;
-					}
-					else
-					{
-						$error = true;
-						$return = false;
-						$errormsg = 'Fehler beim Anlegen des Betriebsmittels';
-					}
+					$errormsg = 'Fehler:'.$bm->errormsg;
+					$return = false;
 				}
-
-				//Zuordnung Betriebsmittel-Person anlegen
-				$bmp = new betriebsmittelperson($conn, null, null, true);
-				if($_POST['neu']!='true')
-				{
-					if($bmp->load($betriebsmittel_id, $_POST['person_id']))
-					{
-						$bmp->updateamum = date('Y-m-d H:i:s');
-						$bmp->updatevon = $user;
-						$bmp->new = false;
-					}
-					else
-					{
-						$error = true;
-						$return = false;
-						$errormsg = 'Fehler beim Laden der Betriebmittelperson Zuordnung'.$betriebsmittel_id.'-'.$_POST['person_id'];
-					}
-				}
-				else
-				{
-					$bmp->insertamum = date('Y-m-d H:i:s');
-					$bmp->insertvon = $user;
-					$bmp->new = true;
-				}
-
-				if(!$error)
-				{
-					$bmp->person_id = $_POST['person_id'];
-					$bmp->betriebsmittel_id=$betriebsmittel_id;
-					$bmp->anmerkung = $_POST['anmerkung'];
-					$bmp->kaution = trim(str_replace(',','.',$_POST['kaution']));
-					$bmp->ausgegebenam = $_POST['ausgegebenam'];
-					$bmp->retouram = $_POST['retouram'];
-
-					if($bmp->save())
-					{
-						$return = true;
-						$data = $betriebsmittel_id;
-					}
-					else
-					{
-						$return = false;
-						$errormsg = $bmp->errormsg;
-					}
-				}
-			}
-			else
-			{
-				$errormsg = 'Fehler:'.$bm->errormsg;
-				$return = false;
 			}
 		}
 	}
diff --git a/include/betriebsmittel.class.php b/include/betriebsmittel.class.php
index 874fe7595..ad48ac7d7 100644
--- a/include/betriebsmittel.class.php
+++ b/include/betriebsmittel.class.php
@@ -272,7 +272,7 @@ class betriebsmittel
 		}
 		else 
 		{
-			$this->errormsg = 'Fehler beim laden der Daten';
+			$this->errormsg = 'Fehler beim Laden der Daten';
 			return false;
 		}
 	}