From af07be5df828821c39d747d758ab033871442e24 Mon Sep 17 00:00:00 2001
From: Nikolaus Krondraf <krondraf@technikum-wien.at>
Date: Thu, 27 Nov 2014 12:07:57 +0000
Subject: [PATCH] Fehlerbehebungen und Layoutanpassung

---
 .../lehrveranstaltungdetailoverlay.xul.php    | 19 ++++++++++++++-----
 .../lvplanung/lehrveranstaltungoverlay.js.php | 12 ++++++++----
 2 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/content/lvplanung/lehrveranstaltungdetailoverlay.xul.php b/content/lvplanung/lehrveranstaltungdetailoverlay.xul.php
index 104f75fa6..fdc6ca142 100644
--- a/content/lvplanung/lehrveranstaltungdetailoverlay.xul.php
+++ b/content/lvplanung/lehrveranstaltungdetailoverlay.xul.php
@@ -46,7 +46,11 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>';
 			<menuitem label="Entfernen" oncommand="LeMitarbeiterDel();" />
 		</menupopup>
 	</popupset>
-
+	<popupset>
+		<menupopup id="lehrveranstaltung-lvangebot-tree-popup">
+			<menuitem label="Entfernen" oncommand="LvAngebotGruppeDel();" />
+		</menupopup>
+	</popupset>
 	<!-- Hidden Fields -->
 	<vbox hidden="true">
 		<grid flex="1" style="padding:5px;">
@@ -412,7 +416,7 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>';
 					ref="http://www.technikum-wien.at/lvangebot/liste"
 					onselect="LvAngebotGruppeAuswahl();" flags="dont-build-content"
 					style="border: 1px solid black;"
-					context="lehrveranstaltung-lektor-tree-popup"
+					context="lehrveranstaltung-lvangebot-tree-popup"
 					onkeypress="LvAngebotTreeKeyPress(event)"
 			>
 				<treecols>
@@ -469,6 +473,12 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>';
 					<column></column>
 				</columns>
 				<rows>
+					<row>
+						<hbox align="left">
+							<button id="lehrveranstaltung-lvangebot-button-new" label="neuer Eintrag" oncommand="LvAngebotNew();" width="130"/>
+						</hbox>
+						<spacer flex="1"/>
+					</row>
 					<row>
 						<label value="Gruppe" />
 						<menulist id="lehrveranstaltung-lvangebot-textbox-gruppe"
@@ -509,9 +519,8 @@ echo '<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>';
 					</row>
 					<row>
 						<spacer flex="1" />
-						<hbox>
-							<button id="lehrveranstaltung-lvangebot-button-save" label="speichern" oncommand="LvAngebotGruppeSave();" flex="3"/>
-							<button id="lehrveranstaltung-lvangebot-button-new" label="neuer Eintrag" oncommand="LvAngebotNew();" flex="1"/>
+						<hbox align="right">
+							<button id="lehrveranstaltung-lvangebot-button-save" label="speichern" oncommand="LvAngebotGruppeSave();" width="130"/>
 						</hbox>
 					</row>
 				</rows>
diff --git a/content/lvplanung/lehrveranstaltungoverlay.js.php b/content/lvplanung/lehrveranstaltungoverlay.js.php
index 91a76ad2a..18bd3f234 100644
--- a/content/lvplanung/lehrveranstaltungoverlay.js.php
+++ b/content/lvplanung/lehrveranstaltungoverlay.js.php
@@ -1993,7 +1993,13 @@ function LvAngebotGruppeSave()
 {
 	var tree = document.getElementById('lehrveranstaltung-tree');
 	var col = tree.columns ? tree.columns["lehrveranstaltung-treecol-lehrveranstaltung_id"] : "lehrveranstaltung-treecol-lehrveranstaltung_id";
-
+	
+	if(tree.currentIndex == -1)
+	{
+		alert('Bitte zuerst eine Lehrveranstaltung auswaehlen');
+		return false;
+	}
+	
 	//Werte holen
 	var lehrveranstaltung_id = tree.view.getCellText(tree.currentIndex,col);
 	var neue_gruppe = document.getElementById('lehrveranstaltung-lvangebot-checkbox-gruppe').checked;
@@ -2224,7 +2230,5 @@ function LvAngebotReset()
 function LvAngebotNew()
 {
 	LvAngebotReset();
-	
-	tree = document.getElementById('lehrveranstaltung-lvangebot-tree-gruppen');
-	tree.currentIndex = -1;
+	LvAngebotGruppeTreeRefresh();
 }
\ No newline at end of file