From 9b3036e95d5234308e0d554d443e15d5bba9233e Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Mon, 7 Sep 2020 15:16:48 +0200
Subject: [PATCH 1/4] Adapted in CIS: Display Zahlungsbestaetigung for FHTW
 studies only.

If config entry ZAHLUNGSBESTAETIGUNG_ANZEIGEN is true and campus is
FH Technikum Wien, only payments of FHTW studies are displayed now.

Signed-off-by: Cris <hainberg@technikum-wien.at>
---
 cis/private/profile/zahlungen.php | 21 +++++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/cis/private/profile/zahlungen.php b/cis/private/profile/zahlungen.php
index b79f7f8b9..ed19e26b6 100644
--- a/cis/private/profile/zahlungen.php
+++ b/cis/private/profile/zahlungen.php
@@ -217,12 +217,22 @@ echo '			<script type="text/javascript" src="../../../vendor/components/jqueryui
 		foreach ($konto->result as $row)
 		{
 			$i=0;  //Zaehler fuer Anzahl Gegenbuchungen
+			$count_studiengangszahlung = 0;
 			$buchungsnummern='';
-			
+
+			// Für die FHTW sollen nur Zahlungsbestaetigungen von FHTW-Studien angezeigt werden. (Nicht von Lehrgaengen)
+			if (ZAHLUNGSBESTAETIGUNG_ANZEIGEN)
+			{
+				if (defined('CAMPUS_NAME') && CAMPUS_NAME == 'FH Technikum Wien')
+				{
+					$is_lehrgang = $row['parent']->studiengang_kz < 0 ? true : false;
+					if ($is_lehrgang) continue;
+				}
+			}
 			if(!isset($row['parent']))
 				continue;
 			$betrag = $row['parent']->betrag;
-
+			$count_studiengangszahlung ++;
 			
 			if(isset($row['childs']))
 			{
@@ -271,6 +281,13 @@ echo '			<script type="text/javascript" src="../../../vendor/components/jqueryui
 			}
 			echo '</tr>';
 		}
+		
+		// Wenn die Tabelle keine Eintraege hat, wird eine Tabellenzeile mit entsprechender Information angezeigt.
+		if ($count_studiengangszahlung == 0)
+		{
+			echo "<tr><td colspan='7' style='background-color: white;'>" .$p->t('tools/keineZahlungenVorhanden'). "</td></tr>";
+		}
+		
 		echo '</tbody></table>';
 	}
 	else 

From 9b78acb65581b10d525e11c8e161011d4f97177e Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Tue, 8 Sep 2020 12:00:04 +0200
Subject: [PATCH 2/4] Added log-entry for Zahlungsbestaetigung-Download

Will log the accounting number and the uid of the person who made the
download to be able to detect:
. how many users downloaded a payment confirmation
. how many payment confirmations were downloaded

Signed-off-by: Cris <hainberg@technikum-wien.at>
---
 cis/private/pdfExport.php | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/cis/private/pdfExport.php b/cis/private/pdfExport.php
index f5322d1ff..ea1fbab66 100644
--- a/cis/private/pdfExport.php
+++ b/cis/private/pdfExport.php
@@ -38,6 +38,7 @@ require_once('../../include/student.class.php');
 require_once('../../include/prestudent.class.php');
 require_once('../../include/dokument_export.class.php');
 require_once('../../include/person.class.php');
+require_once('../../include/webservicelog.class.php');
 
 if (!$db = new basis_db())
 	die('Fehler beim Oeffnen der Datenbankverbindung');
@@ -139,6 +140,21 @@ if (isset($_GET['all']))
 if (isset($_GET['xsl_oe_kurzbz']))
 	$params .= '&xsl_oe_kurzbz='. $_GET['xsl_oe_kurzbz'];
 
+// Logeintrag bei Download von Zahlungsbestaetigungen
+if (isset($_GET['xsl']) && $_GET['xsl'] == 'Zahlung')
+{
+	$requestdata = $_SERVER['QUERY_STRING'];
+	
+	$log = new Webservicelog();
+	$log->webservicetyp_kurzbz = 'content';
+	$log->request_id = isset($_GET['buchungsnummern']) && !empty($_GET['buchungsnummern']) ? $_GET['buchungsnummern'] : NULL;
+	$log->beschreibung = 'Zahlungsbestaetigungsdownload';
+	$log->request_data = $requestdata;
+	$log->execute_user = $user;
+	
+	$log->save(true);
+}
+
 //OE fuer Output ermitteln
 
 if ($xsl_oe_kurzbz != '')

