From ae758e017f8f9800d8fe8b04d992d6dfbbd3210a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Fri, 28 Aug 2009 08:34:45 +0000
Subject: [PATCH] =?UTF-8?q?-=20Status=20Aufgenommener=20muss=20vorhanden?=
 =?UTF-8?q?=20sein,=20bevor=20ein=20Student=20inskirbiert=20werden=20kann.?=
 =?UTF-8?q?=20StSem=20und=20AusbSem=20m=C3=BCssen=20bei=20Aufgenommener=20?=
 =?UTF-8?q?und=20Bewerber=20gleich=20sein=20-=20Diverse=20Bugfixes?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 cis/private/lehre/softgrid.php             |   2 +-
 cis/private/lvplan/index.html              |   1 -
 content/ToDo_FASo.html                     |  30 ++-
 content/student/studentDBDML.php           | 232 +++++++++++----------
 content/student/studentrolledialog.xul.php |   1 +
 5 files changed, 149 insertions(+), 117 deletions(-)
diff --git a/cis/private/lehre/softgrid.php b/cis/private/lehre/softgrid.php
index 370204c87..d1629c3e3 100644
--- a/cis/private/lehre/softgrid.php
+++ b/cis/private/lehre/softgrid.php
@@ -238,7 +238,7 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes"?>';
 					$cDebug.='<td title="Third">'.$tmp_value['ThirdLevel'].'</td>';
 					$cDebug.='<td title="Fourth">'.$tmp_value['FourthLevel'].'</td>';
 					$cDebug.='<td title="Href">'.$tmp_value['APPHref'].'</td>';
-					$cDebug.='<td title="Icon">'.(!empty($tmp_value['Icon'])?'<img onError="this.onerror=null;this.src=\'blank.png\';" height="16" src="'.$tmp_value['Icon'].'" alt="Icon"  />':'').'</td>';
+					$cDebug.='<td title="Icon">'.(!empty($tmp_value['Icon'])?'<img onError="this.onerror=null;this.src=\'../../../skin/images/blank.gif\';" height="16" src="'.$tmp_value['Icon'].'" alt="Icon"  />':'').'</td>';
 					$cDebug.='<td title="Error">'.$tmp_value['Error'].'</td>';
 				$cDebug.='</tr>';
 			}
diff --git a/cis/private/lvplan/index.html b/cis/private/lvplan/index.html
index a0b8ea7b7..aeee5bbcf 100644
--- a/cis/private/lvplan/index.html
+++ b/cis/private/lvplan/index.html
@@ -2,7 +2,6 @@
 <head>
 	<META http-equiv="refresh" content="0;URL=index.php">
 	<title>Stundenplan</title>
-	<link rel="stylesheet" href="../../skin/style.css.php" type="text/css">
 </head>
 <body>
 	Sie werden nicht weitergeleitet? <a class="Item" href="index.php">(Klick here)</a>
diff --git a/content/ToDo_FASo.html b/content/ToDo_FASo.html
index 3d42b2f91..ebe201a3b 100644
--- a/content/ToDo_FASo.html
+++ b/content/ToDo_FASo.html
@@ -11,7 +11,6 @@
 <h2>Offene Themen für die nächsten FAS-Meetings<br>
 </h2>
 <br>
-<br>
 <ul>
   <li> Protokoll für kommissionelle Prüfung -&gt; Vorlage<br>
   </li>
@@ -325,20 +324,15 @@ alle Lektoren eines Studienganges </td>
       <th>Aufwand [h] </th>
     </tr>
     <tr class="liste0">
-      <td>Beim Inskripieren von Studenten werden immer die Daten des
-Bewerberstatus verwendet (StSem, AusbSem). Wenn ein Status
-Wartender/Aufgenommener mit abweichenden Daten angelegt ist, kann dies
-zu Problemen führen. (falsch generierte Matrikelnummer/UID,
-Duplizierter Wert, etc.)<br>
-      </td>
-      <td align="center">high<br>
+      <td>Zeitweise wird die Gegenbuchung mit dem Betrag 0 gebucht
+anstatt mit dem Differenzbetrag </td>
+      <td align="center"><br>
       </td>
       <td><br>
       </td>
     </tr>
     <tr class="liste1">
