Merge branch 'master' into benutzerberechtigungGUIneu

application/controllers/jobs/IssueResolver.php

@@ -44,6 +44,7 @@ class IssueResolver extends IssueResolver_Controller
 			'CORE_STUDENTSTATUS_0012' => 'CORE_STUDENTSTATUS_0012',
 			'CORE_STUDENTSTATUS_0013' => 'CORE_STUDENTSTATUS_0013',
 			'CORE_STUDENTSTATUS_0014' => 'CORE_STUDENTSTATUS_0014',
+			'CORE_STUDENTSTATUS_0015' => 'CORE_STUDENTSTATUS_0015',
 			'CORE_PERSON_0001' => 'CORE_PERSON_0001',
 			'CORE_PERSON_0002' => 'CORE_PERSON_0002',
 			'CORE_PERSON_0003' => 'CORE_PERSON_0003',

application/controllers/system/issues/Plausichecks.php

@@ -4,6 +4,8 @@ if (! defined('BASEPATH')) exit('No direct script access allowed');
 
 class Plausichecks extends Auth_Controller
 {
+	const GENERIC_ISSUE_OCCURED_TEXT = 'Issue aufgetreten';
+
 	public function __construct()
 	{
 		parent::__construct(
@@ -78,20 +80,27 @@ class Plausichecks extends Auth_Controller
 					// optionally replace fehler parameters in text, output the fehlertext
 					if (hasData($fehlerRes))
 					{
+						// use issue fehler text from database if present
 						$fehlerText = getData($fehlerRes)[0]->fehlertext;
 
 						if (!isEmptyArray($fehlertext_params))
 						{
+							// replace placeholder with params, if present
 							if (count($fehlertext_params) != substr_count($fehlerText, '%s'))
 								$this->terminateWithJsonError('Wrong number of parameters for Fehlertext, fehler_kurzbz ' . $fehler_kurzbz);
 
 							$fehlerText = vsprintf($fehlerText, $fehlertext_params);
 						}
-
-						if (isset($person_id)) $fehlerText .= "; person_id: $person_id";
-						if (isset($oe_kurzbz)) $fehlerText .= "; oe_kurzbz: $oe_kurzbz";
-						$issueTexts[$fehler_kurzbz][] = $fehlerText;
 					}
+					else // if no issue text found, use generic text
+					{
+						$fehlerText = self::GENERIC_ISSUE_OCCURED_TEXT;
+					}
+
+					// add generic parameters to issue text
+					if (isset($person_id)) $fehlerText .= "; person_id: $person_id";
+					if (isset($oe_kurzbz)) $fehlerText .= "; oe_kurzbz: $oe_kurzbz";
+					$issueTexts[$fehler_kurzbz][] = $fehlerText;
 				}
 			}
 		}

application/libraries/issues/PlausicheckLib.php

@@ -86,16 +86,22 @@ class PlausicheckLib
 				JOIN public.tbl_prestudentstatus status USING(prestudent_id)
 				JOIN public.tbl_benutzer benutzer on(benutzer.uid = student.student_uid)
 				JOIN public.tbl_studiengang stg ON prestudent.studiengang_kz = stg.studiengang_kz
+				LEFT JOIN lehre.tbl_studienplan stpl USING (studienplan_id)
 			WHERE
 				benutzer.aktiv = true
-				AND status.status_kurzbz='Student'
+				AND status.status_kurzbz IN ('Student', 'Unterbrecher', 'Abbrecher', 'Diplomand', 'Absolvent')
 				AND studiengang.studiengang_kz < 10000
 				AND status.studiensemester_kurzbz = ?
+				AND NOT (status.orgform_kurzbz IS NULL AND studiengang.mischform = FALSE)
 				AND NOT EXISTS(
-					SELECT 1 FROM lehre.tbl_studienplan
-					JOIN lehre.tbl_studienordnung USING(studienordnung_id)
+					SELECT 1
+					FROM
+						lehre.tbl_studienplan
+					JOIN
+						lehre.tbl_studienordnung USING(studienordnung_id)
 					WHERE
-						tbl_studienordnung.studiengang_kz = prestudent.studiengang_kz
+						tbl_studienplan.studienplan_id = stpl.studienplan_id
+						AND tbl_studienordnung.studiengang_kz = prestudent.studiengang_kz
 						AND tbl_studienplan.orgform_kurzbz = status.orgform_kurzbz)";
 
 		if (isset($prestudent_id))
@@ -106,7 +112,7 @@ class PlausicheckLib
 
 		if (isset($studiengang_kz))
 		{
-			$qry .= " AND stg.studiengang_kz = ?";
+			$qry .= " AND studiengang.studiengang_kz = ?";
 			$params[] = $studiengang_kz;
 		}
 
