From cf52c718e863bd9323dddd6a37e344bf2d8e11d8 Mon Sep 17 00:00:00 2001 From: Marvin Flock Date: Thu, 28 Aug 2025 22:24:13 +0200 Subject: [PATCH] chore: some review fixes --- .../com_sportsmanager/sportsmanager.php | 26 ++++--------------- 1 file changed, 5 insertions(+), 21 deletions(-) diff --git a/src/structure/components/com_sportsmanager/sportsmanager.php b/src/structure/components/com_sportsmanager/sportsmanager.php index b1348c5..16e794b 100644 --- a/src/structure/components/com_sportsmanager/sportsmanager.php +++ b/src/structure/components/com_sportsmanager/sportsmanager.php @@ -2099,7 +2099,7 @@ function getTabelleSpieltag($veranstaltung,$spieltag,$mannschaften = "",$offset_ -function erstelleSetzliste($teams) +function erstelleSetzliste($teams): array { $seedingPairs[4] = [[1, 4], [2, 3]]; @@ -2237,7 +2237,7 @@ function getAnzahlRunden($anzahlTeams) } } -function getRundenTitel($anzahlTeams) +function getRundenTitel($anzahlTeams): string { if ($anzahlTeams >= 1 && $anzahlTeams <= 2) return Text::_('COM_SPORTSMANAGER_FINAL'); @@ -2266,11 +2266,7 @@ function turnierbaumAnzeigen($veranstaltung): void //Infos Veranstaltung $query = "SELECT * FROM #__sportsmanager_veranstaltung WHERE veranstaltung_id = $veranstaltungId"; - $db->setQuery($query); - if (!$db->execute()) { - die($db->stderr(true)); - } - $rows = $db->loadObjectList(); + $rows = loadObjectList($db, $query); if (count($rows) < 1) die("Wrong id!"); $veranstaltung = $rows[0]; @@ -2279,11 +2275,7 @@ function turnierbaumAnzeigen($veranstaltung): void . "\n FROM #__sportsmanager_team" . "\n WHERE veranstaltung_id = $veranstaltungId" . "\n ORDER BY setzliste_nr;"; - $db->setQuery($query); - if (!$db->execute()) { - die($db->stderr(true)); - } - $teams_unformatiert = json_decode(json_encode($db->loadObjectList()), true); + $teams_unformatiert = json_decode(json_encode(loadObjectList($db, $query)), true); foreach($teams_unformatiert AS $team){ $teams[$team['setzliste_nr']] = $team; @@ -2352,11 +2344,7 @@ function turnierbaumAnzeigen($veranstaltung): void . "\n LEFT JOIN #__sportsmanager_veranstaltung AS t4 ON t2.veranstaltung_id = t4.veranstaltung_id" . "\n WHERE t4.veranstaltung_id = $veranstaltungId" . "\n ORDER BY t4.veranstaltung_id, t1.spieltag, t1.spiel_nr;"; - $db->setQuery($query); - if (!$db->execute()) { - die($db->stderr(true)); - } - $begegnungen_unformatiert = json_decode(json_encode($db->loadObjectList()), true); + $begegnungen_unformatiert = json_decode(json_encode(loadObjectList($db, $query)), true); foreach($begegnungen_unformatiert as $begegnung) { $begegnungen[$begegnung['Spieltag_Nr']][$begegnung['Spiel_Nr']]['Id'] = $begegnung['begegnung_id']; @@ -2397,10 +2385,6 @@ function turnierbaumAnzeigen($veranstaltung): void HTML_sportsmanager::turnierbaumAnzeigen($veranstaltung,$begegnungen); } - - - - function begegnungenAnzeigen($veranstaltung, $verein_id, $spielort_id, $zeitfenster, $ausgetragene_filtern = false, $auszutragende_filtern = false, $zwischenergebnisse_filtern = false, $titel = "", $umgekehrte_reihenfolge = false, $aus_uebersicht = false, $praesentation = 0, $year = NULL, $month = NULL) { $db = getDatabase();