From d9b05ddfce80dd4de38c7d33f9d7fd708f4e1f04 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= Date: Wed, 7 Mar 2018 18:11:58 +0100 Subject: [PATCH] Cleanup --- cis/private/freifaecher/anmeldung.php | 95 ++++++++++++++------------- 1 file changed, 48 insertions(+), 47 deletions(-) diff --git a/cis/private/freifaecher/anmeldung.php b/cis/private/freifaecher/anmeldung.php index 85c450787..8b9eac33e 100644 --- a/cis/private/freifaecher/anmeldung.php +++ b/cis/private/freifaecher/anmeldung.php @@ -30,11 +30,11 @@ require_once('../../../include/lehrveranstaltung.class.php'); require_once('../../../include/phrasen.class.php'); require_once('../../../include/basis_db.class.php'); -$sprache = getSprache(); +$sprache = getSprache(); $p = new phrasen($sprache); if (!$db = new basis_db()) - $db=false; + die('Unable to Connect'); $user = get_uid(); @@ -42,38 +42,45 @@ $user = get_uid(); $stsem_obj = new studiensemester(); $stsem = $stsem_obj->getaktorNext(); -?> - - +?> + - - - <?php echo $p->t('freifach/freifaecherAnmeldung');?> + + + <?php echo $p->t('freifach/freifaecherAnmeldung');?> -

t('freifach/freifaecherAnmeldung');?>

- t('freifach/markierenFreifachFuerAnmeldung');?>: -
+
db_query('BEGIN'); //Zuerst die alten Eintraege herausloeschen... - $qry = "DELETE FROM campus.tbl_benutzerlvstudiensemester WHERE uid='$user' AND studiensemester_kurzbz='$stsem'"; - if(!$db->db_query($qry)) + $qry = "DELETE FROM campus.tbl_benutzerlvstudiensemester + WHERE + uid=".$db->db_add_param($user)." + AND studiensemester_kurzbz=".$db->db_add_param($stsem); + if (!$db->db_query($qry)) die($p->t('freifach/fehlerBeimAktualisieren')); //...dann die angeklickten FF hinzufuegen foreach ($_POST['chkbox'] as $elem) { - $qry = "INSERT INTO campus.tbl_benutzerlvstudiensemester(uid, lehrveranstaltung_id, studiensemester_kurzbz) VALUES('$user','$elem','$stsem');"; - if(!$db->db_query($qry)) + if (!is_numeric($elem)) + { + $db->db_query('ROLLBACK'); + die('Unbekannter Parameter gefunden - Operation wird abgebrochen'); + } + + $qry = "INSERT INTO campus.tbl_benutzerlvstudiensemester(uid, lehrveranstaltung_id, studiensemester_kurzbz) + VALUES(".$db->db_add_param($user).",".$db->db_add_param($elem).",".$db->db_add_param($stsem).");"; + if (!$db->db_query($qry)) { $db->db_query('ROLLBACK'); die($p->t('freifach/freifaecherNichtZugeteilt')); @@ -85,53 +92,60 @@ if(isset($_POST['submit'])) else { //Wenn keine Checkbox angeklickt wurde, alle Eintraege herausloeschen - $qry = "DELETE FROM campus.tbl_benutzerlvstudiensemester WHERE uid='$user' AND studiensemester_kurzbz='$stsem'"; - if(!$db->db_query($qry)) - die($p->t('freifach/fehlerBeimAktualisieren')); + $qry = "DELETE FROM campus.tbl_benutzerlvstudiensemester + WHERE uid=".$db->db_add_param($user)." AND studiensemester_kurzbz=".$db->db_add_param($stsem); + + if (!$db->db_query($qry)) + die($p->t('freifach/fehlerBeimAktualisieren')); else - echo "".$p->t('freifach/datenErfolgreichAktualisiert')."!
"; + echo "".$p->t('freifach/datenErfolgreichAktualisiert')."!
"; } } //Freifachzuteilungen holen -$qry = "SELECT * FROM campus.tbl_benutzerlvstudiensemester WHERE uid = '$user' AND studiensemester_kurzbz='$stsem'"; -if($result=$db->db_query($qry)) +$qry = "SELECT * FROM campus.tbl_benutzerlvstudiensemester +WHERE uid = ".$db->db_add_param($user)." AND studiensemester_kurzbz=".$db->db_add_param($stsem); + +if ($result = $db->db_query($qry)) { $ff = array(); - while($row=$db->db_fetch_object($result)) - $ff[] = $row->lehrveranstaltung_id; + while ($row = $db->db_fetch_object($result)) + { + $ff[] = $row->lehrveranstaltung_id; + } } else - echo $p->t('freifach/fehlerBeimAuslesen');; + echo $p->t('freifach/fehlerBeimAuslesen'); echo '
'; //Freifaecher laden $lv_obj = new lehrveranstaltung(); -if($lv_obj->load_lva('0',null,null,true,null,'bezeichnung')) +if ($lv_obj->load_lva('0', null, null, true, null, 'bezeichnung')) { $anz = count($lv_obj->lehrveranstaltungen); echo "
"; - $i=0; + $i = 0; echo ""; @@ -144,18 +158,5 @@ else die($p->t('freifach/fehlerBeimAuslesenFreifach')); } ?> - -
"; - foreach($lv_obj->lehrveranstaltungen as $row) + foreach ($lv_obj->lehrveranstaltungen as $row) { //Auftrennen in eine zweite Spalte bei der haelfte der Eintraege - if($i==intval($anz/2)) + if ($i == intval($anz / 2)) echo ""; - if(in_array($row->lehrveranstaltung_id,$ff)) + if (in_array($row->lehrveranstaltung_id, $ff)) $checked = "checked='true'"; else $checked = ''; //Wenn aktiv=false dann ist fuer dieses Lehrfach keine Anmeldung mehr moeglich - if($row->aktiv==false && $checked=='') + if ($row->aktiv == false && $checked == '') $disabled = "disabled='true'"; else $disabled = ""; - echo "$row->bezeichnung
"; + echo "\n"; + echo "$row->bezeichnung
"; $i++; } echo "
 
- - - - -
  -
- - - \ No newline at end of file +