From 9a0ea02e0c332a64d64c06dbc2fdbdca93eae499 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Wed, 20 Jan 2021 15:20:57 +0100
Subject: [PATCH] Minor code enhancement

Signed-off-by: cris-technikum <hainberg@technikum-wien.at>
---
 .../anrechnung/ApproveAnrechnungUebersicht.php   | 16 ++++++++++------
 application/libraries/AnrechnungLib.php          |  5 +++++
 .../anrechnung/approveAnrechnungUebersicht.js    | 11 -----------
 3 files changed, 15 insertions(+), 17 deletions(-)

diff --git a/application/controllers/lehre/anrechnung/ApproveAnrechnungUebersicht.php b/application/controllers/lehre/anrechnung/ApproveAnrechnungUebersicht.php
index 48b8e2aa8..d8d34b981 100644
--- a/application/controllers/lehre/anrechnung/ApproveAnrechnungUebersicht.php
+++ b/application/controllers/lehre/anrechnung/ApproveAnrechnungUebersicht.php
@@ -88,7 +88,7 @@ class approveAnrechnungUebersicht extends Auth_Controller
 	}
 	
 	/**
-	 * Approve multiple Anrechnungen.
+	 * Approve Anrechnungen.
 	 */
 	public function approve()
 	{
@@ -102,6 +102,9 @@ class approveAnrechnungUebersicht extends Auth_Controller
 		// Get statusbezeichnung for 'approved'
 		$this->AnrechnungstatusModel->addSelect('bezeichnung_mehrsprachig');
 		$approved = getData($this->AnrechnungstatusModel->load('approved'))[0];
+		$approved = getUserLanguage() == 'German'
+			? $approved->bezeichnung_mehrsprachig[0]
+			: $approved->bezeichnung_mehrsprachig[1];
 
 		foreach ($data as $item)
 		{
@@ -110,9 +113,7 @@ class approveAnrechnungUebersicht extends Auth_Controller
 			{
 				$json[]= array(
 					'anrechnung_id' => $item['anrechnung_id'],
-					'status_bezeichnung' => getUserLanguage() == 'German'
-						? $approved->bezeichnung_mehrsprachig[0]
-						: $approved->bezeichnung_mehrsprachig[1]
+					'status_bezeichnung' => $approved
 				);
 			}
 		}
@@ -128,6 +129,9 @@ class approveAnrechnungUebersicht extends Auth_Controller
 		}
 	}
 	
+	/**
+	 * Reject Anrechnungen.
+	 */
 	public function reject()
 	{
 		$data = $this->input->post('data');
@@ -137,7 +141,7 @@ class approveAnrechnungUebersicht extends Auth_Controller
 			return $this->outputJsonError('Fehler beim Übertragen der Daten.');
 		}
 		
-		// Get statusbezeichnung for 'approved'
+		// Get statusbezeichnung for 'rejected'
 		$this->AnrechnungstatusModel->addSelect('bezeichnung_mehrsprachig');
 		$rejected = getData($this->AnrechnungstatusModel->load('rejected'))[0];
 		$rejected = getUserLanguage() == 'German'
@@ -146,7 +150,7 @@ class approveAnrechnungUebersicht extends Auth_Controller
 		
 		foreach ($data as $item)
 		{
-			// Approve Anrechnung
+			// Reject Anrechnung
 			if(getData($this->anrechnunglib->rejectAnrechnung($item['anrechnung_id'])))
 			{
 				$json[]= array(
diff --git a/application/libraries/AnrechnungLib.php b/application/libraries/AnrechnungLib.php
index 6a2f1ac78..5d4f48aa0 100644
--- a/application/libraries/AnrechnungLib.php
+++ b/application/libraries/AnrechnungLib.php
@@ -225,6 +225,11 @@ class AnrechnungLib
 		return success(true);   // approved
 	}
 	
+	/**
+	 * Reject Anrechnung.
+	 * @param $anrechnung_id
+	 * @return array
+	 */
 	public function rejectAnrechnung($anrechnung_id)
 	{
 		// Check last Anrechnungstatus
diff --git a/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js b/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
index c746a835c..661946803 100644
--- a/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
+++ b/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
@@ -41,19 +41,8 @@ function func_selectableCheck(row){
 
 // Performes after row was updated
 function func_rowUpdated(row){
-    // TODO: check better solution...
-    // status_kurzbz is not updated until page reload...therefor use the updated status_bezeichnung
-    let status_bezeichnung = row.getData().status_bezeichnung;
-
-    // Deselect and disable new selection of updated rows, but not when
-    if (status_bezeichnung == 'genehmigt' ||
-        status_bezeichnung == 'approved' ||
-        status_bezeichnung == 'abgelehnt' ||
-        status_bezeichnung == 'rejected')
-    {
         row.deselect();
         row.getElement().style["pointerEvents"] = "none";
-    }
 }
 
 // Formats null values to '-'