Merge branch 'master' into feature-16190/Issues_automatisches_Beheben

# Conflicts:
#	system/dbupdate_3.3.php

application/controllers/jobs/AnrechnungJob.php

@@ -188,7 +188,7 @@ class AnrechnungJob extends JOB_Controller
 			$studiengang_bezeichnung = $this->StudiengangModel->load($studiengang_kz)->retval[0]->stg_bezeichnung;
 
 			// Get STGL mail address
-			list ($to, $vorname) = self::_getSTGLMailAddress($studiengang_kz);
+			$stglMailReceiver_arr = self::_getSTGLMailAddress($studiengang_kz);
 
 			// Get HTML table with new Anrechnungen of that STG plus amount of them
 			list ($anrechnungen_amount, $anrechnungen_table) = self::_getSTGLMailDataTable($studiengang_kz, $anrechnungen);
@@ -199,22 +199,25 @@ class AnrechnungJob extends JOB_Controller
 				CIS_ROOT. 'cis/menu.php?content_id=&content='.
 				CIS_ROOT. index_page(). self::APPROVE_ANRECHNUNG_URI;
 
+			foreach ($stglMailReceiver_arr as $stgl)
+			{
 			// Prepare mail content
-			$body_fields = array(
-				'vorname'       => $vorname,
-				'studiengang'   => $studiengang_bezeichnung,
-				'anzahl'        => $anrechnungen_amount,
-				'datentabelle'  => $anrechnungen_table,
-				'link'          => anchor($url, 'Anrechnungsanträge Übersicht')
-			);
-
-			// Send mail
-			sendSanchoMail(
-				'AnrechnungAntragStellen',
-				$body_fields,
-				$to,
-				'Anerkennung nachgewiesener Kenntnisse: Neuer Antrag wurde gestellt'
-			);
+				$body_fields = array(
+					'vorname'       => $stgl['vorname'],
+					'studiengang'   => $studiengang_bezeichnung,
+					'anzahl'        => $anrechnungen_amount,
+					'datentabelle'  => $anrechnungen_table,
+					'link'          => anchor($url, 'Anrechnungsanträge Übersicht')
+				);
+	
+				// Send mail
+				sendSanchoMail(
+					'AnrechnungAntragStellen',
+					$body_fields,
+					$stgl['to'],
+					'Anerkennung nachgewiesener Kenntnisse: Neuer Antrag wurde gestellt'
+				);
+			}
 		}
 
 		$this->logInfo('SUCCEDED: Sending emails to STGL about yesterdays new Anrechnungen succeded.');