From f0ef9ab333e998f6074e8d4effc6f7ff329742c1 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Tue, 8 Sep 2020 12:02:05 +0200
Subject: [PATCH 3/4] Removed unnecessary ";"-character at beginning of
 buchungsnummer

Signed-off-by: Cris <hainberg@technikum-wien.at>
---
 cis/private/profile/zahlungen.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/cis/private/profile/zahlungen.php b/cis/private/profile/zahlungen.php
index ed19e26b6..fd1731868 100644
--- a/cis/private/profile/zahlungen.php
+++ b/cis/private/profile/zahlungen.php
@@ -240,7 +240,8 @@ echo '			<script type="text/javascript" src="../../../vendor/components/jqueryui
 				{
 					$betrag += $row_child->betrag;
 					$betrag = round($betrag, 2);
-					$buchungsnummern .= ';'.$row['childs'][$key]->buchungsnr;
+					$buchungsnummern = !empty($buchungsnummern) ? ';' : '';
+					$buchungsnummern .= $row['childs'][$key]->buchungsnr;
 					$i = $key; //Zaehler auf letzten Gegenbuchungseintrag setzen
 				}
 			}

From 8021b8aef9e7fba6c4928675cd94b5d01c0c7106 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Tue, 8 Sep 2020 16:29:53 +0200
Subject: [PATCH 4/4] Added config variable
 ZAHLUNGSBESTAETIGUNG_ANZEIGEN_FUER_LEHRGAENGE

If set to false, Lehrgang-payment confirmations will not be displayed
for download in CIS.

Signed-off-by: Cris <hainberg@technikum-wien.at>
---
 cis/private/profile/zahlungen.php    | 10 ++++------
 config/global.config-default.inc.php |  3 +++
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/cis/private/profile/zahlungen.php b/cis/private/profile/zahlungen.php
index fd1731868..e255e805b 100644
--- a/cis/private/profile/zahlungen.php
+++ b/cis/private/profile/zahlungen.php
@@ -221,14 +221,12 @@ echo '			<script type="text/javascript" src="../../../vendor/components/jqueryui
 			$buchungsnummern='';
 
 			// Für die FHTW sollen nur Zahlungsbestaetigungen von FHTW-Studien angezeigt werden. (Nicht von Lehrgaengen)
-			if (ZAHLUNGSBESTAETIGUNG_ANZEIGEN)
+			if (defined('ZAHLUNGSBESTAETIGUNG_ANZEIGEN_FUER_LEHRGAENGE') && !ZAHLUNGSBESTAETIGUNG_ANZEIGEN_FUER_LEHRGAENGE)
 			{
-				if (defined('CAMPUS_NAME') && CAMPUS_NAME == 'FH Technikum Wien')
-				{
-					$is_lehrgang = $row['parent']->studiengang_kz < 0 ? true : false;
-					if ($is_lehrgang) continue;
-				}
+				$is_lehrgang = $row['parent']->studiengang_kz < 0 ? true : false;
+				if ($is_lehrgang) continue;
 			}
+			
 			if(!isset($row['parent']))
 				continue;
 			$betrag = $row['parent']->betrag;
diff --git a/config/global.config-default.inc.php b/config/global.config-default.inc.php
index 95bfd98e1..8d4687401 100644
--- a/config/global.config-default.inc.php
+++ b/config/global.config-default.inc.php
@@ -300,4 +300,7 @@ define('BUCHUNGEN_CHECK_SAP', true);
 
 // Gibt an, ob im FAS die Zahlungsbestaetigungen zum Download / im CIS generell die Zahlungen angezeigt werden
 define ('ZAHLUNGSBESTAETIGUNG_ANZEIGEN', true);
+
+// Gibt an, ob im CIS die Zahlungsbestaetigungen fuer Lehrgaenge zum Download angezeigt werden
+define ('ZAHLUNGSBESTAETIGUNG_ANZEIGEN_FUER_LEHRGAENGE', true);
 ?>