@@ -879,6 +885,57 @@ class PlausicheckLib
 		return $this->_db->execReadOnlyQuery($qry, $params);
 	}
 
+	/**
+	 * Student with active status should have been charged, i.e. have a Kontobuchung with a negative or zero value.
+	 * @param studiensemester_kurzbz string if check is to be executed for certain Studiensemester
+	 * @param studiengang_kz int if check is to be executed for certain Studiengang
+	 * @param prestudent_id int if check is to be executed only for one prestudent
+	 * @return success with prestudents or error
+	 */
+	public function getAktiverStudentstatusOhneKontobuchung($studiensemester_kurzbz, $studiengang_kz = null, $prestudent_id = null)
+	{
+		$params = array($studiensemester_kurzbz);
+
+		$qry = "
+			SELECT
+				DISTINCT ON (pre.prestudent_id)
+				pre.person_id, pre.prestudent_id, stg.oe_kurzbz AS prestudent_stg_oe_kurzbz, status.studiensemester_kurzbz
+			FROM
+				public.tbl_prestudent pre
+				JOIN public.tbl_person pers USING(person_id)
+				JOIN public.tbl_prestudentstatus status USING(prestudent_id)
+				JOIN public.tbl_studiengang stg USING(studiengang_kz)
+			WHERE
+				status.studiensemester_kurzbz = ?
+				AND status.status_kurzbz IN ('Student', 'Incoming')
+				AND NOT EXISTS (
+					SELECT 1
+					FROM
+						public.tbl_konto
+					WHERE
+						person_id = pers.person_id
+						AND studiensemester_kurzbz = status.studiensemester_kurzbz
+						AND buchungsnr_verweis IS NULL
+						AND betrag <= 0
+				)
+				AND stg.melderelevant
+				AND pre.bismelden";
+
+		if (isset($studiengang_kz))
+		{
+			$qry .= " AND stg.studiengang_kz = ?";
+			$params[] = $studiengang_kz;
+		}
+
+		if (isset($prestudent_id))
+		{
+			$qry .= " AND pre.prestudent_id = ?";
+			$params[] = $prestudent_id;
+		}
+
+		return $this->_db->execReadOnlyQuery($qry, $params);
+	}
+
 	//------------------------------------------------------------------------------------------------------------------
 	// Person checks
 

application/libraries/issues/PlausicheckProducerLib.php

@@ -18,6 +18,7 @@ class PlausicheckProducerLib
 		'AbschlussstatusFehlt' => 'AbschlussstatusFehlt',
 		'AktSemesterNull' => 'AktSemesterNull',
 		'AktiverStudentOhneStatus' => 'AktiverStudentOhneStatus',
+		'AktiverStudentstatusOhneKontobuchung' => 'AktiverStudentstatusOhneKontobuchung',
 		'AusbildungssemPrestudentUngleichAusbildungssemStatus' => 'AusbildungssemPrestudentUngleichAusbildungssemStatus',
 		'BewerberNichtZumRtAngetreten' => 'BewerberNichtZumRtAngetreten',
 		'DatumAbschlusspruefungFehlt' => 'DatumAbschlusspruefungFehlt',

application/libraries/issues/plausichecks/AktiverStudentstatusOhneKontobuchung.php

@@ -0,0 +1,50 @@
+<?php
+
+if (! defined('BASEPATH')) exit('No direct script access allowed');
+
+require_once('PlausiChecker.php');
+
+/**
+ * Student with active status should have been charged, i.e. have a Kontobuchung with negative or zero value.
+ */
+class AktiverStudentstatusOhneKontobuchung extends PlausiChecker
+{
+	public function executePlausiCheck($params)
+	{
+		$results = array();
+
+		// pass parameters needed for plausicheck
+		$studiensemester_kurzbz = isset($params['studiensemester_kurzbz']) ? $params['studiensemester_kurzbz'] : null;
+		$studiengang_kz = isset($params['studiengang_kz']) ? $params['studiengang_kz'] : null;
+
+		// get all students failing the plausicheck
+		$prestudentRes = $this->_ci->plausichecklib->getAktiverStudentstatusOhneKontobuchung($studiensemester_kurzbz, $studiengang_kz);
+
+		if (isError($prestudentRes)) return $prestudentRes;
+
+		if (hasData($prestudentRes))
+		{
+			$prestudents = getData($prestudentRes);
+
+			// populate results with data necessary for writing issues
+			foreach ($prestudents as $prestudent)
+			{
+				$results[] = array(
+					'person_id' => $prestudent->person_id,
+					'oe_kurzbz' => $prestudent->prestudent_stg_oe_kurzbz,
+					'fehlertext_params' => array(
+						'prestudent_id' => $prestudent->prestudent_id,
+						'studiensemester_kurzbz' => $prestudent->studiensemester_kurzbz
+					),
+					'resolution_params' => array(
+						'prestudent_id' => $prestudent->prestudent_id,
+						'studiensemester_kurzbz' => $prestudent->studiensemester_kurzbz
+					)
+				);
+			}
+		}
+
+		// return the results
+		return success($results);
+	}
+}