@@ -342,15 +345,21 @@ html;
 	// Get STGL mail address
 	private function _getSTGLMailAddress($studiengang_kz)
 	{
+		$stglMailAdress_arr = array();
 		$result = $this->StudiengangModel->getLeitung($studiengang_kz);
 
 		// Get STGL mail address
 		if (hasData($result))
 		{
-			return array(
-				$result->retval[0]->uid. '@'. DOMAIN,
-				$result->retval[0]->vorname
-			);
+			foreach (getData($result) as $stgl)
+			{
+				$stglMailAdress_arr[]= array(
+					'to' => $stgl->uid. '@'. DOMAIN,
+					'vorname' => $stgl->vorname
+				);
+			}
+			
+			return $stglMailAdress_arr;
 		}
 		// If not available, get assistance mail address
 		else

application/controllers/lehre/anrechnung/ApproveAnrechnungDetail.php

@@ -412,28 +412,31 @@ class approveAnrechnungDetail extends Auth_Controller
 	{
 		$result = $this->AnrechnungModel->load($anrechnung_id);
 
-		if(!$result = getData($result)[0])
+		if(!hasData($result))
 		{
 			show_error('Failed loading Anrechnung');
 		}
-
+		
 		$result = $this->LehrveranstaltungModel->loadWhere(array(
-			'lehrveranstaltung_id' => $result->lehrveranstaltung_id
+			'lehrveranstaltung_id' => getData($result)[0]->lehrveranstaltung_id
 		));
 
-		if(!$result = getData($result)[0])
+		if(!hasData($result))
 		{
 			show_error('Failed loading Lehrveranstaltung');
 		}
 
 		// Get STGL
-		$result = $this->StudiengangModel->getLeitung($result->studiengang_kz);
-
-		if($result = getData($result)[0])
+		$result = $this->StudiengangModel->getLeitung(getData($result)[0]->studiengang_kz);
+		
+		if (hasData($result))
 		{
-			if ($result->uid == $this->_uid)
+			foreach (getData($result) as $stgl)
 			{
-				return;
+				if ($stgl->uid == $this->_uid)
+				{
+					return;
+				}
 			}
 		}
 

application/controllers/lehre/anrechnung/ApproveAnrechnungUebersicht.php

@@ -285,11 +285,14 @@ class approveAnrechnungUebersicht extends Auth_Controller
 		// Get STGL
 		$result = $this->StudiengangModel->getLeitung($result->studiengang_kz);
 		
-		if($result = getData($result)[0])
+		if (hasData($result))
 		{
-			if ($result->uid == $this->_uid)
+			foreach (getData($result) as $stgl)
 			{
-				return;
+				if ($stgl->uid == $this->_uid)
+				{
+					return;
+				}
 			}
 		}
 		
@@ -376,23 +379,24 @@ class approveAnrechnungUebersicht extends Auth_Controller
 			$this->load->model('education/Lehrveranstaltung_model', 'LehrveranstaltungModel');
 			$result = $this->LehrveranstaltungModel->getLecturersByLv($anrechnung['studiensemester_kurzbz'], $anrechnung['lehrveranstaltung_id']);
 			
-			if (!$result = getData($result))
+			if (!hasData($result))
 			{
 				show_error('Failed retrieving lectors of Lehrveranstaltung');
 			}
 			
+			$lecturersByLv = getData($result);
+
 			// Check if lv has LV-Leitung
-			$key = array_search(true, array_column($result, 'lvleiter'));
-			
+			$key = array_search(true, array_column($lecturersByLv, 'lvleiter'));
 			// If lv has LV-Leitung, keep only the one
 			if ($key !== false)
 			{
-				$lector_arr[]= $result[$key];
+				$lector_arr[]= $lecturersByLv[$key];
 			}
 			// ...otherwise keep all lectors
 			else
 			{
-				$lector_arr = array_merge($lector_arr, $result);
+				$lector_arr = array_merge($lector_arr, $lecturersByLv);
 			}
 		}
 		

application/controllers/lehre/anrechnung/RequestAnrechnung.php

@@ -80,7 +80,7 @@ class requestAnrechnung extends Auth_Controller
 		$prestudent_id = getData($result)[0]->prestudent_id;
 		
 		// Check if application deadline is expired
-		$is_expired = self::_checkAntragDeadline(
+		$is_expired = self::_isExpired(
 			$this->config->item('submit_application_start'),
 			$this->config->item('submit_application_end'),
 			$studiensemester_kurzbz
@@ -226,10 +226,10 @@ class requestAnrechnung extends Auth_Controller
 	 * @param $start Start date for application submission.
 	 * @param $ende End date for application submission.
 	 * @param $studiensemester_kurzbz
-	 * @return bool True if today is not during the start- and ending deadlines (= if is expired)
+	 * @return bool True if deadline is expired
 	 * @throws Exception
 	 */
-	private function _checkAntragDeadline($start, $ende, $studiensemester_kurzbz)
+	private function _isExpired($start, $ende, $studiensemester_kurzbz)
 	{
 		$this->load->model('organisation/Studiensemester_model', 'StudiensemesterModel');
 		
@@ -253,8 +253,8 @@ class requestAnrechnung extends Auth_Controller
 		$start = new DateTime($start);
 		$ende = new DateTime($ende);
 		
-		// True if today is not during the start- and ending deadlines (= if is expired)
-		return ($today <= $start || $today >= $ende);
+		// True if expired
+		return ($today < $start || $today > $ende);
 	}
 	
 	/**
@@ -324,8 +324,8 @@ class requestAnrechnung extends Auth_Controller
 	private function _LVhasBlockingGrades($studiensemester_kurzbz, $lehrveranstaltung_id)
 	{
 		// Get Note of Lehrveranstaltung
-		$this->load->model('education/Lvgesamtnote_model', 'LvgesamtnoteModel');
-		$result = $this->LvgesamtnoteModel->load(array(
+		$this->load->model('education/Zeugnisnote_model', 'ZeugnisnoteModel');
+		$result = $this->ZeugnisnoteModel->load(array(
 				'student_uid' => $this->_uid,
 				'studiensemester_kurzbz' => $studiensemester_kurzbz,
 				'lehrveranstaltung_id' => $lehrveranstaltung_id

application/helpers/hlp_common_helper.php

@@ -314,32 +314,40 @@ function sanitizeProblemChars($str)
 	$enc = 'UTF-8';
 
 	$acentos = array(
-		'A' => '/À|Á|Â|Ã|Å/',
+		'A' => '/À|Á|Â|Ã|Å|Ă|Ǎ/',
 		'Ae' => '/Ä/',
-		'a' => '/à|á|â|ã|å/',
-		'ae'=> '/ä/',
-		'C' => '/Ç/',
-		'c' => '/ç/',
+		'a' => '/à|á|â|ã|å|ă|ǎ/',
+		'ae' => '/ä/',
+		'C' => '/Ç|Č/',
+		'c' => '/ç|č/',
 		'E' => '/È|É|Ê|Ë/',
 		'e' => '/è|é|ê|ë/',
-		'I' => '/Ì|Í|Î|Ï/',
-		'i' => '/ì|í|î|ï/',
-		'N' => '/Ñ/',
+		'I' => '/Ì|Í|Î|Ï|Ǐ/',
+		'i' => '/ì|í|î|ï|ǐ/',
+		'N' => '/Ñ|Ň|ň/',
 		'n' => '/ñ/',
-		'O' => '/Ò|Ó|Ô|Õ/',
+		'O' => '/Ò|Ó|Ô|Õ|Ǒ/',
 		'Oe' => '/Ö/',
-		'o' => '/ò|ó|ô|õ/',
+		'o' => '/ò|ó|ô|õ|ǒ/',
 		'oe' => '/ö/',
-		'U' => '/Ù|Ú|Û/',
+		'R' => '/Ř/',
+		'r' => '/ř/',
+		'S' => '/Š/',
+		's' => '/š/',
+		'T' => '/Ť/',
+		't' => '/ť/',
+		'U' => '/Ù|Ú|Û|Ŭ|Ǔ/',
 		'Ue' => '/Ü/',
-		'u' => '/ù|ú|û/',
+		'u' => '/ù|ú|û|ŭ|ǔ/',
 		'ue' => '/ü/',
 		'Y' => '/Ý/',
 		'y' => '/ý|ÿ/',
+		'Z' => '/Ž/',
+		'z' => '/ž/',
 		'a.' => '/ª/',
 		'o.' => '/º/',
 		'ss' => '/ß/'
 	);
 
-	return preg_replace($acentos, array_keys($acentos), htmlentities($str,ENT_NOQUOTES, $enc));
+	return preg_replace($acentos, array_keys($acentos), htmlentities($str, ENT_NOQUOTES | ENT_HTML5, $enc));
 }

application/models/crm/Konto_model.php

@@ -18,45 +18,50 @@ class Konto_model extends DB_Model
 	public function setPaid($buchungsnr)
 	{
 		// get payment
-		$buchungResult =  $this->loadWhere(array('buchungsnr' => $buchungsnr));
+		$buchungResult = $this->loadWhere(array('buchungsnr' => $buchungsnr));
 
-		if(isSuccess($buchungResult) && hasData($buchungResult))
+		if (isSuccess($buchungResult) && hasData($buchungResult))
 		{
+			$buchung = getData($buchungResult)[0];
+
 			// get already paid amount
 			$this->addSelect('sum(betrag) as bezahlt');
 			$this->addGroupBy('buchungsnr_verweis');
-			$buchungVerweisResult =  $this->loadWhere(array('buchungsnr_verweis' => $buchungsnr));
+			$buchungVerweisResult = $this->loadWhere(array('buchungsnr_verweis' => $buchungsnr));
 
-			if(isSuccess($buchungVerweisResult))
+			if (isSuccess($buchungVerweisResult))
 			{
-				if(hasData($buchungVerweisResult))
-				{
-					$betragBezahltResult = getData($buchungVerweisResult);
-					$betragBezahlt = $betragBezahltResult->bezahlt;
-				}
-				else
-					$betragBezahlt = 0;
+				$betragBezahlt = 0;
 
-				$buchung = getData($buchungResult);
-				$buchung = $buchung[0];
+				if (hasData($buchungVerweisResult))
+				{
+					$betragBezahlt = getData($buchungVerweisResult)[0]->bezahlt;
+				}
 
 				// calculate open amount
-				$betragOffen = $betragBezahlt - $buchung->betrag*(-1);
+				$betragOffen = $betragBezahlt - $buchung->betrag * (-1);
 
-				$data = array(
-					'person_id' => $buchung->person_id,
-					'studiengang_kz' => $buchung->studiengang_kz,
-					'studiensemester_kurzbz' => $buchung->studiensemester_kurzbz,
-					'buchungsnr_verweis' => $buchungsnr,
-					'betrag' => str_replace(',','.',$betragOffen*(-1)),
-					'buchungsdatum' => date('Y-m-d'),
-					'buchungstext' => $buchung->buchungstext,
-					'insertamum' => date('Y-m-d H:i:s'),
-					'insertvon' => '',
-					'buchungstyp_kurzbz' => $buchung->buchungstyp_kurzbz,
-				);
+				if ($betragOffen != 0)
+				{
+					$data = array(
+						'person_id' => $buchung->person_id,
+						'studiengang_kz' => $buchung->studiengang_kz,
+						'studiensemester_kurzbz' => $buchung->studiensemester_kurzbz,
+						'buchungsnr_verweis' => $buchungsnr,
+						'betrag' => str_replace(',', '.', $betragOffen * (-1)),
+						'buchungsdatum' => date('Y-m-d'),
+						'buchungstext' => $buchung->buchungstext,
+						'insertamum' => date('Y-m-d H:i:s'),
+						'insertvon' => '',
+						'buchungstyp_kurzbz' => $buchung->buchungstyp_kurzbz,
+					);
 
-				return $this->insert($data);
+					return $this->insert($data);
+				}
+				else
+				{
+					return success();
+				}
 			}
 			else
 			{