From 5d85be8937609aec79a35ff57553cd475fd96f62 Mon Sep 17 00:00:00 2001 From: Marvin Flock Date: Tue, 18 Mar 2025 20:34:09 +0100 Subject: [PATCH] fix: implement some minor fixes --- .../com_sportsmanager/sportsmanager.php | 88 ++++--------------- 1 file changed, 15 insertions(+), 73 deletions(-) diff --git a/src/structure/components/com_sportsmanager/sportsmanager.php b/src/structure/components/com_sportsmanager/sportsmanager.php index 723ea03..42e9aa8 100644 --- a/src/structure/components/com_sportsmanager/sportsmanager.php +++ b/src/structure/components/com_sportsmanager/sportsmanager.php @@ -5,7 +5,7 @@ // kein direkter Zugriff defined('_JEXEC') or die('Restricted access'); -define("SPORTS_MANAGER_VERSION", "2.0.0"); +const SPORTS_MANAGER_VERSION = "2.0.0"; //ini_set('display_errors', '1'); @@ -28,25 +28,24 @@ function laufzeitAusgeben($zwischenstand = NULL) { $laufzeit_delta_zwischenstand = $laufzeit_aktuell - $laufzeit_zwischenstand; $laufzeit_delta_zwischenstand = $laufzeit_aktuell - (empty($zwischenstand) ? $laufzeit_beginn : $laufzeit_zwischenstand); $laufzeit_zwischenstand = $laufzeit_aktuell; - echo("" . (!empty($zwischenstand) ? htmlentities_utf8($zwischenstand) : "Seitenaufbau in") . ": " . sprintf('%f', $laufzeit_delta) . " Sekunden" . (empty($zwischenstand) ? "" : (sprintf(' (+%f)', $laufzeit_delta_zwischenstand))) . ".
"); + echo(!empty($zwischenstand) ? htmlentities_utf8($zwischenstand) : "Seitenaufbau in" . ": " . sprintf('%f', $laufzeit_delta) . " Sekunden" . (empty($zwischenstand) ? "" : (sprintf(' (+%f)', $laufzeit_delta_zwischenstand))) . ".
"); } laufzeitInit(); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'tools.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'tools.php'); setMinMemoryLimit('64M'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'api.php'); - -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'admin.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'controller.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'database.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'mathparser.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'views/sportsmanager/view.html.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'views/sportsmanager/view.json.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'views/sportsmanager/view_admin.php'); -require_once (JPATH_COMPONENT . DIRECTORY_SEPARATOR . 'views/sportsmanager/view_ticker.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'api.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'admin.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'controller.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'database.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'mathparser.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'views/sportsmanager/view.html.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'views/sportsmanager/view.json.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'views/sportsmanager/view_admin.php'); +require_once (JPATH_ROOT . DIRECTORY_SEPARATOR . 'components' . DIRECTORY_SEPARATOR . 'com_sportsmanager' . DIRECTORY_SEPARATOR . 'views/sportsmanager/view_ticker.php'); jimport('joomla.filesystem.file'); @@ -79,7 +78,7 @@ if ($task == "spielerbild") { terminDokument(); } else if ($task == "spieler_details") { spielerDetails(); -} else if (substr($task, 0, 6) == "admin_") { +} else if (str_starts_with($task, "admin_")) { switch ($task) { case 'admin_einstellungen': adminEinstellungen(); @@ -1067,7 +1066,6 @@ function redirectSportsManagerURL($weitereParameter = NULL, $nachricht = '') { } $app->redirect(SportsManagerURL($weitereParameter), 200); exit; - return; } function redirectCurrentURL() { @@ -1109,7 +1107,7 @@ function berechnungen() { . "\n WHERE " . (empty($session_id) ? ("session_id < SUBTIME(NOW(), '00:10:00')") : ("session_id = '" . $session_id . "'")) . "\n ), 1, 0)"; $db->setQuery($query); - if (!$result = $db->execute()) { + if (!$db->execute()) { die($db->stderr(true)); } $berechnungen = $db->loadResult(); @@ -1155,7 +1153,7 @@ function benutzerModerator() { . "\n FROM #__sportsmanager_moderator" . "\n WHERE moderator_user_id = $user_id"; $db->setQuery($query); - if (!$result = $db->execute()) { + if (!$db->execute()) { die($db->stderr(true)); } $gefunden = $db->loadResult(); @@ -4040,62 +4038,6 @@ function teamSpielplanXML() { die(); } - function bildKopierenAngepasst($quelle, $ziel, $ziel_breite, $ziel_hoehe, $zuschneiden) { - $len = strlen($ziel); - if ($len < 4 || $ziel[$len - 4] != ".") - return false; - $ext = strtolower(substr($ziel, $len - 3)); - if ($ext != "jpg" && $ext != "png") - return false; - $quelle_image = $ext == "png" ? imagecreatefrompng($quelle) : imagecreatefromjpeg($quelle); - if ($quelle_image === false) - return false; - $quelle_breite = imagesx($quelle_image); - $quelle_hoehe = imagesy($quelle_image); - if ($quelle_breite == $ziel_breite && $quelle_hoehe == $ziel_hoehe) - return JFile::copy($quelle, $ziel); - - $ziel_image = imagecreatetruecolor($ziel_breite, $ziel_hoehe); - $hintergrund = $ext == "png" ? imagecolorallocatealpha($ziel_image, 0, 0, 0, 127) : imagecolorallocate($ziel_image, 64, 64, 64); - imagefill($ziel_image, 0, 0, $hintergrund); - $quelle_proportionen = $quelle_breite / $quelle_hoehe; - $ziel_proportionen = $ziel_breite / $ziel_hoehe; - if ($zuschneiden) { - if ($ziel_proportionen >= $quelle_proportionen) { - $quelle_teilhoehe = round($quelle_breite / $ziel_proportionen); - if (!imagecopyresampled($ziel_image, $quelle_image, 0, 0, 0, round(($quelle_hoehe - $quelle_teilhoehe) / 2), $ziel_breite, $ziel_hoehe, $quelle_breite, $quelle_teilhoehe)) - return false; - } else { - $quelle_teilbreite = round($quelle_hoehe * $ziel_proportionen); - if (!imagecopyresampled($ziel_image, $quelle_image, 0, 0, round(($quelle_breite - $quelle_teilbreite) / 2), 0, $ziel_breite, $ziel_hoehe, $quelle_teilbreite, $quelle_hoehe)) - return false; - } - } else { - if ($ziel_proportionen >= $quelle_proportionen) { - $ziel_teilbreite = round($ziel_hoehe * $quelle_proportionen); - if (!imagecopyresampled($ziel_image, $quelle_image, round(($ziel_breite - $ziel_teilbreite) / 2), 0, 0, 0, $ziel_teilbreite, $ziel_hoehe, $quelle_breite, $quelle_hoehe)) - return false; - } else { - $ziel_teilhoehe = round($ziel_breite / $quelle_proportionen); - if (!imagecopyresampled($ziel_image, $quelle_image, 0, round(($ziel_hoehe - $ziel_teilhoehe) / 2), 0, 0, $ziel_breite, $ziel_teilhoehe, $quelle_breite, $quelle_hoehe)) - return false; - } - } - ob_start(); - if ($ext == "png") { - imagesavealpha($ziel_image, true); - if (!imagepng($ziel_image)) - return false; - } else { - if (!imagejpeg($ziel_image)) - return false; - } - $output = ob_get_contents(); - ob_end_clean(); - JFile::write($ziel, $output); - return true; - } - function bildAnpassen($typ, $id = 0) { global $sportsmanager_joomla_path; $jinput = JFactory::getApplication()->input;