application/libraries/issues/resolvers/CORE_STUDENTSTATUS_0015.php

@@ -0,0 +1,36 @@
+<?php
+
+if (! defined('BASEPATH')) exit('No direct script access allowed');
+
+/**
+ * Student with active status should have been charged, i.e. have a Kontobuchung with negative or zero value.
+ */
+class CORE_STUDENTSTATUS_0015 implements IIssueResolvedChecker
+{
+	public function checkIfIssueIsResolved($params)
+	{
+		if (!isset($params['prestudent_id']) || !is_numeric($params['prestudent_id']))
+			return error('Prestudent Id missing, issue_id: '.$params['issue_id']);
+
+		if (!isset($params['studiensemester_kurzbz']) || isEmptyString($params['studiensemester_kurzbz']))
+			return error('Studiensemester missing, issue_id: '.$params['issue_id']);
+
+		$this->_ci =& get_instance(); // get code igniter instance
+
+		$this->_ci->load->library('issues/PlausicheckLib');
+
+		// check if issue persists
+		$checkRes = $this->_ci->plausichecklib->getAktiverStudentstatusOhneKontobuchung(
+			$params['studiensemester_kurzbz'],
+			null,
+			$params['prestudent_id']
+		);
+
+		if (isError($checkRes)) return $checkRes;
+
+		if (hasData($checkRes))
+			return success(false); // not resolved if issue is still present
+		else
+			return success(true); // resolved otherwise
+	}
+}

cis/private/lehre/abgabe_student.php

@@ -108,7 +108,8 @@ $sql_query = "SELECT (SELECT nachname FROM public.tbl_person  WHERE person_id=tb
 			lehre.tbl_projektbetreuer.note as note,
 			public.tbl_benutzer.aktiv as aktiv,
 			(SELECT abgeschicktvon FROM extension.tbl_projektarbeitsbeurteilung WHERE projektarbeit_id = tbl_projektarbeit.projektarbeit_id AND betreuer_person_id = tbl_projektbetreuer.person_id) AS babgeschickt,
-			(SELECT abgeschicktvon FROM extension.tbl_projektarbeitsbeurteilung WHERE projektarbeit_id = tbl_projektarbeit.projektarbeit_id AND betreuerart_kurzbz IN ('Zweitbetreuer', 'Zweitbegutachter') LIMIT 1) AS zweitbetreuer_abgeschickt
+			(SELECT abgeschicktvon FROM extension.tbl_projektarbeitsbeurteilung WHERE projektarbeit_id = tbl_projektarbeit.projektarbeit_id AND betreuerart_kurzbz IN ('Zweitbetreuer', 'Zweitbegutachter') LIMIT 1) AS zweitbetreuer_abgeschickt,
+			(SELECT datum FROM campus.tbl_paabgabe WHERE paabgabetyp_kurzbz = 'end' AND abgabedatum IS NOT NULL AND projektarbeit_id = tbl_projektarbeit.projektarbeit_id LIMIT 1) AS abgegeben
 		FROM lehre.tbl_projektarbeit
 		LEFT JOIN lehre.tbl_projektbetreuer USING(projektarbeit_id)
 		LEFT JOIN public.tbl_benutzer ON(uid=student_uid)
@@ -204,11 +205,15 @@ else
 				$htmlstr .= "<a href='../pdfExport.php?xml=projektarbeitsbeurteilung.xml.php&xsl=Projektbeurteilung&betreuerart_kurzbz=" . $row->zweitbetreuer_betreuerart_kurzbz . "&projektarbeit_id=" . $row->projektarbeit_id . "&person_id=" . $row->zweitbetreuer_person_id."' title='".$p->t('abgabetool/projektbeurteilungDownload')."'>".$p->t('abgabetool/projektbeurteilungZweitDownload')."</a>";
 
 			$htmlstr .= "</td>";
-		} else
-		{
-			$htmlstr  .= "<td>".$row->note."</td>";
 		}
-
+		elseif (!is_null($row->abgegeben))
+		{
+			$htmlstr  .= "<td>".$p->t('abgabetool/abgegeben')."</td>";
+		}
+		else
+		{
+			$htmlstr  .= "<td>-</td>";
+		}
 
 		$htmlstr .= "       <td>".$row->studiensemester_kurzbz."</td>\n";
 		$htmlstr .= "       <td>".strtoupper($row->typ.$row->kurzbz)."</td>\n";

locale/de-AT/abgabetool.php

