From 8d693b36f72ab45d1fc8d5f791a9695fd0c7fa1b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Wed, 7 Feb 2018 11:48:45 +0100
Subject: [PATCH 1/2] Fixed Warning if no entry is found

---
 vilesci/personen/wiederholer.php | 113 ++++++++++++++++---------------
 1 file changed, 59 insertions(+), 54 deletions(-)

diff --git a/vilesci/personen/wiederholer.php b/vilesci/personen/wiederholer.php
index 874f2fbda..6507b63c8 100644
--- a/vilesci/personen/wiederholer.php
+++ b/vilesci/personen/wiederholer.php
@@ -21,14 +21,14 @@
  */
 /**
  * Dieses Script ist fuer Personen, die das Studium abgebrochen haben und von neuem starten.
- * Dazu muessen Studenten ein neues Personenkennzeichen etc erhalten da es sonst zu Problemen bei 
+ * Dazu muessen Studenten ein neues Personenkennzeichen etc erhalten da es sonst zu Problemen bei
  * der BIS-Meldung kommt.
- * 
+ *
  * Mit diesem Script kann der Student gesucht werden und ein neuer Prestudent Eintrag angelegt werden.
  * Zusätzlich wird ein Interessentenstatus im ausgewaehlten Semester angelegt.
- * 
+ *
  * Die Prestudentdaten (ZGV etc) werden fuer den neuen Eintrag uebernommen.
- * 
+ *
  * Dieses Script ist NICHT fuer Studenten die nur ein Semester/Jahr wiederholen. Es ist nur fuer Abbrecher die
  * erneut in diesem Studiengang studieren moechten.
  */
@@ -78,30 +78,30 @@ if(isset($_POST['save']))
 		die('Studiensemester muss uebergeben werden');
 	if(!isset($_POST['ausbildungssemester']))
 		die('Ausbildungssemester muss uebergeben werden');
-	
+
 	$prestudent_id=$_POST['prestudent_id'];
 	$ausbildungssemester=$_POST['ausbildungssemester'];
 	$stsem_kurzbz=$_POST['stsem_kurzbz'];
 
 	if(!is_numeric($prestudent_id))
 		die('PrestudentID ist ungueltig');
-	
+
 	$prestd_obj = new prestudent();
 	if(!$prestd_obj->load($prestudent_id))
 		die('PrestudentID ist ungueltig');