-      <td>Zeitweise wird die Gegenbuchung mit dem Betrag 0 gebucht
-anstatt mit dem Differenzbetrag<br>
+      <td><br>
       </td>
       <td align="center"><br>
       </td>
@@ -478,6 +472,22 @@ Vorlage?</td>
 <h2>Erledigte Anforderungen</h2>
 <br>
 <ul>
+  <li>Bewerber können nun nur noch dann inskribiert werden, wenn ein
+"Aufgenommener" Status vorhanden ist. Ausserdem müssen Studiensemester
+und Ausbildungssemester der beiden Stati (Aufgenommener/Bewerber)
+übereinstimmen. Diese Änderung ist nötig, um aussagekräftigere
+Statistiken erstellen zu können.<br>
+28.08.2009<br>
+    <br>
+  </li>
+  <li>Beim Inskripieren von Studenten werden immer die Daten des
+Bewerberstatus verwendet (StSem, AusbSem). Wenn ein Status
+Wartender/Aufgenommener mit abweichenden Daten angelegt ist, kann dies
+zu Problemen führen. (falsch generierte Matrikelnummer/UID,
+Duplizierter Wert, etc.)<br>
+28.08.2009<br>
+    <br>
+  </li>
   <li>Funktionen: Im DropDown für die Institute werden die inaktiven
 Institute nun durchgestrichen dargestellt.<br>
 27.08.2009<br>
