From 2e3075b8c4ef356e5bd2a0aa7df714e273b922b9 Mon Sep 17 00:00:00 2001
From: Manfred Kindl <kindlm@technikum-wien.at>
Date: Wed, 14 Aug 2019 11:57:12 +0200
Subject: [PATCH] Aktualisieren funktioniert nun wieder
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Studiengänge in Lektorenliste werden richtig sortiert
---
 content/fasoverlay.js.php                  | 32 +++++++++++-----------
 content/lvplanung/stpl-week-overlay.js.php | 12 ++++----
 content/tempusoverlay.js.php               |  5 +++-
 include/mitarbeiter.class.php              |  2 +-
 rdf/mitarbeiter.rdf.php                    |  2 +-
 5 files changed, 27 insertions(+), 26 deletions(-)

diff --git a/content/fasoverlay.js.php b/content/fasoverlay.js.php
index bf0da34ae..9d426d4dc 100644
--- a/content/fasoverlay.js.php
+++ b/content/fasoverlay.js.php
@@ -103,24 +103,24 @@ var LektorTreeListener =
 function LektorTreeSelectMitarbeiter()
 {
 	var tree=document.getElementById('tree-lektor');
-	var items = tree.view.rowCount; //Anzahl der Zeilen ermitteln
-	if(LektorTreeOpenStudiengang!=null)
+	if (tree.view != null)
 	{
-	   	for(var i=0;i<items;i++)
-	   	{
-	   		//Lehreinheit_id der row holen
-			col = tree.columns ? tree.columns["studiengang_kz"] : "studiengang_kz";
-			var studiengang_kz=tree.view.getCellText(i,col);
-			if(studiengang_kz == LektorTreeOpenStudiengang)
-			{
-				tree.view.toggleOpenState(i);
-				break;
+		var items = tree.view.rowCount; //Anzahl der Zeilen ermitteln
+		if (LektorTreeOpenStudiengang != null) {
+			for (var i = 0; i < items; i++) {
+				//Lehreinheit_id der row holen
+				col = tree.columns ? tree.columns["studiengang_kz"] : "studiengang_kz";
+				var studiengang_kz = tree.view.getCellText(i, col);
+				if (studiengang_kz == LektorTreeOpenStudiengang) {
+					tree.view.toggleOpenState(i);
+					break;
+				}
 			}
-	   	}
-	   	//nach dem laden der daten wieder ganz oben im tree positionieren da es sonst vorkommt, dass
-	   	//der scrollbalken unterhalb aller eintraege rutscht und dann nichts mehr im tree sichtbar ist.
-	   	//(funktioniert anscheinend auch nur mit setTimeout)
-	   	window.setTimeout("document.getElementById('tree-lektor').treeBoxObject.scrollToRow(0)",10);
+			//nach dem laden der daten wieder ganz oben im tree positionieren da es sonst vorkommt, dass
+			//der scrollbalken unterhalb aller eintraege rutscht und dann nichts mehr im tree sichtbar ist.
+			//(funktioniert anscheinend auch nur mit setTimeout)
+			window.setTimeout("document.getElementById('tree-lektor').treeBoxObject.scrollToRow(0)", 10);
+		}
 	}
 }
 
diff --git a/content/lvplanung/stpl-week-overlay.js.php b/content/lvplanung/stpl-week-overlay.js.php
index f2e86abb5..beebfa2ef 100644
--- a/content/lvplanung/stpl-week-overlay.js.php
+++ b/content/lvplanung/stpl-week-overlay.js.php
@@ -68,24 +68,22 @@ function onLektorRefresh()
 {
 	netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
 	var treeLektorenTree=document.getElementById('tree-lektor');
+	// Input-Feld leeren
+	document.getElementById('tempus-lektor-filter').value = '';
 	//var datasources=vboxLehrveranstalungPlanung.getAttribute('datasources');
 	var url = '<?php echo APP_ROOT; ?>rdf/mitarbeiter.rdf.php?user=user&'+gettimestamp();
 
 	var rdfService = Components.classes["@mozilla.org/rdf/rdf-service;1"].getService(Components.interfaces.nsIRDFService);
 	var datasource = rdfService.GetDataSource(url);
-	var oldDatasources = treeLektorenTree.GetDataSources();
+	var oldDatasources = treeLektorenTree.database.GetDataSources();
 
 	datasource.QueryInterface(Components.interfaces.nsIRDFRemoteDataSource);
 	datasource.QueryInterface(Components.interfaces.nsIRDFXMLSink);
 
-	treeLektorenTree.database.RemoveDataSource(oldDatasources);
+	treeLektorenTree.database.RemoveDataSource(oldDatasources.getNext());
 	treeLektorenTree.database.AddDataSource(datasource);
-	if(typeof(filter)!='undefined')
-		treeLektorenTree.builder.rebuild();
+	treeLektorenTree.builder.rebuild();
 
-	//vboxLehrveranstalungPlanung.setAttribute('datasources',datasources+"&bla=");
-
-	//datasources="rdf:null" ref="http://www.technikum-wien.at/mitarbeiter/liste"
 }
 
 // Lektorenliste filtern
diff --git a/content/tempusoverlay.js.php b/content/tempusoverlay.js.php
index 1593def08..1b796fada 100644
--- a/content/tempusoverlay.js.php
+++ b/content/tempusoverlay.js.php
@@ -282,7 +282,10 @@ function onLektorSelect(event)
 
 	var stg_idx = tree.view.getParentIndex(tree.currentIndex);
 	var col = tree.columns ? tree.columns["studiengang_kz"] : "studiengang_kz";
-	var stg_kz=tree.view.getCellText(stg_idx,col);
+	if (stg_idx != -1)
+		var stg_kz=tree.view.getCellText(stg_idx,col);
+	else
+		var stg_kz = 0;
 
 	document.getElementById('LehrveranstaltungEditor').setAttribute('stg_kz',stg_kz);
 	document.getElementById('LehrveranstaltungEditor').setAttribute('uid',uid);
diff --git a/include/mitarbeiter.class.php b/include/mitarbeiter.class.php
index b6997d2fa..7073c763a 100644
--- a/include/mitarbeiter.class.php
+++ b/include/mitarbeiter.class.php
@@ -404,7 +404,7 @@ class mitarbeiter extends benutzer
 	 */
 	public function getMitarbeiterStg($lektor=true,$fixangestellt, $stge, $fkt_kurzbz, $order='studiengang_kz, nachname, vorname, kurzbz', $datum_von='', $datum_bis='')
 	{
-		$sql_query='SELECT DISTINCT campus.vw_mitarbeiter.*, studiengang_kz FROM campus.vw_mitarbeiter
+		$sql_query='SELECT DISTINCT campus.vw_mitarbeiter.*, studiengang_kz, tbl_studiengang.typ, tbl_studiengang.kurzbz AS stg_kurzbz FROM campus.vw_mitarbeiter
 					JOIN public.tbl_benutzerfunktion USING (uid) JOIN public.tbl_studiengang USING(oe_kurzbz)
 					WHERE true';
 		if(!is_null($lektor))
diff --git a/rdf/mitarbeiter.rdf.php b/rdf/mitarbeiter.rdf.php
index 819d5ddf0..aa04a3bcb 100644
--- a/rdf/mitarbeiter.rdf.php
+++ b/rdf/mitarbeiter.rdf.php
@@ -166,7 +166,7 @@ if($lehrveranstaltung_id==null && $filter==null && $mitarbeiter_uid==null)
 		{
 			$stge=$bb->getStgKz('admin');
 			$stge=array_merge($stge, $bb->getStgKz('assistenz'));
-			$ma=$mitarbeiter->getMitarbeiterStg($lektor,$fixangestellt,$stge, 'lkt');
+			$ma=$mitarbeiter->getMitarbeiterStg($lektor,$fixangestellt,$stge, 'lkt', 'typ, stg_kurzbz, nachname, vorname, vw_mitarbeiter.kurzbz');
 			$laststg=-1;
 			if(count($ma)>0)
 			{