From 5892496706e5f732f2d67078dc34061537bae735 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Fri, 25 Jan 2019 08:24:05 +0100
Subject: [PATCH] Keine Kollision bei Verschiebung

---
 include/lehrstunde.class.php | 32 +++++++++++++++++++++++---------
 skin/tempus.css              |  2 +-
 2 files changed, 24 insertions(+), 10 deletions(-)

diff --git a/include/lehrstunde.class.php b/include/lehrstunde.class.php
index b3249822b..dd821f3d8 100644
--- a/include/lehrstunde.class.php
+++ b/include/lehrstunde.class.php
@@ -741,20 +741,34 @@ class lehrstunde extends basis_db
 		//Wenn eine Kollisionspruefung auf Studentenebene durchgefuehrt wird, werden die LVB nicht gecheckt
 		if($kollision_student=='false')
 		{
-			$sql_query.=" OR (studiengang_kz=".$this->db_add_param($this->studiengang_kz)." AND semester=".$this->db_add_param($this->sem);
+			// Direkte Gruppen kollidieren nicht
+			$direktgruppe = false;
 			if($this->gruppe_kurzbz!=null && $this->gruppe_kurzbz!='' && $this->gruppe_kurzbz!=' ')
 			{
-				$sql_query.=" OR (gruppe_kurzbz=".$this->db_add_param($this->gruppe_kurzbz).")";
+				$grp_obj = new gruppe();
+				$grp_obj->load($this->gruppe_kurzbz);
+				if($grp_obj->direktinskription)
+				{
+					$direktgruppe = true;
+				}
 			}
-			else
+			if(!$direktgruppe)
 			{
-				if ($this->ver!=null && $this->ver!='' && $this->ver!=' ')
-					$sql_query.=" AND (verband=".$this->db_add_param($this->ver)." OR verband IS NULL OR verband='' OR verband=' ')";
-				if ($this->grp!=null && $this->grp!='' && $this->grp!=' ')
-					$sql_query.=" AND (gruppe=".$this->db_add_param($this->grp)." OR gruppe IS NULL OR gruppe='' OR gruppe=' ')";
-			}
+				$sql_query.=" OR (studiengang_kz=".$this->db_add_param($this->studiengang_kz)." AND semester=".$this->db_add_param($this->sem);
+				if($this->gruppe_kurzbz!=null && $this->gruppe_kurzbz!='' && $this->gruppe_kurzbz!=' ')
+				{
+					$sql_query.=" OR (gruppe_kurzbz=".$this->db_add_param($this->gruppe_kurzbz).")";
+				}
+				else
+				{
+					if ($this->ver!=null && $this->ver!='' && $this->ver!=' ')
+						$sql_query.=" AND (verband=".$this->db_add_param($this->ver)." OR verband IS NULL OR verband='' OR verband=' ')";
+					if ($this->grp!=null && $this->grp!='' && $this->grp!=' ')
+						$sql_query.=" AND (gruppe=".$this->db_add_param($this->grp)." OR gruppe IS NULL OR gruppe='' OR gruppe=' ')";
+				}
 
-			$sql_query.=")";
+				$sql_query.=")";
+			}
 		}
 		$sql_query.=") AND unr!=".$this->db_add_param($this->unr);
 
diff --git a/skin/tempus.css b/skin/tempus.css
index 444ab4140..909ef8fe8 100644
--- a/skin/tempus.css
+++ b/skin/tempus.css
@@ -162,7 +162,7 @@ toolbarbutton.stplweekoverlay-toolbarbutton:hover
 	color: #AAAAAA;
 	visibility:hidden;
 	font-size: small;
-	margin-top: 10px;
+	margin-bottom: 15px;
 }
 
 treechildren::-moz-tree-row(makeItred)