diff --git a/content/student/studentDBDML.php b/content/student/studentDBDML.php
index b9044fc1c..43b2ce67e 100644
--- a/content/student/studentDBDML.php
+++ b/content/student/studentDBDML.php
@@ -901,141 +901,163 @@ if(!$error)
 		
 										if(count($hlp->result)>0)
 										{
-											//pruefen ob schon eine Studentenrolle Existiert
-											$hlp1 = new prestudent();
-											$hlp1->getPrestudentRolle($prestudent_id, 'Student', $hlp->result[0]->studiensemester_kurzbz);
-											if(count($hlp1->result)>0)
+											$aufgenommener = new prestudent();
+											$aufgenommener ->getPrestudentRolle($prestudent_id, 'Aufgenommener', null,'datum DESC, insertamum DESC');
+											
+											if(count($aufgenommener->result)>0)
 											{
-												$return = false;
-												$errormsg .= "\n$prestd->vorname $prestd->nachname: Diese Person ist bereits Student";
-												$anzahl_fehler++;
-											}
-											else
-											{
-												$db->db_query('BEGIN;');
-		
-												//Matrikelnummer und UID generieren
-												$matrikelnr = generateMatrikelnummer($prestd->studiengang_kz, $hlp->result[0]->studiensemester_kurzbz);
-												$jahr = substr($matrikelnr,0, 2);
-												$stg = $prestd->studiengang_kz;
-												$stg_obj = new studiengang();
-												$stg_obj->load(ltrim($stg,'0'));
-												$uid = generateUID($stg_obj->kurzbz,$jahr,$stg_obj->typ,$matrikelnr);
-		
-												//Benutzerdatensatz anlegen
-												$benutzer = new benutzer();
-												$benutzer->uid = $uid;
-												$benutzer->person_id = $prestd->person_id;
-												$benutzer->aktiv = true;
-		
-												$qry_alias = "SELECT * FROM public.tbl_benutzer WHERE alias=LOWER('".clean_string($prestd->vorname).".".clean_string($prestd->nachname)."')";
-												$result_alias = $db->db_query($qry_alias);
-												if($db->db_num_rows($result_alias)==0)
-													$benutzer->alias = strtolower(clean_string($prestd->vorname).'.'.clean_string($prestd->nachname));
-												else
-													$benutzer->alias = '';
-		
-												$benutzer->insertamum = date('Y-m-d H:i:s');
-												$benutzer->insertvon = $user;
-		
-												if($benutzer->save(true, false))
+												if($hlp->result[0]->studiensemester_kurzbz==$aufgenommener->result[0]->studiensemester_kurzbz
+												   && $hlp->result[0]->ausbildungssemester==$aufgenommener->result[0]->ausbildungssemester)
 												{
-													//Studentendatensatz anlegen
-													$student = new student();
-													$student->uid = $uid;
-													$student->matrikelnr = $matrikelnr;
-													$student->prestudent_id = $prestd->prestudent_id;
-													$student->studiengang_kz = $prestd->studiengang_kz;
-													$student->semester = $hlp->result[0]->ausbildungssemester;
-													$student->verband = ' ';
-													$student->gruppe = ' ';
-													$student->insertamum = date('Y-m-d H:i:s');
-													$student->insertvon = $user;
-		
-													//Pruefen ob der Lehrverband exisitert, falls nicht dann anlegen
-													$lehrverband = new lehrverband();
-													if(!$lehrverband->load($student->studiengang_kz, $student->semester, $student->verband, $student->gruppe))
+													//pruefen ob schon eine Studentenrolle Existiert
+													$hlp1 = new prestudent();
+													$hlp1->getPrestudentRolle($prestudent_id, 'Student', $hlp->result[0]->studiensemester_kurzbz);
+													if(count($hlp1->result)>0)
 													{
-														$lehrverband->studiengang_kz = $student->studiengang_kz;
-														$lehrverband->semester = $student->semester;
-														$lehrverband->verband = $student->verband;
-														$lehrverband->gruppe = $student->gruppe;
-														$lehrverband->aktiv = true;
-														if(!$lehrverband->save(true))
-														{
-															$error = true; 
-															$errormsg = 'Fehler beim Speichern des Lehrverbandes';
-															$return = false;
-														}
+														$return = false;
+														$errormsg .= "\n$prestd->vorname $prestd->nachname: Diese Person ist bereits Student";
+														$anzahl_fehler++;
 													}
-													
-													if(!$error)
+													else
 													{
-														if($student->save(true, false))
+														$db->db_query('BEGIN;');
+				
+														//Matrikelnummer und UID generieren
+														$matrikelnr = generateMatrikelnummer($prestd->studiengang_kz, $hlp->result[0]->studiensemester_kurzbz);
+														$jahr = substr($matrikelnr,0, 2);
+														$stg = $prestd->studiengang_kz;
+														$stg_obj = new studiengang();
+														$stg_obj->load(ltrim($stg,'0'));
+														$uid = generateUID($stg_obj->kurzbz,$jahr,$stg_obj->typ,$matrikelnr);
+				
+														//Benutzerdatensatz anlegen
+														$benutzer = new benutzer();
+														$benutzer->uid = $uid;
+														$benutzer->person_id = $prestd->person_id;
+														$benutzer->aktiv = true;
+				
+														$qry_alias = "SELECT * FROM public.tbl_benutzer WHERE alias=LOWER('".clean_string($prestd->vorname).".".clean_string($prestd->nachname)."')";
+														$result_alias = $db->db_query($qry_alias);
+														if($db->db_num_rows($result_alias)==0)
+															$benutzer->alias = strtolower(clean_string($prestd->vorname).'.'.clean_string($prestd->nachname));
+														else
+															$benutzer->alias = '';
+				
+														$benutzer->insertamum = date('Y-m-d H:i:s');
+														$benutzer->insertvon = $user;
+				
+														if($benutzer->save(true, false))
 														{
-															//Prestudentrolle hinzugfuegen
-															$rolle = new prestudent();
-															$rolle->prestudent_id = $prestd->prestudent_id;
-															$rolle->status_kurzbz = 'Student';
-															$rolle->studiensemester_kurzbz = $hlp->result[0]->studiensemester_kurzbz;
-															$rolle->ausbildungssemester = $hlp->result[0]->ausbildungssemester;
-															$rolle->orgform_kurzbz = $hlp->result[0]->orgform_kurzbz;
-															$rolle->datum = date('Y-m-d');
-															$rolle->insertamum = date('Y-m-d H:i:s');
-															$rolle->insertvon = $user;
-															$rolle->new = true;
-			
-															if($rolle->save_rolle())
+															//Studentendatensatz anlegen
+															$student = new student();
+															$student->uid = $uid;
+															$student->matrikelnr = $matrikelnr;
+															$student->prestudent_id = $prestd->prestudent_id;
+															$student->studiengang_kz = $prestd->studiengang_kz;
+															$student->semester = $hlp->result[0]->ausbildungssemester;
+															$student->verband = ' ';
+															$student->gruppe = ' ';
+															$student->insertamum = date('Y-m-d H:i:s');
+															$student->insertvon = $user;
+				
+															//Pruefen ob der Lehrverband exisitert, falls nicht dann anlegen
+															$lehrverband = new lehrverband();
+															if(!$lehrverband->load($student->studiengang_kz, $student->semester, $student->verband, $student->gruppe))
 															{
-																//StudentLehrverband anlegen
-																$studentlehrverband = new student();
-																$studentlehrverband->uid = $uid;
-																$studentlehrverband->studiensemester_kurzbz = $hlp->result[0]->studiensemester_kurzbz;
-																$studentlehrverband->studiengang_kz = $prestd->studiengang_kz;
-																$studentlehrverband->semester = $hlp->result[0]->ausbildungssemester;
-																$studentlehrverband->verband = ' ';
-																$studentlehrverband->gruppe = ' ';
-																$studentlehrverband->insertamum = date('Y-m-d H:i:s');
-																$studentlehrverband->insertvon = $user;
-			
-																if($studentlehrverband->save_studentlehrverband(true))
+																$lehrverband->studiengang_kz = $student->studiengang_kz;
+																$lehrverband->semester = $student->semester;
+																$lehrverband->verband = $student->verband;
+																$lehrverband->gruppe = $student->gruppe;
+																$lehrverband->aktiv = true;
+																if(!$lehrverband->save(true))
 																{
-																	$return = true;
-																	$db->db_query('COMMIT;');
+																	$error = true; 
+																	$errormsg = 'Fehler beim Speichern des Lehrverbandes';
+																	$return = false;
+																}
+															}
+															
+															if(!$error)
+															{
+																if($student->save(true, false))
+																{
+																	//Prestudentrolle hinzugfuegen
+																	$rolle = new prestudent();
+																	$rolle->prestudent_id = $prestd->prestudent_id;
+																	$rolle->status_kurzbz = 'Student';
+																	$rolle->studiensemester_kurzbz = $hlp->result[0]->studiensemester_kurzbz;
+																	$rolle->ausbildungssemester = $hlp->result[0]->ausbildungssemester;
+																	$rolle->orgform_kurzbz = $hlp->result[0]->orgform_kurzbz;
+																	$rolle->datum = date('Y-m-d');
+																	$rolle->insertamum = date('Y-m-d H:i:s');
+																	$rolle->insertvon = $user;
+																	$rolle->new = true;
+					
+																	if($rolle->save_rolle())
+																	{
+																		//StudentLehrverband anlegen
+																		$studentlehrverband = new student();
+																		$studentlehrverband->uid = $uid;
+																		$studentlehrverband->studiensemester_kurzbz = $hlp->result[0]->studiensemester_kurzbz;
+																		$studentlehrverband->studiengang_kz = $prestd->studiengang_kz;
+																		$studentlehrverband->semester = $hlp->result[0]->ausbildungssemester;
+																		$studentlehrverband->verband = ' ';
+																		$studentlehrverband->gruppe = ' ';
+																		$studentlehrverband->insertamum = date('Y-m-d H:i:s');
+																		$studentlehrverband->insertvon = $user;
+					
+																		if($studentlehrverband->save_studentlehrverband(true))
+																		{
+																			$return = true;
+																			$db->db_query('COMMIT;');
+																		}
+																		else
+																		{
+																			$errormsg .= "\n$prestd->vorname $prestd->nachname: Fehler beim Speichern des Studentlehrverbandes: ".$studentlehrverband->errormsg;
+																			$return = false;
+																			$anzahl_fehler++;
+																			$db->db_query('ROLLBACK;');
+																		}
+																	}
+																	else
+																	{
+																		$errormsg .= "\n$prestd->vorname $prestd->nachname: Fehler beim Speichern des Rolle: ".$rolle->errormsg;
+																		$return = false;
+																		$anzahl_fehler++;
+																		$db->db_query('ROLLBACK;');
+																	}
 																}
 																else
 																{
-																	$errormsg .= "\n$prestd->vorname $prestd->nachname: Fehler beim Speichern des Studentlehrverbandes: ".$studentlehrverband->errormsg;
+																	$errormsg .= "\n$prestd->vorname $prestd->nachname: Fehler beim Speichern des Studenten: ".$student->errormsg;
 																	$return = false;
 																	$anzahl_fehler++;
 																	$db->db_query('ROLLBACK;');
 																}
 															}
-															else
-															{
-																$errormsg .= "\n$prestd->vorname $prestd->nachname: Fehler beim Speichern des Rolle: ".$rolle->errormsg;
-																$return = false;
-																$anzahl_fehler++;
-																$db->db_query('ROLLBACK;');
-															}
 														}
 														else
 														{
-															$errormsg .= "\n$prestd->vorname $prestd->nachname: Fehler beim Speichern des Studenten: ".$student->errormsg;
+															$errormsg .= "\n$prestd->vorname $prestd->nachname $matrikelnr: Fehler beim Speichern des Benutzers: ".$benutzer->errormsg;
 															$return = false;
 															$anzahl_fehler++;
 															$db->db_query('ROLLBACK;');
 														}
 													}
 												}
-												else
+												else 
 												{
-													$errormsg .= "\n$prestd->vorname $prestd->nachname $matrikelnr: Fehler beim Speichern des Benutzers: ".$benutzer->errormsg;
 													$return = false;
+													$errormsg .= "\n$prestd->vorname $prestd->nachname: Das Studiensemester oder Ausbildungsemester des Berwerberstatus und des Aufgenommenenstatus passen nicht überein";
 													$anzahl_fehler++;
-													$db->db_query('ROLLBACK;');
 												}
 											}
+											else 
+											{
+												$return = false;
+												$errormsg .= "\n$prestd->vorname $prestd->nachname: Die Person muss zuerst Aufgenommener sein bevor Sie zum Studenten gemacht werden kann";
+												$anzahl_fehler++;
+											}
 										}
 										else
 										{
diff --git a/content/student/studentrolledialog.xul.php b/content/student/studentrolledialog.xul.php
index 695ae4891..f279b34c7 100644
--- a/content/student/studentrolledialog.xul.php
+++ b/content/student/studentrolledialog.xul.php
@@ -97,6 +97,7 @@ $db = new basis_db();
 						<menupopup>
 							<menuitem value="Interessent" label="Interessent"/>
 							<menuitem value="Bewerber" label="Bewerber"/>
+							<menuitem value="Aufgenommener" label="Aufgenommener"/>
 							<menuitem value="Student" label="Student"/>
 							<menuitem value="Unterbrecher" label="Unterbrecher"/>
 							<menuitem value="Diplomand" label="Diplomand"/>