-	
+
 	$prestd_obj->new = true;
 	if($prestd_obj->save())
 	{
 		$prestudent_id_neu=$prestd_obj->prestudent_id;
-		
+
 		if($prestd_obj->getLastStatus($prestudent_id))
 			$orgform_kurzbz = $prestd_obj->orgform_kurzbz;
 		else
 			$orgform_kurzbz = null;
-		
+
 		$prestd_obj = new prestudent();
-		
+
 		$prestd_obj->prestudent_id=$prestudent_id_neu;
 		$prestd_obj->status_kurzbz='Interessent';
 		$prestd_obj->studiensemester_kurzbz = $stsem_kurzbz;
@@ -113,7 +113,7 @@ if(isset($_POST['save']))
 		$prestd_obj->updatevon = $user;
 		$prestd_obj->orgform_kurzbz = $orgform_kurzbz;
 		$prestd_obj->new = true;;
-		
+
 		if(!$prestd_obj->save_rolle())
 		{
 			echo 'Fehler beim Speichern der Rolle:'.$prestd_obj->errormsg;
@@ -136,51 +136,56 @@ if(isset($_POST['filter']))
 	$stsem_obj->getAll();
 	if($std_obj->getTab($filter, 'nachname, vorname'))
 	{
-		echo '<table class="liste">';
-		echo '<tr>';
-		echo '<th>PersonID</td>';
-		echo '<th>Vorname</td>';
-		echo '<th>Nachname</td>';
-		echo '<th>Studiengang</td>';
-		echo '<th>Semester</td>';
-		echo '<th>Action</td>';
-		echo '</tr>';
-		$i=0;
-		foreach($std_obj->result as $row)
+		if(is_array($std_obj->result) && count($std_obj->result)>0)
 		{
-			$i++;
-			echo '<tr class="liste'.($i%2).'">';
-			echo '<td>'.$row->person_id.'</td>';
-			echo '<td>'.$row->vorname.'</td>';
-			echo '<td>'.$row->nachname.'</td>';
-			echo '<td>'.$stg_obj->kuerzel_arr[$row->studiengang_kz].'</td>';
-			echo '<td>'.$row->semester.'</td>';
-			echo '<td>';
-			echo '<form action="'.$_SERVER['PHP_SELF'].'" method="POST">';
-			echo '<input type="hidden" name="prestudent_id" value="'.$row->prestudent_id.'" />';
-			echo 'als neuen Interessenten in Studiensemester ';
-			echo '<SELECT name="stsem_kurzbz">';
-			foreach($stsem_obj->studiensemester as $row_stsem)
-			{
-				if($row_stsem->studiensemester_kurzbz==$stsem)
-					$selected='selected';
-				else
-					$selected='';
-				echo '<OPTION value="'.$row_stsem->studiensemester_kurzbz.'" '.$selected.'>'.$row_stsem->studiensemester_kurzbz.'</OPTION>';
-			}
-			echo '</SELECT>';
-			echo ' in Ausbildungssemester <SELECT name="ausbildungssemester">';
-			for($sem=1;$sem<=10;$sem++)
-			{
-				echo '<OPTION value="'.$sem.'">'.$sem.'</OPTION>';
-			}
-			echo '</SELECT>';
-			echo '<input type="submit" name="save" value="anlegen">';
-			echo '</form>';
-			echo '</td>';
+			echo '<table class="liste">';
+			echo '<tr>';
+			echo '<th>PersonID</td>';
+			echo '<th>Vorname</td>';
+			echo '<th>Nachname</td>';
+			echo '<th>Studiengang</td>';
+			echo '<th>Semester</td>';
+			echo '<th>Action</td>';
 			echo '</tr>';
+			$i=0;
+			foreach($std_obj->result as $row)
+			{
+				$i++;
+				echo '<tr class="liste'.($i%2).'">';
+				echo '<td>'.$row->person_id.'</td>';
+				echo '<td>'.$row->vorname.'</td>';
+				echo '<td>'.$row->nachname.'</td>';
+				echo '<td>'.$stg_obj->kuerzel_arr[$row->studiengang_kz].'</td>';
+				echo '<td>'.$row->semester.'</td>';
+				echo '<td>';
+				echo '<form action="'.$_SERVER['PHP_SELF'].'" method="POST">';
+				echo '<input type="hidden" name="prestudent_id" value="'.$row->prestudent_id.'" />';
+				echo 'als neuen Interessenten in Studiensemester ';
+				echo '<SELECT name="stsem_kurzbz">';
+				foreach($stsem_obj->studiensemester as $row_stsem)
+				{
+					if($row_stsem->studiensemester_kurzbz==$stsem)
+						$selected='selected';
+					else
+						$selected='';
+					echo '<OPTION value="'.$row_stsem->studiensemester_kurzbz.'" '.$selected.'>'.$row_stsem->studiensemester_kurzbz.'</OPTION>';
+				}
+				echo '</SELECT>';
+				echo ' in Ausbildungssemester <SELECT name="ausbildungssemester">';
+				for($sem=1;$sem<=10;$sem++)
+				{
+					echo '<OPTION value="'.$sem.'">'.$sem.'</OPTION>';
+				}
+				echo '</SELECT>';
+				echo '<input type="submit" name="save" value="anlegen">';
+				echo '</form>';
+				echo '</td>';
+				echo '</tr>';
+			}
+			echo '</table>';
 		}
-		echo '</table>';
+		else
+			echo 'Keine Eintraege gefunden';
 	}
 	else
 	{
@@ -190,4 +195,4 @@ if(isset($_POST['filter']))
 ?>
 <br /><font color="gray">Studienabbrecher, welche den selben Studiengang erneut besuchen wollen können hier angelegt werden. <br />Diese Seite ist NICHT für das Wiederholen eines einzelnen Semesters/Jahres gedacht!</font>
 </body>
-</html>
\ No newline at end of file
+</html>

From 3b168c912c2c99a33e76574b35463525adb9d84b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Wed, 7 Feb 2018 12:10:31 +0100
Subject: [PATCH 2/2] Fixed Warning if Image is missing

---
 vilesci/personen/personen_details.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/vilesci/personen/personen_details.php b/vilesci/personen/personen_details.php
index 796175774..c87826a7b 100644
--- a/vilesci/personen/personen_details.php
+++ b/vilesci/personen/personen_details.php
@@ -124,6 +124,7 @@ $matrikelnummer = (isset($_POST['matrikelnummer'])?$_POST['matrikelnummer']:'');
 $semester = (isset($_POST['semester'])?$_POST['semester']:'');
 $verband = (isset($_POST['verband'])?$_POST['verband']:'');
 $gruppe = (isset($_POST['gruppe'])?$_POST['gruppe']:'');
+$dms_id_lichtbild = '';
 
 if($uid!='')
 {