From ef7bb766e109b0e47727849611a33066149dcb75 Mon Sep 17 00:00:00 2001
From: kindlm <kindlm@technikum-wien.at>
Date: Fri, 16 Oct 2015 16:40:35 +0200
Subject: [PATCH] =?UTF-8?q?R=C3=BCckg=C3=A4ngig=20OE=20zu=20Funktion-Berec?=
 =?UTF-8?q?htigung=20speichern?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Rückgängig Commit f40a0eef14055e090a87097d2b1d1c0095ee3502
Wenn eine OE bei der Berechtigung einer Funktion gespeichert wird, wird
diese genommen, ansonsten die von der Benutzerfunktion.
---
 include/benutzerberechtigung.class.php        |  6 +-
 .../benutzerberechtigung_details.php          | 58 +++++++++++--------
 2 files changed, 37 insertions(+), 27 deletions(-)

diff --git a/include/benutzerberechtigung.class.php b/include/benutzerberechtigung.class.php
index e90b49e8d..206d7a222 100644
--- a/include/benutzerberechtigung.class.php
+++ b/include/benutzerberechtigung.class.php
@@ -169,11 +169,11 @@ class benutzerberechtigung extends basis_db
 			return false;
 		}
 		
-		/*if($this->funktion_kurzbz!='' && $this->oe_kurzbz!='') Auskommentiert von kindlm am 28.08.2015. Wenn beim Recht eine OE angegeben wird, wird diese genommen, ansonsten die von der Funktion.  
+		if($this->funktion_kurzbz!='' && $this->oe_kurzbz!='')
 		{
 			$this->errormsg = 'Wenn eine Funktion_kurzbz angegeben wird, darf keine Organisationseinheit eingetragen sein';
 			return false;
-		}*/
+		}
 		
 		if($this->art=='')
 		{
@@ -389,7 +389,7 @@ class benutzerberechtigung extends basis_db
 				SELECT 
 					benutzerberechtigung_id, tbl_benutzerfunktion.uid, tbl_benutzerrolle.funktion_kurzbz,
 					tbl_benutzerrolle.rolle_kurzbz, tbl_benutzerrolle.berechtigung_kurzbz, tbl_benutzerrolle.art, tbl_benutzerrolle.art art1,
-					COALESCE (tbl_benutzerrolle.oe_kurzbz,tbl_benutzerfunktion.oe_kurzbz) AS oe_kurzbz, tbl_benutzerrolle.studiensemester_kurzbz, tbl_benutzerrolle.start,
+					tbl_benutzerfunktion.oe_kurzbz, tbl_benutzerrolle.studiensemester_kurzbz, tbl_benutzerrolle.start,
 					tbl_benutzerrolle.ende, tbl_benutzerrolle.negativ, tbl_benutzerrolle.updateamum, tbl_benutzerrolle.updatevon,
 					tbl_benutzerrolle.insertamum, tbl_benutzerrolle.insertvon,tbl_benutzerrolle.kostenstelle_id,tbl_benutzerrolle.anmerkung
 				FROM 
diff --git a/vilesci/stammdaten/benutzerberechtigung_details.php b/vilesci/stammdaten/benutzerberechtigung_details.php
index 4eaa12a15..6a57cd1ef 100644
--- a/vilesci/stammdaten/benutzerberechtigung_details.php
+++ b/vilesci/stammdaten/benutzerberechtigung_details.php
@@ -354,22 +354,27 @@ if (isset($_REQUEST['uid']) || isset($_REQUEST['funktion_kurzbz']))
 			$htmlstr .= "		<td name='td_$b->benutzerberechtigung_id'><input id='art_$b->benutzerberechtigung_id' type='text' name='art' value='".$b->art."' size='5' maxlength='5' onChange='validateArt(\"art_$b->benutzerberechtigung_id\"); markier(\"td_".$b->benutzerberechtigung_id."\")'></td>\n";
 			
 			//Organisationseinheit
-			$htmlstr .= "		<td class='oe_column' name='td_$b->benutzerberechtigung_id'><select id='oe_".$b->benutzerberechtigung_id."' name='oe_kurzbz' ".($b->kostenstelle_id!=''?'disabled':'')." onchange='".($funktion_kurzbz!=''?"alert(\"Achtung! Durch diese Auswahl wird die Organisationseinheit der Funktion überschrieben!\");":"")."' onchange='markier(\"td_".$b->benutzerberechtigung_id."\")' style='width: 200px;'>\n";
-			$htmlstr .= "		<option value='' onclick='enable(\"kostenstelle_".$b->benutzerberechtigung_id."\");'>-- Alle --</option>\n";
-			
-			foreach ($oe->result as $oekey)
+			if($funktion_kurzbz!='')
+				$htmlstr .= "		<td class='oe_column' name='td_$b->benutzerberechtigung_id'>OE aus MA-Funktion</td>\n";
+			else 
 			{
-				if ($b->oe_kurzbz == $oekey->oe_kurzbz && $b->oe_kurzbz != null)
-					$sel = " selected";
-				else
-					$sel = "";
-				if(!$oekey->aktiv)
-					$class='class="inactive"';
-				else
-					$class='';
-				$htmlstr .= "	<option value='".$oekey->oe_kurzbz."' ".$sel." ".$class." onclick='disable(\"kostenstelle_".$b->benutzerberechtigung_id."\");'>".$oekey->organisationseinheittyp_kurzbz.' '.$oekey->bezeichnung.'</option>';
+				$htmlstr .= "		<td class='oe_column' name='td_$b->benutzerberechtigung_id'><select id='oe_".$b->benutzerberechtigung_id."' name='oe_kurzbz' ".($b->kostenstelle_id!=''?'disabled':'')." onchange='markier(\"td_".$b->benutzerberechtigung_id."\")' style='width: 200px;'>\n";
+				$htmlstr .= "		<option value='' onclick='enable(\"kostenstelle_".$b->benutzerberechtigung_id."\");'>-- Alle --</option>\n";
+				
+				foreach ($oe->result as $oekey)
+				{
+					if ($b->oe_kurzbz == $oekey->oe_kurzbz && $b->oe_kurzbz != null)
+						$sel = " selected";
+					else
+						$sel = "";
+					if(!$oekey->aktiv)
+						$class='class="inactive"';
+					else
+						$class='';
+					$htmlstr .= "	<option value='".$oekey->oe_kurzbz."' ".$sel." ".$class." onclick='disable(\"kostenstelle_".$b->benutzerberechtigung_id."\");'>".$oekey->organisationseinheittyp_kurzbz.' '.$oekey->bezeichnung.'</option>';
+				}
+				$htmlstr .= "		</select></td>\n";
 			}
-			$htmlstr .= "		</select></td>\n";
 			
 			//Kostenstelle
 			$htmlstr .= "		<td class='ks_column' name='td_$b->benutzerberechtigung_id'><select id='kostenstelle_".$b->benutzerberechtigung_id."'name='kostenstelle_id' ".($b->oe_kurzbz!=''?'disabled':'')." onchange='markier(\"td_".$b->benutzerberechtigung_id."\")' style='width: 200px;'>\n";
@@ -500,18 +505,23 @@ if (isset($_REQUEST['uid']) || isset($_REQUEST['funktion_kurzbz']))
 	$htmlstr .= "		<td style='padding-top: 15px;'><input id='art_neu' type='text' name='art' value='' size='5' maxlength='5' onBlur='checkrequired(\"art_neu\")' onChange='validateArt(\"art_neu\")' placeholder='suid'></td>\n";
 	
 	//Organisationseinheit
-	$htmlstr .= "		<td class='oe_column' style='padding-top: 15px;'><select id='oe_kurzbz_neu' name='oe_kurzbz' onchange='".($funktion_kurzbz!=''?"alert(\"Achtung! Durch diese Auswahl wird die Organisationseinheit der Funktion überschrieben!\");":"")."' onchange='markier(\"neu\")' style='width: 200px;'>\n";
-	$htmlstr .= "			<option value='' onclick='enable(\"kostenstelle_neu\");'>-- Alle --</option>\n";
-	
-	foreach ($oe->result as $oekey)
+	if($funktion_kurzbz!='')
+		$htmlstr .= "		<td class='oe_column' style='padding-top: 15px;'>OE aus MA-Funktion</td>\n";
+	else 
 	{
-		if(!$oekey->aktiv)
-				$class='class="inactive"';
-			else
-				$class='';
-		$htmlstr .= "				<option value='".$oekey->oe_kurzbz."' ".$class." onclick='disable(\"kostenstelle_neu\");'>".$oekey->organisationseinheittyp_kurzbz.' '.$oekey->bezeichnung.'</option>';
+		$htmlstr .= "		<td class='oe_column' style='padding-top: 15px;'><select id='oe_kurzbz_neu' name='oe_kurzbz' onchange='markier(\"neu\")' style='width: 200px;'>\n";
+		$htmlstr .= "			<option value='' onclick='enable(\"kostenstelle_neu\");'>-- Alle --</option>\n";
+		
+		foreach ($oe->result as $oekey)
+		{
+			if(!$oekey->aktiv)
+					$class='class="inactive"';
+				else
+					$class='';
+			$htmlstr .= "				<option value='".$oekey->oe_kurzbz."' ".$class." onclick='disable(\"kostenstelle_neu\");'>".$oekey->organisationseinheittyp_kurzbz.' '.$oekey->bezeichnung.'</option>';
+		}
+		$htmlstr .= "		</select></td>\n";
 	}
-	$htmlstr .= "		</select></td>\n";
 	
 	//Kostenstelle
 	$htmlstr .= "		<td class='ks_column' style='padding-top: 15px;'><select id='kostenstelle_neu' name='kostenstelle_id' onchange='markier(\"".(isset($b->benutzerberechtigung_id)?$b->benutzerberechtigung_id:'')."\")' style='width: 200px;'>\n";