From da9fa893fd717beb5a4353eba4af99dff210afa3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Wed, 31 Oct 2007 16:03:28 +0000
Subject: [PATCH] Neuer Status kann jetzt ueber Rechte Maustaste->"Neuen Status
 hinzufuegen" angelegt werden.

---
 content/ToDo_FASo.html                       | 44 +++++++++++++-------
 content/student/studentDBDML.php             | 38 +++++++++++++----
 content/student/studentdetailoverlay.xul.php | 38 +----------------
 content/student/studentoverlay.js.php        | 28 +++----------
 content/student/studentrolledialog.js.php    | 16 ++++++-
 content/student/studentrolledialog.xul.php   | 12 +++++-
 include/prestudent.class.php                 |  3 ++
 7 files changed, 96 insertions(+), 83 deletions(-)

diff --git a/content/ToDo_FASo.html b/content/ToDo_FASo.html
index f8f2d4d62..a6180c2f7 100644
--- a/content/ToDo_FASo.html
+++ b/content/ToDo_FASo.html
@@ -145,8 +145,7 @@ sichtbar.(CIS)</td>
     <tr>
     </tr>
     <tr class="liste1">
-      <td>Neue Interessentrolle bei bereits vorhandenen PreStudenten
-anlegbar.</td>
+      <td></td>
       <td align="center">med </td>
     </tr>
     <tr>
@@ -241,8 +240,7 @@ Studenten<br>
       <th>Priorit&auml;t</th>
     </tr>
     <tr class="liste0">
-      <td>Beim Starten kommt manchmal folgende Fehlermeldung in der
-Errorconsole: Error: can't decode principals (failure code 8007000e)</td>
+      <td></td>
       <td align="center">low</td>
     </tr>
     <tr>
@@ -315,17 +313,33 @@ Errorconsole: Error: can't decode principals (failure code 8007000e)</td>
 <hr>
 <h2>Erledigte Anforderungen</h2>
 <ul>
-<li>Studienerfolgsbestaetigung: Fehlerhafte Anzeige bei zu langen Lehrveranstaltungsbezeichnungen<br>
-31.10.2007 oesi<br><br></li>
-<li>Beim Anlegen vom 2. Termin wird automatisch ein 1.Termin mit alter Zeugnisnote angelegt.<br>
-31.10.2007 oesi<br><br></li>
-<li>Pr&uuml;fungsnote autom. als Zeugnisnote &uuml;bernehmen.(Nur im FAS, nicht im CIS!)<br>
-31.10.2007 oesi<br><br></li>
-<li>Beim Anlegen neuer Personen ist die Staatsbuergerschaft und Geburtsnation nun standardmäßig Österreich<br>
-30.10.2007 oesi<br><br></li>
-  <li>L&ouml;schen der Gruppenzuteilung zu einer Lehreinheit verhindern
-wenn bereits eine Kreuzerlliste angelegt wurde.<br>
-29.10.2007 oesi<br>
+	<li>Beim Starten kommt manchmal folgende Fehlermeldung in der Errorconsole: Error: can't decode principals (failure code 8007000e)<br>
+		31.10.2007 oesi - Neuinstallation des Seamonkey-Profils beseitigt die Fehlermeldung<br>
+		<br>
+	</li>
+   <li>Neue Interessentrolle bei bereits vorhandenen PreStudenten anlegbar.<br>
+		31.10.2007 oesi - PrestudentTab, im Statusfenster Rechte Maustaste->Neuen Status hinzufuegen<br>
+		<br>
+	</li>
+	<li>Studienerfolgsbestaetigung: Fehlerhafte Anzeige bei zu langen Lehrveranstaltungsbezeichnungen<br>
+		31.10.2007 oesi<br>
+		<br>
+	</li>
+	<li>Beim Anlegen vom 2. Termin wird automatisch ein 1.Termin mit alter Zeugnisnote angelegt.<br>
+		31.10.2007 oesi<br>
+		<br>
+	</li>
+	<li>Pr&uuml;fungsnote autom. als Zeugnisnote &uuml;bernehmen.(Nur im FAS, nicht im CIS!)<br>
+		31.10.2007 oesi<br>
+		<br>
+	</li>
+	<li>Beim Anlegen neuer Personen ist die Staatsbuergerschaft und Geburtsnation nun standardmäßig Österreich<br>
+		30.10.2007 oesi<br>
+		<br>
+	</li>
+	<li>L&ouml;schen der Gruppenzuteilung zu einer Lehreinheit verhindern
+		wenn bereits eine Kreuzerlliste angelegt wurde.<br>
+		29.10.2007 oesi<br>
     <br>
   </li>
   <li>Studienerfolgsbestaetigung ausdrucken fuer alle Semester die der
