From 5c21e29a0acf4155e47fe79d05d04d89231400f7 Mon Sep 17 00:00:00 2001
From: alex <karpen_ko@hotmail.com>
Date: Wed, 6 Nov 2019 16:57:58 +0100
Subject: [PATCH] - Studiensemester + Studienjahr anlegen: if
 Studiensemester/Studienjahr to add already exist, more beautiful error
 message displayed instead of page crash with database error - renamed methods
 "saveStudienjahr" and "saveStudiensemester" und Studienjahr and
 Studiensemester controller to clearer "updateStudienjahr" and
 "updateStudiensemester"

---
 application/controllers/organisation/Studienjahr.php   |  8 ++++++--
 .../controllers/organisation/Studiensemester.php       | 10 +++++++---
 application/views/organisation/studienjahrEdit.php     |  2 +-
 application/views/organisation/studiensemesterEdit.php |  2 +-
 4 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/application/controllers/organisation/Studienjahr.php b/application/controllers/organisation/Studienjahr.php
index c3c841df9..e154734d4 100644
--- a/application/controllers/organisation/Studienjahr.php
+++ b/application/controllers/organisation/Studienjahr.php
@@ -20,7 +20,7 @@ class Studienjahr extends Auth_Controller
 				'editStudienjahr' => 'basis/studiensemester:rw',
 				'newStudienjahr' => 'basis/studiensemester:rw',
 				'insStudienjahr' => 'basis/studiensemester:rw',
-				'saveStudienjahr' => 'basis/studiensemester:rw',
+				'updateStudienjahr' => 'basis/studiensemester:rw',
 				'deleteStudienjahr' => 'basis/studiensemester:rw'
 			)
 		);
@@ -117,6 +117,10 @@ class Studienjahr extends Auth_Controller
 	public function insStudienjahr()
 	{
 		$data = $this->__retrieveStudienjahrData();
+		$studienjahr_exists = $this->StudienjahrModel->load($data['studienjahr_kurzbz']);
+		if (hasData($studienjahr_exists))
+			show_error("Studienjahr existiert bereits");
+
 		$studienjahr = $this->StudienjahrModel->insert($data);
 
 		if ($studienjahr->error)
@@ -177,7 +181,7 @@ class Studienjahr extends Auth_Controller
 	 * replaces slash in Kurzbezeichnung with underscore
 	 * saved=true is a GET parameter passed for showing save message
 	 */
-	public function saveStudienjahr()
+	public function updateStudienjahr()
 	{
 		$data = $this->__retrieveStudienjahrData();
 		$studienjahr = $this->StudienjahrModel->update($data['studienjahr_kurzbz'], $data);
diff --git a/application/controllers/organisation/Studiensemester.php b/application/controllers/organisation/Studiensemester.php
index cdca4997d..1e1aa45ee 100644
--- a/application/controllers/organisation/Studiensemester.php
+++ b/application/controllers/organisation/Studiensemester.php
@@ -20,7 +20,7 @@ class Studiensemester extends Auth_Controller
 				'editStudiensemester' => 'basis/studiensemester:rw',
 				'newStudiensemester' => 'basis/studiensemester:rw',
 				'insStudiensemester' => 'basis/studiensemester:rw',
-				'saveStudiensemester' => 'basis/studiensemester:rw',
+				'updateStudiensemester' => 'basis/studiensemester:rw',
 				'deleteStudiensemester' => 'basis/studiensemester:rw'
 			)
 		);
@@ -108,6 +108,11 @@ class Studiensemester extends Auth_Controller
 	public function insStudiensemester()
 	{
 		$data = $this->__retrieveStudiensemesterData();
+
+		$studiensemester_exists = $this->StudiensemesterModel->load($data['studiensemester_kurzbz']);
+		if (hasData($studiensemester_exists))
+			show_error("Studiensemester existiert bereits");
+
 		$semester = $this->StudiensemesterModel->insert($data);
 
 		if ($semester->error)
@@ -185,7 +190,7 @@ class Studiensemester extends Auth_Controller
 	 * redirects to edit page after inserting
 	 * saved=true is a GET parameter passed for showing save message
 	 */
-	public function saveStudiensemester()
+	public function updateStudiensemester()
 	{
 		$data = $this->__retrieveStudiensemesterData();
 		$semester = $this->StudiensemesterModel->update($data['studiensemester_kurzbz'], $data);
@@ -214,5 +219,4 @@ class Studiensemester extends Auth_Controller
 
 		redirect("/organisation/studiensemester/listStudiensemester");
 	}
-
 }
diff --git a/application/views/organisation/studienjahrEdit.php b/application/views/organisation/studienjahrEdit.php
index 4a23217b1..df0397cdf 100644
--- a/application/views/organisation/studienjahrEdit.php
+++ b/application/views/organisation/studienjahrEdit.php
@@ -7,7 +7,7 @@ $this->load->view('templates/header', array('title' => 'StudienjahrEdit', 'jquer
 	<div class="row">
 		<div class="span4">
 			<h2>Studienjahr bearbeiten: <?php echo $jahr->studienjahr_kurzbz; ?></h2>
-			<form method="post" action="<?php echo site_url("organisation/studienjahr/saveStudienjahr"); ?>">
+			<form method="post" action="<?php echo site_url("organisation/studienjahr/updateStudienjahr"); ?>">
 
 				<table>
 					<?php include('studienjahrForm.php'); ?>
diff --git a/application/views/organisation/studiensemesterEdit.php b/application/views/organisation/studiensemesterEdit.php
index 9f0abc0cb..98c4ca243 100644
--- a/application/views/organisation/studiensemesterEdit.php
+++ b/application/views/organisation/studiensemesterEdit.php
@@ -8,7 +8,7 @@ $this->load->view('templates/header', array('title' => 'StudiensemesterEdit', 'd
 		<div class="span4">
 			<h2>Studiensemester bearbeiten: <?php echo $sem->studiensemester_kurzbz; ?></h2>
 			<form method="post"
-				  action="<?php echo site_url("organisation/studiensemester/saveStudiensemester") ?>">
+				  action="<?php echo site_url("organisation/studiensemester/updateStudiensemester") ?>">
 				<table>
 					<?php include('studiensemesterForm.php'); ?>
 					<input type="hidden" name="semkurzbz" value="<?php echo $sem->studiensemester_kurzbz; ?>"/>