diff --git a/cis/private/lvplan/stpl_week.php b/cis/private/lvplan/stpl_week.php index 56b06f292..edc50e8c2 100644 --- a/cis/private/lvplan/stpl_week.php +++ b/cis/private/lvplan/stpl_week.php @@ -368,7 +368,7 @@ if (isset($reservtodelete)) { if ($reservierung->load($delete_id)) { - if ($reservberechtigt || $reservierung->uid==$uid || $reservierung->insertvon==$uid) + if ($reservberechtigt && ($reservierung->insertvon==$uid || $reservierung->uid==$uid)) { if($reservierung->delete($delete_id)) $reservdelcount++; diff --git a/content/lvplanung/timetable-week.xul.php b/content/lvplanung/timetable-week.xul.php index 32042e9e8..18eaf6076 100644 --- a/content/lvplanung/timetable-week.xul.php +++ b/content/lvplanung/timetable-week.xul.php @@ -384,18 +384,6 @@ elseif ($aktion=='stpl_delete_single' || $aktion=='stpl_delete_block') $logdata = var_export($logdata_reservierung, true); $reservierung->delete($reservierung_id); $error_msg.=$reservierung->errormsg; - // Logeintrag zum mitloggen von geloeschten Reservierungen - if($error_msg=='') - { - $log = new log(); - $log->executetime = date('Y-m-d H:i:s'); - $log->sqlundo = ''; - $log->sql = 'DELETE FROM campus.tbl_reservierung WHERE reservierung_id='.$reservierung_id.'; LogData:'.$logdata; - $log->beschreibung = 'Löschen der Reservierung '.$reservierung_id; - $log->mitarbeiter_uid = $uid; - if(!$log->save(true)) - $error_msg.='Fehler: '.$log->errormsg; - } } } @@ -410,18 +398,6 @@ elseif ($aktion=='stpl_delete_single' || $aktion=='stpl_delete_block') $logdata = var_export($logdata_reservierung, true); $reservierung->delete($reservierung_id); $error_msg.=$reservierung->errormsg; - // Logeintrag zum mitloggen von geloeschten Reservierungen - if($error_msg=='') - { - $log = new log(); - $log->executetime = date('Y-m-d H:i:s'); - $log->sqlundo = ''; - $log->sql = 'DELETE FROM campus.tbl_reservierung WHERE reservierung_id='.$reservierung_id.'; LogData:'.$logdata; - $log->beschreibung = 'Löschen der Reservierung '.$reservierung_id; - $log->mitarbeiter_uid = $uid; - if(!$log->save(true)) - $error_msg.='Fehler: '.$log->errormsg; - } } } } diff --git a/include/reservierung.class.php b/include/reservierung.class.php index 2edac6844..da5ccdb12 100644 --- a/include/reservierung.class.php +++ b/include/reservierung.class.php @@ -21,6 +21,7 @@ */ require_once(dirname(__FILE__).'/basis_db.class.php'); require_once(dirname(__FILE__).'/datum.class.php'); +require_once(dirname(__FILE__).'/log.class.php'); class reservierung extends basis_db { @@ -242,7 +243,23 @@ class reservierung extends basis_db $qry = "DELETE FROM campus.tbl_reservierung WHERE reservierung_id=".$this->db_add_param($reservierung_id, FHC_INTEGER); if($this->db_query($qry)) + { + $reservierung = new reservierung($reservierung_id); + $logdata_reservierung = (array)$reservierung; + $logdata = var_export($logdata_reservierung, true); + $log = new log(); + $log->executetime = date('Y-m-d H:i:s'); + $log->sqlundo = ''; + $log->sql = 'DELETE FROM campus.tbl_reservierung WHERE reservierung_id='.$reservierung_id.'; LogData:'.$logdata; + $log->beschreibung = 'Löschen der Reservierung '.$reservierung_id; + $log->mitarbeiter_uid = $this->uid; + if(!$log->save(true)) + { + $this->errormsg = 'Fehler: '.$log->errormsg; + return false; + } return true; + } else { $this->errormsg = 'Fehler beim Loeschen der Reservierung'; diff --git a/include/wochenplan.class.php b/include/wochenplan.class.php index 5b4e47894..9af13b8ab 100644 --- a/include/wochenplan.class.php +++ b/include/wochenplan.class.php @@ -2754,7 +2754,7 @@ class wochenplan extends basis_db { while ($row = $this->db_fetch_object($result)) { - if ($reservberechtigt || $row->uid == $user_uid || $row->insertvon == $user_uid) + if ($reservberechtigt && ($row->uid == $user_uid || $row->insertvon == $user_uid)) { $deleteberechtigt = true; $reservtodelete[] = $row->reservierung_id;