diff --git a/content/student/studentDBDML.php b/content/student/studentDBDML.php
index 314d8cfd6..d5b457799 100644
--- a/content/student/studentDBDML.php
+++ b/content/student/studentDBDML.php
@@ -612,20 +612,44 @@ if(!$error)
 				
 				if(!$error)
 				{
-					if(!$rolle->load_rolle($_POST['prestudent_id'], $_POST['rolle_kurzbz'], $_POST['studiensemester_old'], $_POST['ausbildungssemester_old']))
+					if(($_POST['studiensemester_old']=='') || (!$rolle->load_rolle($_POST['prestudent_id'], $_POST['rolle_kurzbz'], $_POST['studiensemester_old'], $_POST['ausbildungssemester_old'])))
 					{
-						$errormsg = 'Rolle konnte nicht geladen werden';
-						$return = false;
+						$rolle->new = true;
+						$rolle->insertamum = date('Y-m-d H:i:s');
+						$rolle->insertvon = $user;
+						$rolle->rolle_kurzbz = $_POST['rolle_kurzbz'];
+						
+						if($_POST['rolle_kurzbz']=='Student')
+						{
+							//Die Rolle Student darf nur eingefuegt werden, wenn schon eine Studentenrolle vorhanden ist
+							$qry = "SELECT count(*) as anzahl FROM public.tbl_student WHERE prestudent_id='".addslashes($_POST['prestudent_id'])."'";
+							if($result = pg_query($conn, $qry))
+							{
+								if($row = pg_fetch_object($result))
+								{
+									if($row->anzahl==0)
+									{
+										$error = true;
+										$errormsg = 'Ein Studentenstatus kann hier nur hinzugefuegt werden wenn die Person bereits Student ist. Um einen Bewerber zum Studenten zu machen waehlen Sie bitte unter "Status aendern" den Punkt "Student".';
+										$return = false;
+									}
+								}
+							}								
+						}
 					}
 					else 
-					{				
-						$rolle->ausbildungssemester = $_POST['ausbildungssemester'];
+					{
 						$rolle->ausbildungssemester_old = $_POST['ausbildungssemester_old'];
-						$rolle->studiensemester_kurzbz = $_POST['studiensemester_kurzbz'];
 						$rolle->studiensemester_old = $_POST['studiensemester_old'];
+						$rolle->new = false;
+					}
+								
+					if(!$error)
+					{	
+						$rolle->ausbildungssemester = $_POST['ausbildungssemester'];					
+						$rolle->studiensemester_kurzbz = $_POST['studiensemester_kurzbz'];					
 						$rolle->datum = $_POST['datum'];
 						$rolle->orgform_kurzbz = $_POST['orgform_kurzbz'];
-						$rolle->new = false;
 						
 						if($rolle->save_rolle())
 							$return = true;
diff --git a/content/student/studentdetailoverlay.xul.php b/content/student/studentdetailoverlay.xul.php
index 90811a5af..66eec3c6a 100644
--- a/content/student/studentdetailoverlay.xul.php
+++ b/content/student/studentdetailoverlay.xul.php
@@ -230,6 +230,7 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>';
 <popupset>
 	<popup id="student-prestudent-rolle-tree-popup">
 		<menuitem label="Bearbeiten" oncommand="StudentRolleBearbeiten();" id="student-prestudent-rolle-tree-popup-edit" hidden="false"/>
+		<menuitem label="Neuen Status hinzufuegen" oncommand="StudentRolleAdd();" id="student-prestudent-rolle-tree-popup-add" hidden="false"/>
 		<menuitem label="Entfernen" oncommand="StudentPrestudentRolleDelete();" id="student-prestudent-rolle-tree-popup-delete" hidden="false"/>		
 	</popup>
 </popupset>
@@ -499,43 +500,6 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>';
 							</rule>
 						</template>
 					</tree>
-				<!--<hbox>
-				<menulist id="student-prestudent-rolle-menulist-rolle_kurzbz" disabled="false"
-				          datasources="<?php echo APP_ROOT ?>rdf/rolle.rdf.php"
-			              ref="http://www.technikum-wien.at/rolle/liste" >
-					<template>
-						<menupopup>
-							<menuitem value="rdf:http://www.technikum-wien.at/rolle/rdf#rolle_kurzbz"
-				        		      label="rdf:http://www.technikum-wien.at/rolle/rdf#rolle_kurzbz"
-							  		  uri="rdf:*"/>
-							</menupopup>
-					</template>
-				</menulist>
-				<menulist id="student-prestudent-rolle-menulist-studiensemester" disabled="false"
-				          datasources="<?php echo APP_ROOT ?>rdf/studiensemester.rdf.php"
-			              ref="http://www.technikum-wien.at/studiensemester/liste" >
-					<template>
-						<menupopup>
-							<menuitem value="rdf:http://www.technikum-wien.at/studiensemester/rdf#kurzbz"
-				        		      label="rdf:http://www.technikum-wien.at/studiensemester/rdf#kurzbz"
-							  		  uri="rdf:*"/>
-							</menupopup>
-					</template>
-				</menulist>
-				<menulist id="student-prestudent-rolle-menulist-semester" disabled="false">
-					<menupopup>
-						<menuitem value="1" label="1"/>
-						<menuitem value="2" label="2"/>
-						<menuitem value="3" label="3"/>
-						<menuitem value="4" label="4"/>
-						<menuitem value="5" label="5"/>
-						<menuitem value="6" label="6"/>
-						<menuitem value="7" label="7"/>
-						<menuitem value="8" label="8"/>
-					</menupopup>
-				</menulist>
-				<button id="student-prestudent-rolle-button-save" label="Rolle hinzufuegen" disabled="false" oncommand="StudentRolleAdd()" />
-				</hbox>-->
 			</groupbox>
 			<hbox>
 				<spacer flex="1" />
diff --git a/content/student/studentoverlay.js.php b/content/student/studentoverlay.js.php
index c00233d3a..f48331ea0 100644
--- a/content/student/studentoverlay.js.php
+++ b/content/student/studentoverlay.js.php
@@ -1448,7 +1448,10 @@ function StudentRolleSpeichern(dialog, studiensemester_old, ausbildungssemester_
 {
 	netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
 	prestudent_id = dialog.getElementById('student-rolle-textbox-prestudent_id').value;
-	rolle_kurzbz = dialog.getElementById('student-rolle-textbox-rolle_kurzbz').value;
+	if(studiensemester_old=='')
+		rolle_kurzbz = dialog.getElementById('student-rolle-menulist-rolle_kurzbz').value;
+	else
+		rolle_kurzbz = dialog.getElementById('student-rolle-textbox-rolle_kurzbz').value;
 	studiensemester_kurzbz = dialog.getElementById('student-rolle-menulist-studiensemester').value;
 	ausbildungssemester = dialog.getElementById('student-rolle-menulist-ausbildungssemester').value;
 	datum = dialog.getElementById('student-rolle-datum-datum').value;
@@ -1538,27 +1541,8 @@ function StudentUnterbrecherZuStudent()
 // ****
 function StudentRolleAdd()
 {
-	var rolle_kurzbz = document.getElementById('student-prestudent-rolle-menulist-rolle_kurzbz').value;
-	var studiensemester = document.getElementById('student-prestudent-rolle-menulist-studiensemester').value;
-	var ausbildungssemester = document.getElementById('student-prestudent-rolle-menulist-semester').value;
-	
-	if(rolle_kurzbz=='')
-	{
-		alert('Typ muss ausgewaehlt werden');
-		return false;
-	}
-	if(studiensemester=='')
-	{
-		alert('Studiensemester muss ausgwaehlt werden');
-		return false;
-	}
-	if(ausbildungssemester=='')
-	{
-		alert('Semester muss ausgewaehlt werden');
-		return false;
-	}
-	debug('StudentAddRolle('+rolle_kurzbz+','+ ausbildungssemester+','+ studiensemester+');');
-	//StudentAddRolle(rolle_kurzbz, ausbildungssemester, studiensemester);
+	var prestudent_id = document.getElementById('student-prestudent-textbox-prestudent_id').value;
+	window.open('<?php echo APP_ROOT?>content/student/studentrolledialog.xul.php?prestudent_id='+prestudent_id,"","chrome, status=no, width=500, height=300, centerscreen, resizable");
 }
 
 // ****
diff --git a/content/student/studentrolledialog.js.php b/content/student/studentrolledialog.js.php
index 7fb498af8..b86a8dfa2 100644
--- a/content/student/studentrolledialog.js.php
+++ b/content/student/studentrolledialog.js.php
@@ -38,8 +38,11 @@ function StudentRolleInit(prestudent_id, rolle_kurzbz, studiensemester_kurzbz, a
 {
 	netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
 	
-	if(prestudent_id!='')
+	if(rolle_kurzbz!='')
 	{
+		document.getElementById('student-rolle-grid-row-textbox').hidden=false;
+		document.getElementById('student-rolle-grid-row-menulist').hidden=true;
+		
 		studiensemester_old=studiensemester_kurzbz;
 		ausbildungssemester_old=ausbildungssemester;
 		
@@ -62,7 +65,18 @@ function StudentRolleInit(prestudent_id, rolle_kurzbz, studiensemester_kurzbz, a
 	}
 	else
 	{
+		studiensemester_old='';
+		ausbildungssemester_old='';
+		
+		document.getElementById('student-rolle-grid-row-textbox').hidden=true;
+		document.getElementById('student-rolle-grid-row-menulist').hidden=false;
+
 		//Defaultwerte bei Neuem Datensatz
+		rolle_kurzbz = 'Interessent';
+		studiensemester_kurzbz=window.opener.getStudiensemester();
+		ausbildungssemester='1';
+		datum = '<?php echo date('d.m.Y') ?>';
+		orgform_kurzbz='';
 	}		
 	
 	document.getElementById('student-rolle-textbox-prestudent_id').value=prestudent_id;
diff --git a/content/student/studentrolledialog.xul.php b/content/student/studentrolledialog.xul.php
index 29d6d1f2c..4e5025d29 100644
--- a/content/student/studentrolledialog.xul.php
+++ b/content/student/studentrolledialog.xul.php
@@ -75,10 +75,20 @@ else
 				<column flex="5"/>
 			</columns>
 			<rows>
-				<row>
+				<row id="student-rolle-grid-row-textbox" hidden="false">
 					<label value="Rolle"/>
 					<textbox disabled="true" id="student-rolle-textbox-rolle_kurzbz" />
 				</row>
+				<row id="student-rolle-grid-row-menulist" hidden="true">
+					<label value="Rolle"/>
+					<menulist id="student-rolle-menulist-rolle_kurzbz" disabled="false">
+						<menupopup>
+							<menuitem value="Interessent" label="Interessent"/>
+							<menuitem value="Bewerber" label="Bewerber"/>
+							<menuitem value="Student" label="Student"/>
+						</menupopup>
+					</menulist>
+				</row>
 				<row>
 					<label value="Studiensemester" control="student-rolle-menulist-studiensemester"/>
 					<menulist id="student-rolle-menulist-studiensemester" 
diff --git a/include/prestudent.class.php b/include/prestudent.class.php
index bf18faec6..80bdd03ae 100644
--- a/include/prestudent.class.php
+++ b/include/prestudent.class.php
@@ -54,6 +54,9 @@ class prestudent extends person
 	var $updatevon;
 	var $orgform_kurzbz;
 	
+	var $studiensemester_old='';
+	var $ausbildungssemester_old='';
+	
 	// ErgebnisArray
 	var $result=array();
 	var $num_rows=0;