@@ -92,4 +92,5 @@ $this->phrasen['abgabetool/projektbeurteilungErstDownload']='Erst-/Begutachter';
 $this->phrasen['abgabetool/projektbeurteilungZweitDownload']='Zweitbegutachter';
 $this->phrasen['abgabetool/fehlerErmittelnEndabgabeProjektarbeit']='Fehler beim Ermitteln des Enduplaods der Projektarbeit';
 $this->phrasen['abgabetool/senatsMitglied']='Mitglied Prüfungssenat';
+$this->phrasen['abgabetool/abgegeben']='Abgegeben, in Beurteilung';
 ?>

locale/en-US/abgabetool.php

@@ -92,4 +92,5 @@ $this->phrasen['abgabetool/projektbeurteilungErstDownload']='First-/Assessor';
 $this->phrasen['abgabetool/projektbeurteilungZweitDownload']='Second Assessor';
 $this->phrasen['abgabetool/fehlerErmittelnEndabgabeProjektarbeit']='Error when getting endupload of project work';
 $this->phrasen['abgabetool/senatsMitglied']='Examiner';
+$this->phrasen['abgabetool/abgegeben']='handed in, in assessment';
 ?>

system/fehlerupdate.php

@@ -277,6 +277,14 @@ $fehlerArr = array(
 		'fehlertyp_kurzbz' => 'error',
 		'app' => 'core'
 	),
+	array(
+		'fehlercode' => 'CORE_STUDENTSTATUS_0015',
+		'fehler_kurzbz' => 'AktiverStudentstatusOhneKontobuchung',
+		'fehlercode_extern' => null,
+		'fehlertext' => 'Keine Kontobuchung bei aktivem Studentstatus (prestudent_id %s, Studiensemester %s)',
+		'fehlertyp_kurzbz' => 'error',
+		'app' => 'core'
+	),
 	array(
 		'fehlercode' => 'CORE_PERSON_0001',
 		'fehler_kurzbz' => 'GbDatumWeitZurueck',

system/templates/contentmittitel_filterwidget_xslt_xhtml.xslt

@@ -0,0 +1,69 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<xsl:stylesheet version="1.0" xmlns:fo="http://www.w3.org/1999/XSL/Format"
+xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
+	<xsl:template match="content">
+		<html>
+		<head>
+			<title><xsl:value-of select="titel" /></title>
+			<link rel="stylesheet" href="../skin/style.css.php" type="text/css"  />
+			<link rel="stylesheet" href="../skin/jquery.css" type="text/css" />
+			<link rel="stylesheet" type="text/css" href="../skin/jquery-ui-1.9.2.custom.min.css" />
+			
+			<script type="text/javascript" src="../vendor/jquery/jquery1/jquery-1.12.4.min.js"></script>
+			<link rel="stylesheet" type="text/css" href="../FHC-vendor/jquery-tablesorter/css/theme.default.css"/>
+			<script src="../FHC-vendor/jquery-tablesorter/js/jquery.tablesorter.js"></script>
+			<script src="../FHC-vendor/jquery-tablesorter/js/jquery.tablesorter.widgets.js"></script>
+			<link rel="stylesheet" type="text/css" href="../include/vendor_custom/jquery-tablesorter/tablesort.css"/>
+
+			<script type="text/javascript" src="../vendor/components/jqueryui/jquery-ui.min.js"></script>
+			<script type="text/javascript" src="../include/js/jquery.ui.datepicker.translation.js"></script>
+			<script type="text/javascript" src="../vendor/jquery/sizzle/sizzle.js"></script>
+			<link rel="stylesheet" href="../skin/tablesort.css" type="text/css"/>
+			<script type="text/javascript">
+
+			$(document).ready(function()
+			{
+				$(".tablesorter").each(function()
+				{
+					var col=0;
+					var sort=0;
+					var no_sort=1;
+					var classes = $(this).attr("class");
+					var class_arr = classes.split(" ");
+					var headersobj={};
+
+					for(i in class_arr)
+					{
+						if(class_arr[i].indexOf("tablesorter_col_")!=-1)
+						{
+							col = class_arr[i].substr(16);
+						}
+						if(class_arr[i].indexOf("tablesorter_sort_")!=-1)
+						{
+							 sort = class_arr[i].substr(17);
+						}
+						if(class_arr[i].indexOf("tablesorter_no_sort_")!=-1)
+						{
+							 no_sort = class_arr[i].substr(20);
+							 headersobj[no_sort]={sorter:false};
+						}
+					}
+
+					$(this).tablesorter(
+					{
+						sortList: [[col,sort]],
+						widgets: ["zebra","filter"],
+						headers: headersobj
+					});
+				});
+			});
+
+			</script>
+		</head>
+		<body>
+	    <h1><xsl:value-of select="titel" /></h1>
+		<xsl:value-of select="inhalt" disable-output-escaping="yes" />
+		</body>
+		</html>
+	</xsl:template>
+</xsl:stylesheet >