LMO Addons ausprobieren
+ Entwicklungsumgebung: + Ligaseite · + Admin-Bereich · + Tippspiel · + Postfach +'; @@ -1191,16 +1251,64 @@ function fileContent($htmlContent = false) { * @package classLib * @access privat */ - function strafen(&$team, $spTag) { - $stda = isset($team['team']->keyValues['STDA']) ? $team['team']->keyValues['STDA'] : 0; - if ($stda == $spTag || ($stda == 0 && $spTag == 1)) { - $team['pPkt'] -= isset($team['team']->keyValues['SP']) ? $team['team']->keyValues['SP'] : 0; - $team['pTor'] -= isset($team['team']->keyValues['TOR1']) ? $team['team']->keyValues['TOR1'] : 0; - $team['mTor'] -= isset($team['team']->keyValues['TOR2']) ? abs($team['team']->keyValues['TOR2']) : 0; - if ($this->options->keyValues['MinusPoints'] == 2 && isset($team['team']->keyValues['SM'])) { - $team['mPkt'] -= $team['team']->keyValues['SM']; - } + function strafen(&$team) { + $keyValues = $team['team']->keyValues; + $team['pPkt'] -= isset($keyValues['SP']) ? $keyValues['SP'] : 0; + // TOR1/TOR2 wie in der Ligadatei: lmo-adminteams.php speichert "+x" als -x + $team['pTor'] -= isset($keyValues['TOR1']) ? $keyValues['TOR1'] : 0; + $team['mTor'] -= isset($keyValues['TOR2']) ? $keyValues['TOR2'] : 0; + if ($this->options->keyValues['MinusPoints'] == 2 && isset($keyValues['SM'])) { + $team['mPkt'] -= $keyValues['SM']; + } + } + + /** + * Ob die Strafe eines Teams in dieser Tabelle zaehlt (wie lmo-calctable.php): + * nicht in Heim-/Auswaertstabelle, in der Hinrunde nur Strafen ab einem Spieltag der Hinrunde, + * in der Rueckrunde nur solche ab einem Spieltag der Rueckrunde, und erst wenn der Spieltag + * STDA erreicht ist (Ergebnisse hat). + * + * @access private + * @param array $team Tabellenzeile + * @param string $tableArt Art der Tabelle + * @param integer $lastPlayed letzter Spieltag mit einem Ergebnis + * @return boolean + */ + function strafeFaellig($team, $tableArt, $lastPlayed) { + $stda = isset($team['team']->keyValues['STDA']) ? (int) $team['team']->keyValues['STDA'] : 0; + $half = (int) ($this->spieltageCount() / 2); + if ($tableArt == 'heim' || $tableArt == 'gast') { + return false; + } + if (($tableArt == 'hin' && $stda > $half) || ($tableArt == 'rueck' && $stda <= $half)) { + return false; } + return $stda <= $lastPlayed; + } + + /** + * Bucht ein Ergebnis fuer ein Team: Tore, Spiele, Sieg/Remis/Niederlage und Punkte. + * + * @access private + */ + function addResult(&$row, $own, $other, $pointsForWin, $pointsForDraw, $pointsForLost) { + $row['pTor'] += $own; + $row['mTor'] += $other; + $row['spiele']++; + if ($own > $other) { + $result = array('s', $pointsForWin, $pointsForLost); + } + elseif ($own < $other) { + $result = array('n', $pointsForLost, $pointsForWin); + } + else { + $result = array('u', $pointsForDraw, $pointsForDraw); + } + if (isset($row[$result[0]])) { + $row[$result[0]]++; + } + $row['pPkt'] += $result[1]; + $row['mPkt'] += $result[2]; } /** @@ -1223,9 +1331,9 @@ function strafen(&$team, $spTag) { * $myArray[0..n]["possp"] Position an dem $key Spieltag * * @access public - * @param integer spTag Spieltag für den die Tabelle berechnet werden soll. - * @param string tableArt Art der Tabelle "all","heim","gast","hin","rueck" - * @param boolean possp wenn true wird für jeden Spieltag die Position bestimmt (Statistik) + * @param integer $spTag Spieltag für den die Tabelle berechnet werden soll. + * @param string $tableArt Art der Tabelle "all","heim","gast","hin","rueck" + * @param boolean $possp wenn true wird für jeden Spieltag die Position bestimmt (Statistik) * @return array Tabelle des Angegebenen Spieltages */ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { @@ -1253,6 +1361,19 @@ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { 'possp' => array() ); } + // Strafen: einmal je Team, sobald faellig - auch wenn das Team an dem Spieltag spielfrei ist + $lastPlayed = 0; + $bestraft = array(); + $strafenBuchen = function () use (&$tableArray, &$bestraft, &$lastPlayed, $tableArt) { + foreach ($tableArray as $index => $row) { + if (empty($bestraft[$index]) && $this->strafeFaellig($row, $tableArt, $lastPlayed)) { + $this->strafen($tableArray[$index]); + $bestraft[$index] = true; + } + } + }; + $strafenBuchen(); // Strafen ab Spieltag 0 + $processed = array(); // gewertete Spieltage, fuer den direkten Vergleich foreach ($this->spieltage as $spieltag) { if ($spieltag->getModus() > 0) { continue; @@ -1275,6 +1396,7 @@ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { } continue; } + $processed[$spieltag->nr] = true; foreach ($spieltag->partien as $partie) { $heimCount = -1; $gastCount = -1; @@ -1302,12 +1424,18 @@ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { $gastCount = $count; } } - // Strafen berücksichtigen - if ($heimCount != -1) { - $this->strafen($tableArray[$heimCount], $spTagCount); + if ($partie->valuateGame() != -1) { + $lastPlayed = $spieltag->nr; } - if ($gastCount != -1) { - $this->strafen($tableArray[$gastCount], $spTagCount); + // ET=3: beidseitiges Ergebnis, gilt fuer beide Teams aus Sicht der Heimmannschaft + if ($partie->getParameter('ET') == 3 && $partie->hTore > -1 && $partie->gTore > -1) { + if ($tableArt != 'gast') { + $this->addResult($tableArray[$heimCount], $partie->hTore, $partie->gTore, $pointsForWin, $pointsForDraw, $pointsForLost); + } + if ($tableArt != 'heim') { + $this->addResult($tableArray[$gastCount], $partie->hTore, $partie->gTore, $pointsForWin, $pointsForDraw, $pointsForLost); + } + continue; } if ($partie->hTore > -1) { @@ -1397,11 +1525,13 @@ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { } } } // foreach Partien + $strafenBuchen(); //Zusätzliche Berechnung der Position für den jeweiligen Spieltag (Statisktik) if ($possp == true) { for ($i = 0; $i < count($tableArray); $i++) { // Tordiff. $tableArray[$i]['dTor'] = $tableArray[$i]['pTor'] - $tableArray[$i]['mTor']; } + $this->directContext = array('rounds' => $processed, 'art' => $tableArt); $tableArray = $this->sortTable($tableArray); for ($i = 0; $i < count($tableArray); $i++) { $tableArray[$i]['possp'][$spTagCount - 1] = $tableArray[$i]['pos']; @@ -1418,7 +1548,47 @@ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { for ($i = 0; $i < count($tableArray); $i++) { // Tordiff. $tableArray[$i]['dTor'] = $tableArray[$i]['pTor'] - $tableArray[$i]['mTor']; } - return $this->sortTable($tableArray); + $this->directContext = array('rounds' => $processed, 'art' => $tableArt); + $tableArray = $this->sortTable($tableArray); + $this->directContext = null; + if ($this->options->keyValues['HandS'] == 1 && $tableArt == 'all') { + $tableArray = $this->sortHandicap($tableArray, $spTag); + } + return $tableArray; + } + + /** + * Handicap-Reihenfolge (Option HandS, nur Gesamttabelle), wie lmo-calctable.php: + * Der Wert HS des Spieltags enthaelt je Platz der errechneten Tabelle zwei Ziffern mit dem + * Platz, auf den das dort stehende Team gesetzt wird. Bei gleichen Ziffern stehen die Teams + * in umgekehrter Reihenfolge der errechneten Tabelle. + * Fuer die Tabelle der ganzen Saison gilt die Reihenfolge des aktuellen Spieltags (Actual), + * sonst die des angegebenen Spieltags. + * + * @access protected + * @param array $tableArray sortierte Tabelle + * @param integer $spTag Spieltag der Tabelle + * @return array + */ + function sortHandicap($tableArray, $spTag) { + if ($spTag >= $this->spieltageCount()) { + $spTag = max(1, (int) $this->options->keyValues['Actual']); + } + $spieltag = $this->spieltagForNumber($spTag); + $handicap = is_object($spieltag) ? (string) $spieltag->getParameter('HS') : ''; + if ($handicap === '' || $handicap == 0) { + return $tableArray; + } + $order = array_keys($tableArray); + usort($order, function ($a, $b) use ($handicap) { + return array(intval(substr($handicap, $a * 2, 2)), $b) <=> array(intval(substr($handicap, $b * 2, 2)), $a); + }); + $sorted = array(); + foreach ($order as $pos => $index) { + $sorted[$pos] = $tableArray[$index]; + $sorted[$pos]['pos'] = $pos + 1; + } + return $sorted; } /** @@ -1428,52 +1598,33 @@ function calcTable($spTag = 1, $tableArt = 'all', $possp = false) { * ->keyValues['Kegel'] = 1 -> Pluspunkte - NegativPunkte - erziehlte Tore - Tor Differenz * * @access protected - * @param array tableArray Die zu sortierende Tabelle + * @param array $tableArray Die zu sortierende Tabelle * @return array */ function sortTable($tableArray) { + // auch fuer eine leere Tabelle definiert + $sort_pPkt = $sort_mPkt = $sort_pTor = $sort_mTor = $sort_dTor = $sort_nr = array(); foreach ($tableArray as $table) { $sort_pPkt[] = $table['pPkt']; $sort_mPkt[] = $this->options->keyValues['MinusPoints'] == 2 ? $table['mPkt'] : 0; $sort_pTor[] = $table['pTor']; $sort_mTor[] = $table['mTor']; $sort_dTor[] = $table['dTor']; + $sort_nr[] = (int) $table['team']->nr; } // ASC = auf-, DESC = absteigend + // Bei voelligem Gleichstand steht das Team mit der hoeheren Nummer vorn (wie lmo-calctable.php) if ($this->options->keyValues['Kegel'] == 1) { // Sortierung Punkte,erzielte Tore - array_multisort($sort_pPkt, SORT_DESC, $sort_mPkt, SORT_ASC, $sort_pTor, SORT_DESC, $sort_dTor, SORT_DESC, $tableArray, SORT_DESC); + array_multisort($sort_pPkt, SORT_DESC, $sort_mPkt, SORT_ASC, $sort_pTor, SORT_DESC, $sort_dTor, SORT_DESC, $sort_nr, SORT_DESC, $tableArray, SORT_DESC); } else { // Sortierung PlusPkt,Tordiff - array_multisort($sort_pPkt, SORT_DESC, $sort_mPkt, SORT_ASC, $sort_dTor, SORT_DESC, $sort_pTor, SORT_DESC, $sort_mTor, SORT_ASC, $tableArray, SORT_DESC); + array_multisort($sort_pPkt, SORT_DESC, $sort_mPkt, SORT_ASC, $sort_dTor, SORT_DESC, $sort_pTor, SORT_DESC, $sort_mTor, SORT_ASC, $sort_nr, SORT_DESC, $tableArray, SORT_DESC); } // BEGIN Direkter Vergleich if ($this->options->keyValues['Direct'] == 1) { - $subteams = array(); - $pPkt = 0; - $mPkt = 0; - for ($abc = 0; $abc < count($tableArray); $abc++) { - if ($pPkt == $tableArray[$abc]['pPkt'] && $mPkt == $tableArray[$abc]['mPkt']) { - $subteams[$tableArray[$abc]['team']->nr] = $tableArray[$abc]['team']; - } - else { - if (count($subteams) > 1) { - $tmp_table = $this->calcTableforTeams($subteams); - $tmp_tablearray = $tableArray; - $nextpos = $abc - count($tmp_table); - for ($b = 0; $b < count($tmp_table); $b++) { - for ($f = $nextpos; $f < $abc; $f++) { - if ($tmp_tablearray[$f]['team'] === $tmp_table[$b]['team']) { - $tableArray[$nextpos + $b] = $tmp_tablearray[$f]; - } - } - } - } // END if (count($subteams)>1) - $subteams = array(); - $pPkt = $tableArray[$abc]['pPkt']; - $mPkt = $tableArray[$abc]['mPkt']; - $subteams[$tableArray[$abc]['team']->nr] = $tableArray[$abc]['team']; - } - } // END for ($abc = 0; $abc < count($tableArray); $abc++) + // Minuspunkte zaehlen nur bei der Zwei-Punkte-Regel (wie lmo-calctable.php) + $keys = $this->options->keyValues['MinusPoints'] == 2 ? array('pPkt', 'mPkt') : array('pPkt'); + $tableArray = $this->sortTiedGroups($tableArray, $keys); } // END Direkter Vergleich for ($i = 0; $i < count($tableArray); $i++) { // Position setzen $tableArray[$i]['pos'] = $i + 1; @@ -1481,11 +1632,73 @@ function sortTable($tableArray) { return $tableArray; } + /** + * Direkter Vergleich: Gruppen gleichauf liegender Teams (gleiche Werte in $keys) werden + * nach der Tabelle ihrer Partien untereinander (calcTableforTeams) sortiert, siehe + * compareDirect(). Bleiben Teams auch dort gleichauf, behalten sie ihre bisherige + * Reihenfolge (z.B. nach Tordifferenz) - wie lmo-calctable.php. + * + * @access protected + * @param array $tableArray sortierte Tabelle + * @param array $keys Spalten, die für einen Gleichstand übereinstimmen müssen + * @return array + */ + function sortTiedGroups($tableArray, $keys) { + $count = count($tableArray); + $start = 0; + for ($i = 1; $i <= $count; $i++) { + $tied = $i < $count; + foreach ($keys as $key) { + $tied = $tied && $tableArray[$i][$key] == $tableArray[$start][$key]; + } + if ($tied) { + continue; + } + $size = $i - $start; + if ($size > 1) { + $group = array_slice($tableArray, $start, $size); + $subteams = array(); + foreach ($group as $row) { + $subteams[$row['team']->nr] = $row['team']; + } + $direct = array(); + foreach ($this->calcTableforTeams($subteams) as $directRow) { + $direct[$directRow['team']->nr] = $directRow; + } + $order = array_keys($group); + usort($order, function ($a, $b) use ($group, $direct) { + $result = $this->compareDirect($direct[$group[$a]['team']->nr], $direct[$group[$b]['team']->nr]); + return $result != 0 ? $result : $a - $b; // sonst bisherige Reihenfolge + }); + foreach ($order as $b => $index) { + $tableArray[$start + $b] = $group[$index]; + } + } + $start = $i; + } + return $tableArray; + } + + /** + * Vergleich zweier Zeilen der Tabelle des direkten Vergleichs (calcTableforTeams): + * Punkte, Minuspunkte (nur bei der Zwei-Punkte-Regel), Tordifferenz, Tore; + * bei Kegelwertung Tore vor Tordifferenz. Negativ: $a steht vor $b. + * + * @access protected + * @return integer + */ + function compareDirect($a, $b) { + $minus = $this->options->keyValues['MinusPoints'] == 2; + $goals = $this->options->keyValues['Kegel'] == 1 ? array('pTor', 'dTor') : array('dTor', 'pTor'); + return array($b['pPkt'], $minus ? $a['mPkt'] : 0, $b[$goals[0]], $b[$goals[1]]) + <=> array($a['pPkt'], $minus ? $b['mPkt'] : 0, $a[$goals[0]], $a[$goals[1]]); + } + /** * Errechnet eine Tabelle für den Direkten vergleich von mehreren Mannschaften * * @access protected - * @param array subteams Alle Teams die Miteinander verglichen werden sollen + * @param array $subteams Alle Teams die Miteinander verglichen werden sollen * @return array */ function calcTableforTeams($subteams) { @@ -1505,7 +1718,13 @@ function calcTableforTeams($subteams) { 'mPkt' => 0 ); } + // nur die Partien der Tabelle, die calcTable() gerade sortiert (Spieltage, Heim/Gast) + $context = $this->directContext; + $art = $context === null ? 'all' : $context['art']; foreach ($this->spieltage as $spieltag) { + if ($spieltag->getModus() > 0 || ($context !== null && empty($context['rounds'][$spieltag->nr]))) { + continue; + } foreach ($spieltag->partien as $partie) { if ($partie->spielEnde == 2) { // Nach Verlängerung $pointsForWin = $this->options->keyValues['XtraS']; @@ -1532,6 +1751,15 @@ function calcTableforTeams($subteams) { } if ($heimCount == -1 OR $gastCount == -1) continue; + // Heim-/Auswaertstabelle: die andere Seite der Partie wird danach zurueckgesetzt + $before = array($heimCount => $tableArray[$heimCount], $gastCount => $tableArray[$gastCount]); + // ET=3: beidseitiges Ergebnis, gilt fuer beide Teams aus Sicht der Heimmannschaft + if ($partie->getParameter('ET') == 3 && $partie->hTore > -1 && $partie->gTore > -1) { + $this->addResult($tableArray[$heimCount], $partie->hTore, $partie->gTore, $pointsForWin, $pointsForDraw, $pointsForLost); + $this->addResult($tableArray[$gastCount], $partie->hTore, $partie->gTore, $pointsForWin, $pointsForDraw, $pointsForLost); + $this->keepSide($tableArray, $before, $heimCount, $gastCount, $art); + continue; + } if ($partie->hTore > -1) { // Tore für Heim hinzufügen $tableArray[$heimCount]['pTor'] += $partie->hTore; @@ -1577,9 +1805,10 @@ function calcTableforTeams($subteams) { elseif ($partie->gTore == -2) { // O:0 Tore Gast gewinnt $tableArray[$heimCount]['mPkt'] += $pointsForWin; $tableArray[$gastCount]['pPkt'] += $pointsForWin; - $tableArray[$heimCount]['mPkt'] += $pointsForLost; - $tableArray[$gastCount]['pPkt'] += $pointsForLost; + $tableArray[$heimCount]['pPkt'] += $pointsForLost; + $tableArray[$gastCount]['mPkt'] += $pointsForLost; } + $this->keepSide($tableArray, $before, $heimCount, $gastCount, $art); } // foreach Partien } // foreach Spieltage for ($i = 0; $i < count($tableArray); $i++) { // Tordiff. @@ -1588,6 +1817,21 @@ function calcTableforTeams($subteams) { return $this->sortDirectTable($tableArray); } + /** + * Heimtabelle: nur die Heimmannschaft einer Partie wird gewertet, Auswaertstabelle nur die + * Gastmannschaft. Setzt die Zeile der anderen Seite auf den Stand vor der Partie zurueck. + * + * @access private + */ + function keepSide(&$tableArray, $before, $heimCount, $gastCount, $art) { + if ($art == 'heim') { + $tableArray[$gastCount] = $before[$gastCount]; + } + elseif ($art == 'gast') { + $tableArray[$heimCount] = $before[$heimCount]; + } + } + /** * Sortiert die errechnete Tabelle bei Direktem Vergleich und gibt diese als Array zurück * Ist seperat da bei vielen Ligen die Sortierung bei Direktem Vergleich @@ -1598,7 +1842,13 @@ function calcTableforTeams($subteams) { * @return array */ function sortDirectTable($tableArray) { - return liga::sortTable($tableArray); + // ohne erneuten direkten Vergleich: sortTiedGroups() wertet nur einmal aus + // (wie lmo-calctable.php), sonst rechnet sortTable() die Gruppe endlos weiter + $direct = $this->options->keyValues['Direct']; + $this->options->keyValues['Direct'] = 0; + $tableArray = liga::sortTable($tableArray); + $this->options->keyValues['Direct'] = $direct; + return $tableArray; } } // End Class Liga diff --git a/lmo/addon/classlib/classes/ligaFussball.class.php b/lmo/addon/classlib/classes/ligaFussball.class.php index 0b976435..1ca6d415 100644 --- a/lmo/addon/classlib/classes/ligaFussball.class.php +++ b/lmo/addon/classlib/classes/ligaFussball.class.php @@ -21,6 +21,10 @@ class ligaFussball extends liga { * @return array */ function sortTable($tableArray) { + if (empty($tableArray)) { + return $tableArray; + } + $sort_pPkt = $sort_spiele = $sort_dTor = $sort_pTor = array(); foreach($tableArray as $table) { $sort_pPkt[] = $table['pPkt']; $sort_spiele[] = $table['spiele']; @@ -29,56 +33,54 @@ function sortTable($tableArray) { } // ASC = auf-, DESC = absteigend array_multisort($sort_pPkt, SORT_DESC, $sort_spiele, SORT_ASC, $sort_dTor, SORT_DESC, $sort_pTor, SORT_DESC, $tableArray, SORT_DESC); - for ($i = 0; $i < count($tableArray); $i++) { - if ($i > 200) { - die('Fehler im Script'); + $count = count($tableArray); + for ($i = 0; $i + 1 < $count; $i++) { + $a = $tableArray[$i]['team']; + $b = $tableArray[$i + 1]['team']; + if ($sort_pPkt[$i] != $sort_pPkt[$i + 1] || $sort_spiele[$i] != $sort_spiele[$i + 1] || $sort_dTor[$i] != $sort_dTor[$i + 1] || $sort_pTor[$i] != $sort_pTor[$i + 1]) { + continue; } - if ( $sort_pPkt[$i] == $sort_pPkt[$i + 1] && $sort_spiele[$i] == $sort_spiele[$i + 1] && $sort_dTor[$i] == $sort_dTor[$i + 1] && $sort_pTor[$i] == $sort_pTor[$i + 1]) { - // Partien für den Direkten Vergleich suchen. - $allParties = $this->allPartieForTeams($tableArray[$i]['team'], $tableArray[$i + 1]['team'], true); - foreach ($allParties as $partie) { - $tore[$partie->heim->nr] += $partie->hTore > -1 ? $partie->hTore : 0; - $tore[$partie->gast->nr] += $partie->gTore > -1 ? $partie->gTore : 0; - $gastTore[$partie->gast->nr] = $partie->gTore > -1 ? $partie->gTore : 0; - } - // das Gesamtergebnis aus Hin- und Rückspiel im direkten Vergleich - if ($tore[$tableArray[$i]['team']->nr] == $tore[$tableArray[$i + 1]['team']->nr]) { + // Partien für den Direkten Vergleich + $tore = array($a->nr => 0, $b->nr => 0); + $gastTore = array($a->nr => 0, $b->nr => 0); + foreach ((array) $this->allPartieForTeams($a, $b, true) as $partie) { + $tore[$partie->heim->nr] += max(0, (int) $partie->hTore); + $tore[$partie->gast->nr] += max(0, (int) $partie->gTore); + $gastTore[$partie->gast->nr] += max(0, (int) $partie->gTore); + } + // das Gesamtergebnis aus Hin- und Rückspiel im direkten Vergleich + if ($tore[$a->nr] == $tore[$b->nr]) { + if ($gastTore[$a->nr] != $gastTore[$b->nr]) { // die Anzahl der auswärts erzielten Tore im direkten Vergleich - if ( $gastTore[$tableArray[$i]['team']->nr] == $gastTore[$tableArray[$i + 1]['team']->nr] ) { - unset ($tore); - // die Anzahl aller auswärts erzielten Tore - foreach ($this->spieltage as $spieltag) { - foreach ($spieltag->partien as $partie) { - $tore[$partie->gast->nr] += $partie->gTore > -1 ? $partie->gTore : 0; + $tore = $gastTore; + } + else { + // die Anzahl aller auswärts erzielten Tore + $tore = array($a->nr => 0, $b->nr => 0); + foreach ($this->spieltage as $spieltag) { + foreach ($spieltag->partien as $partie) { + if (isset($tore[$partie->gast->nr])) { + $tore[$partie->gast->nr] += max(0, (int) $partie->gTore); } } } - else { - $tore[$tableArray[$i]['team']->nr] = $gastTore[$tableArray[$i]['team']->nr]; - $tore[$tableArray[$i + 1]['team']->nr] = $gastTore[$tableArray[$i + 1]['team']->nr]; - } - } - // Auswertung des Direkten Vergleichs - if ($tore[$tableArray[$i]['team']->nr] < $tore[$tableArray[$i + 1]['team']->nr]) { - $newTable[$i] = $tableArray[$i + 1]; - $newTable[$i + 1] = $tableArray[$i]; - $newTable[$i]['pos'] = ++$i; - } - else { - $newTable[$i] = $tableArray[$i]; } } - else { // END if ($sort_pPkt[$i] == $sort... - $newTable[$i] = $tableArray[$i]; + if ($tore[$a->nr] < $tore[$b->nr]) { + $row = $tableArray[$i]; + $tableArray[$i] = $tableArray[$i + 1]; + $tableArray[$i + 1] = $row; + $i++; // getauschtes Paar ist entschieden } - $newTable[$i]['pos'] = $i + 1; } - return $newTable; + for ($i = 0; $i < $count; $i++) { + $tableArray[$i]['pos'] = $i + 1; + } + return $tableArray; } // END function sortTable function sortDirectTable($tableArray) { - echo "function sortDirectTable($tableArray) {"; - return false; + return liga::sortTable($tableArray); } } // END class FussballLiga diff --git a/lmo/addon/classlib/classes/ligaHandball.class.php b/lmo/addon/classlib/classes/ligaHandball.class.php index 03394c53..409dc8a7 100644 --- a/lmo/addon/classlib/classes/ligaHandball.class.php +++ b/lmo/addon/classlib/classes/ligaHandball.class.php @@ -21,46 +21,31 @@ class ligaHandball extends liga { * @return array */ function sortTable($tableArray) { + $sort_pPkt = $sort_spiele = array(); foreach($tableArray as $table) { $sort_pPkt[] = $table['pPkt']; $sort_spiele[] = $table['spiele']; } // Sortierung PlusPkt / Anzahl Spiele array_multisort($sort_pPkt, SORT_DESC, $sort_spiele, SORT_ASC, $tableArray, SORT_DESC); - // BEGIN Direkter Vergleich - $subteams = array(); - $pPkt = 0; - $spiele = 0; - for ($position = 0; $position < count($tableArray); $position++) { - if ($pPkt == $tableArray[$position]['pPkt'] && $spiele == $tableArray[$position]['spiele']) { - $subteams[$tableArray[$position]['team']->nr] = $tableArray[$position]['team']; - } - else { - if (count($subteams) > 1) { - $tmp_table = $this->calcTableforTeams($subteams); - $tmp_tablearray = $tableArray; - $nextpos = $position - count($tmp_table) ; - for ($b = 0; $b < count($tmp_table); $b++) { - for ($f = $nextpos; $f < $position; $f++) { - if ($tmp_tablearray[$f]['team'] === $tmp_table[$b]['team']) { - $tableArray[$nextpos + $b] = $tmp_tablearray[$f]; - } - } - } - } // END if (count($subteams) > 1) - $subteams = array(); - $pPkt = $tableArray[$position]['pPkt']; - $spiele = $tableArray[$position]['spiele']; - $subteams[$tableArray[$position]['team']->nr] = $tableArray[$position]['team']; - } - } // END for ($abc = 0; $abc < count($tableArray); $abc++) - // END Direkter Vergleich + // Direkter Vergleich bei gleichen Punkten und gleicher Anzahl Spiele + $tableArray = $this->sortTiedGroups($tableArray, array('pPkt', 'spiele')); for ($i = 0; $i < count($tableArray); $i++) { // Position setzen $tableArray[$i]['pos'] = $i + 1; } return $tableArray; } + /** + * Vergleich im direkten Vergleich: Punkte, Anzahl Spiele, Tordifferenz (wie sortDirectTable) + * + * @access protected + * @return integer + */ + function compareDirect($a, $b) { + return array($b['pPkt'], $a['spiele'], $b['dTor']) <=> array($a['pPkt'], $b['spiele'], $a['dTor']); + } + /** * Sortiert die errechnete Tabelle bei Direktem Vergleich und gibt diese als Array zurück * Sortierung PlusPkt / Anzahl Spiele / Differenz Tore @@ -70,6 +55,7 @@ function sortTable($tableArray) { * @return array */ function sortDirectTable($tableArray) { + $sort_pPkt = $sort_spiele = $sort_dTor = array(); foreach($tableArray as $table) { $sort_pPkt[] = $table['pPkt']; $sort_spiele[] = $table['spiele']; diff --git a/lmo/addon/classlib/classes/optionsSektion.class.php b/lmo/addon/classlib/classes/optionsSektion.class.php index ce5687fc..00c0db5a 100644 --- a/lmo/addon/classlib/classes/optionsSektion.class.php +++ b/lmo/addon/classlib/classes/optionsSektion.class.php @@ -77,8 +77,8 @@ class optionsSektion extends sektion { /** * Konstruktor * - * @param liga $aLiga - * @param array $optionDetails + * @param liga|string $aLiga + * @param array|string $optionDetails * @return optionsSektion */ function __construct($aLiga = '', $optionDetails='') { diff --git a/lmo/addon/classlib/classes/partie.class.php b/lmo/addon/classlib/classes/partie.class.php index a3bd1c9c..2bc77bc5 100644 --- a/lmo/addon/classlib/classes/partie.class.php +++ b/lmo/addon/classlib/classes/partie.class.php @@ -19,7 +19,7 @@ class partie { /** * Datum der Partie, - * @var date + * @var integer|string|null Unix-Zeitstempel, '' oder null ohne Termin * @access public */ @@ -33,14 +33,14 @@ class partie { /** * Heimmannschaft der Partie, - * @var array Team Objekt + * @var team Team Objekt * @access public */ var $heim; /** * Gastmannschaft der Partie, - * @var array Team Objekt + * @var team Team Objekt * @access public */ var $gast; @@ -103,13 +103,13 @@ class partie { /** * Konstruktor * - * @param integer $n_spNr - * @param string $n_time + * @param integer|string $n_spNr + * @param integer|string|null $n_time Unix-Zeitstempel, '' oder null ohne Termin * @param string $n_notiz * @param team $n_heim * @param team $n_gast - * @param integer $n_htore - * @param integer $n_gtore + * @param integer|string $n_htore + * @param integer|string $n_gtore * @param integer $n_hpunkte * @param integer $n_gpunkte * @return partie @@ -204,15 +204,15 @@ function valuateGame() * Gibt das SpielDatum als formatierten String zurück. 'd.m.Y' = Standard * * @access public - * @param string leer Ausgabe, falls kein Datum vorhanden - * @param string format Datumsformat in date()-Notation Standard = d.m.Y + * @param string $leer Ausgabe, falls kein Datum vorhanden + * @param string $format Datumsformat in date()-Notation Standard = d.m.Y * @return string */ function datumString($leer = '', $format = 'd.m.Y') { $dt = new DateTime(); $dt->setTimeStamp((int)$this->zeit); - $str = is_numeric($this->zeit) ? $dt->format($format) : $leer; + $str = is_numeric($this->zeit) && $this->zeit > 0 ? $dt->format($format) : $leer; return $str; } @@ -220,15 +220,15 @@ function datumString($leer = '', $format = 'd.m.Y') * Gibt die Anfangszeit als formatierten String zurück 'Stunden:Minuten' = Standard * * @access public - * @param string leer Ausgabe, falls keine Zeit vorhanden - * @param string format Zeitformat in date()-Notation Standard = H:i + * @param string $leer Ausgabe, falls keine Zeit vorhanden + * @param string $format Zeitformat in date()-Notation Standard = H:i * @return string */ function zeitString($leer = '', $format = 'H:i') { $dt = new DateTime(); $dt->setTimeStamp((int)$this->zeit); - $str = is_numeric($this->zeit) ? $dt->format($format) : $leer; + $str = is_numeric($this->zeit) && $this->zeit > 0 ? $dt->format($format) : $leer; return $str; } @@ -236,8 +236,8 @@ function zeitString($leer = '', $format = 'H:i') * Gibt für eine Partie aus, ob Verlängerung oder 11/7-Meterschießen * * @access public - * @param array text Referenz auf Sprachvariablen - * @param string leer Ausgabe, falls keine n.V./i.E vorhanden + * @param array $text Referenz auf Sprachvariablen + * @param string $leer Ausgabe, falls keine n.V./i.E vorhanden * @return string */ function spielEndeString(&$text, $leer = '') @@ -284,8 +284,8 @@ function showDetailsHTML() * * @since 2.7 * @access public - * @param mixed parameter Parameter die zu der Partie gespeichert werden sollen - * @param string key Bezeichner unter dem der Parameter gespeichert werden soll + * @param mixed $parameter Parameter die zu der Partie gespeichert werden sollen + * @param string $key Bezeichner unter dem der Parameter gespeichert werden soll */ function setParameter($parameter, $key = '') { @@ -302,7 +302,7 @@ function setParameter($parameter, $key = '') * * @since 2.7 * @access public - * @param string key Bezeichner des Parameters der ausgegeben werden soll, ist kein Parameter angegeben, wird das komplette array zurückgegeben. + * @param string $key Bezeichner des Parameters der ausgegeben werden soll, ist kein Parameter angegeben, wird das komplette array zurückgegeben. * @return mixed Parameter */ function getParameter($key = '') @@ -320,7 +320,7 @@ function getParameter($key = '') * * @since 2.7 * @access public - * @param string value Link zum Spielbericht + * @param string $value Link zum Spielbericht */ function setreportUrl($value) { @@ -333,9 +333,9 @@ function setreportUrl($value) * * @since 2.7 * @access public - * @param string ziel Zielfenster des Links - * @param string html Parameter - * @return string Link zum Spielbericht + * @param string|null $ziel Zielfenster des Links + * @param string $param html Parameter + * @return string|null Link zum Spielbericht */ function getreportUrl($ziel = '_blank', $param = '') { @@ -357,7 +357,7 @@ function getreportUrl($ziel = '_blank', $param = '') * * @access public * @since 2.7 - * @param integer Art des Spielendes + * @param integer|string $value Art des Spielendes */ function setSpielEnde($value) { diff --git a/lmo/addon/classlib/classes/pokalRunde.class.php b/lmo/addon/classlib/classes/pokalRunde.class.php index 84af5acf..69a73ffc 100644 --- a/lmo/addon/classlib/classes/pokalRunde.class.php +++ b/lmo/addon/classlib/classes/pokalRunde.class.php @@ -32,7 +32,7 @@ class pokalRunde { /** * Ende der Runde (Datum) als Unix Timestamp, - * @var string + * @var integer * @access privat */ var $end = 0; @@ -55,11 +55,11 @@ class pokalRunde { * Konstruktor der Klasse PokalRunde, setz die wichtigsten Parameter der Runde * * @access public - * @param integer roundNumber Nummer der Runde - * @param integer mode Spielmodus - * @param interger begin Begin der Runde als Unix Timestamp - * @param interger end Ende der Runde als Unix Timestamp - * @param array spieltage Spieltage (objecte) dieser Runde + * @param integer $roundNumber Nummer der Runde + * @param integer $mode Spielmodus + * @param integer $begin Begin der Runde als Unix Timestamp + * @param integer $end Ende der Runde als Unix Timestamp + * @param array $spieltage Spieltage (objecte) dieser Runde * @return void */ function __construct($roundNumber, $mode, $begin = 0, $end = 0, $spieltage = null) { @@ -85,7 +85,7 @@ function getRoundNumber() { * Setzt den Modus dieser Runde zurück * * @access public - * @param integer value Modus der Runde + * @param integer $value Modus der Runde * @return void */ function setMode($value) { @@ -106,7 +106,7 @@ function getMode() { * Setzt das Datum an dem diese Runde beginnt als Unix Timestamp * * @access public - * @param integer value Begin der Runde als Unix Timestamp + * @param integer $value Begin der Runde als Unix Timestamp * @return void */ function setBegin($value) { @@ -118,7 +118,7 @@ function setBegin($value) { * das Datum im angegeben Format zurückgegeben * * @access public - * @param string format der Datumsausgabe + * @param string $format der Datumsausgabe * @see date() * @return mixed Datum des Rundenbegins */ @@ -137,7 +137,7 @@ function getBegin($format = '') { * Setzt das Datum an dem diese Runde Endet als Unixtimestamp * * @access public - * @param integer value Ende der Runde als Unix Timestamp + * @param integer $value Ende der Runde als Unix Timestamp * @return void */ function setEnd($value) { @@ -149,7 +149,7 @@ function setEnd($value) { * das Datum im angegeben Format zurückgegeben * * @access public - * @param string format der Datumsausgabe + * @param string $format der Datumsausgabe * @see date() * @return mixed Ende dér Runde */ @@ -168,7 +168,7 @@ function getEnd($format = '') { * Fügt Referenzen auf Partien dieser Runde hinzu * * @access public - * @param array/object value Objecte der partien + * @param array|object $value Objecte der partien * @return void */ function addPartien(&$value) { @@ -187,8 +187,8 @@ function addPartien(&$value) { * Gibt Partien dieser Runde zurück, ist $Number ="" wird das komplette Array übergeben * * @access public - * @param integer Nummer der Partie - * @return object Partie + * @param integer|string $number Nummer der Partie, '' fuer alle + * @return partie|array|false Partie, alle Partien oder false, wenn es die Nummer nicht gibt */ function getPartien($number = '') { if ($number == '' ) @@ -207,7 +207,7 @@ function getPartien($number = '') { * Fügt Referenzen auf die Spieltage (und die Partien dieses Spieltags) dieser Runde hinzu * * @access public - * @param array/object value Objecte der Spieltage + * @param array|object $value Objecte der Spieltage * @return void */ function addSpieltage(&$value) { @@ -229,8 +229,8 @@ function addSpieltage(&$value) { * Gibt Spieltage dieser Runde zurück, ist $Number="" wird das komplette Array übergeben * * @access public - * @param integer Nummer des Spieltags - * @return object Spieltag + * @param integer|string $number Nummer des Spieltags, '' fuer alle + * @return spieltag|array|false Spieltag, alle Spieltage oder false, wenn es die Nummer nicht gibt */ function getSpieltage($number = '') { if ($number == '' ) @@ -246,7 +246,7 @@ function getSpieltage($number = '') { * Gibt die Anzahl der Spieltage dieser Pokal Runde zurück * * @access public - * @return interger Anzahl der Spieltage + * @return integer Anzahl der Spieltage */ function spieltageCount() { return count($this->spieltage); diff --git a/lmo/addon/classlib/classes/spieltag.class.php b/lmo/addon/classlib/classes/spieltag.class.php index 1c2382db..d0157b7e 100644 --- a/lmo/addon/classlib/classes/spieltag.class.php +++ b/lmo/addon/classlib/classes/spieltag.class.php @@ -21,14 +21,14 @@ class spieltag { /** * vonDatum als string, - * @var string + * @var integer|string|null Unix-Zeitstempel * @access public */ var $von; /** * bisDatum als string, - * @var string + * @var integer|string|null Unix-Zeitstempel * @access public */ var $bis; @@ -50,12 +50,19 @@ class spieltag { */ var $partien; + /** + * Weitere Werte des Spieltags aus der Ligadatei, z.B. HS (Handicap-Reihenfolge) + * @var array Schluessel => Wert + * @access private + */ + var $parameter = array(); + /** * Konstruktor * * @param integer $new_nr - * @param string $new_von - * @param string $new_bis + * @param integer|string $new_von Unix-Zeitstempel oder '' + * @param integer|string $new_bis Unix-Zeitstempel oder '' * @param array $partien * @return spieltag */ @@ -80,7 +87,8 @@ function __construct($new_nr, $new_von, $new_bis, $partien = array()) { * * @since 2.8 * @access public - * @param string Sortierung + * @param string $sorted Sortierung + * @param string $sort_dir Richtung: ASC oder DESC * @return array Partien */ function getPartien($sorted = '', $sort_dir = 'ASC') { @@ -129,12 +137,42 @@ function getModus() { return $this->modus; } + /** + * Weiteren Wert des Spieltags setzen, z.B. HS (Handicap-Reihenfolge) + * + * @access public + * @param mixed $value Wert, oder ein Array Schluessel => Wert + * @param string $key Schluessel wie in der Ligadatei + */ + function setParameter($value, $key = '') { + if (is_array($value)) { + $this->parameter = array_merge($this->parameter, $value); + } + else { + $this->parameter[$key] = $value; + } + } + + /** + * Weiteren Wert des Spieltags lesen; ohne Schluessel alle Werte + * + * @access public + * @param string $key Schluessel wie in der Ligadatei + * @return mixed Wert, null wenn nicht gesetzt + */ + function getParameter($key = '') { + if ($key == '') { + return $this->parameter; + } + return $this->parameter[$key] ?? null; + } + /** * Gibt Partie der angegebener Nummer zurück * * @access public - * @param integer Partienummer - * @return object Die Partie + * @param integer|null $number Partienummer + * @return partie|null Die Partie */ function &partieForNumber($number) { $result = null; @@ -149,8 +187,9 @@ function &partieForNumber($number) { * Gibt Partie der angegebener Teamnummern zurück * * @access public - * @param integer Heimmannschaftsnummer integer Gastmannschaftsnummer - * @return object Die Partie + * @param integer $heimNr Heimmannschaftsnummer + * @param integer $gastNr Gastmannschaftsnummer + * @return partie|null Die Partie */ function &partieForTeams($heimNr, $gastNr) { $count = $this->partienCount(); @@ -182,8 +221,9 @@ function &partieForTeams($heimNr, $gastNr) { * Gibt Partie der angegebener Teamnamen zurück * * @access public - * @param string Heimmannschaftsname string Gastmannschaftsname - * @return object Die Partie + * @param string $heimName Heimmannschaftsname + * @param string $gastName Gastmannschaftsname + * @return partie|null Die Partie */ function &partieForTeamNames($heimName, $gastName) { $result = null; @@ -200,7 +240,7 @@ function &partieForTeamNames($heimName, $gastName) { * Löscht Partie * * @access public - * @param objekt Partie + * @param object $rmvPartie Partie * @return bool Partie wurde gelöscht true / false */ function removePartie(&$rmvPartie) { @@ -220,12 +260,7 @@ function removePartie(&$rmvPartie) { $index++; } } - if (isset($partienArray)) { - $this->partien = &$partienArray; - } - else { - $this->partien = null; - } + $this->partien = $partienArray; return $result; } @@ -243,7 +278,7 @@ function partienCount() { * Fügt Partie zum Spieltage hinzu * * @access public - * @param Object die Partie + * @param object $neuePartie die Partie */ function addPartie(&$neuePartie) { $this->partien[] = $neuePartie; // &$ Das muss so sein diff --git a/lmo/addon/classlib/classes/stats.class.php b/lmo/addon/classlib/classes/stats.class.php index c98442c0..177aef0d 100644 --- a/lmo/addon/classlib/classes/stats.class.php +++ b/lmo/addon/classlib/classes/stats.class.php @@ -15,7 +15,7 @@ class stats { /** * Referenz auf das Ligaobject * - * @var Object Typ liga + * @var liga|null */ var $liga = null; @@ -51,7 +51,7 @@ function __construct(&$ligaObject) { * $statsArray['maxTor'][] object(partie) meiste Tore der Saison * * @access public - * @param integer spTag Spieltag für den die Statistik berechnet werden soll. + * @param integer $spTag Spieltag für den die Statistik berechnet werden soll. * @return array Statistik des Angegebenen Spieltages */ function calcLigaStats($spTag = 1) { @@ -71,6 +71,9 @@ function calcLigaStats($spTag = 1) { ); foreach ($this->liga->spieltage as $spieltag) { foreach ($spieltag->partien as $partie) { + if ($partie->valuateGame() == -1) { // noch nicht gespielt + continue; + } $statsArray['spiele']++; if ($partie->hTore> - 1) { $statsArray['hTore'] += $partie->hTore; @@ -168,9 +171,8 @@ function calcLigaStats($spTag = 1) { * $statsArray['tabelle'] Tabellenzeile des Teams * * @author Marcus Schug - * @param teamObject $teama Referenz auf ein TeamObject A - * @param teamObject $teamb Referenz auf ein TeamObject B - * @return array Statistik der beiden teams analog calcStatsForTeam + * @param team $team Referenz auf ein TeamObject + * @return array Statistik des Teams */ function calcStatsForTeam(&$team) { $statsArray = array( @@ -208,8 +210,8 @@ function calcStatsForTeam(&$team) { /** * Berechnung für den Statistikvergleich zweier Teams * - * @param teamObject $teama Referenz auf ein TeamObject A - * @param teamObject $teamb Referenz auf ein TeamObject B + * @param team $teama Referenz auf ein TeamObject A + * @param team $teamb Referenz auf ein TeamObject B * @return array Statistik der beiden teams analog calcStatsForTeam */ function calcStatsForTeams(&$teama, &$teamb) { @@ -222,14 +224,14 @@ function calcStatsForTeams(&$teama, &$teamb) { $dx =($statsArrayB['tabelle']['pTor'] / $statsArrayB['tabelle']['spiele']) + ($statsArrayA['tabelle']['mTor'] / $statsArrayA['tabelle']['spiele']); $ex = $ax + $bx; $fx = $cx + $dx; - } - if (isset($ex) && ($ex > 0) && isset($fx) &&($fx > 0)) { - $ax = round(10000 * $ax / $ex); - $bx = round(10000 * $bx / $ex); - $cx = round(10000 * $cx / $fx); - $dx = round(10000 * $dx / $fx); - $statsArrayA['chance'] = number_format((($ax + $cx) / 200), 2, ',', '.'); - $statsArrayB['chance'] = number_format((($bx + $dx) / 200), 2, ',', '.'); + if ($ex > 0 && $fx > 0) { + $ax = round(10000 * $ax / $ex); + $bx = round(10000 * $bx / $ex); + $cx = round(10000 * $cx / $fx); + $dx = round(10000 * $dx / $fx); + $statsArrayA['chance'] = number_format((($ax + $cx) / 200), 2, ',', '.'); + $statsArrayB['chance'] = number_format((($bx + $dx) / 200), 2, ',', '.'); + } } $resultArray[] = $statsArrayA; $resultArray[] = $statsArrayB; @@ -240,7 +242,7 @@ function calcStatsForTeams(&$teama, &$teamb) { * Berechnung der Serien für ein Team * * @access public - * @param teamObject $team Referenz auf ein TeamObject + * @param team $team Referenz auf ein TeamObject * @return array SerienArray, array(s,n,u,su,nu) */ function getSerien(&$team){ @@ -344,7 +346,7 @@ function getSerien(&$team){ * $maxArray['Tor'][n]['partie'] mit der Referenz auf eine Partie * * @access public - * @param teamObject $team Referenz auf ein TeamObject + * @param team $team Referenz auf ein TeamObject * @return array MaxArray, array(sH,sA,nH,nA,Tor) */ function getMaxResults(&$team){ @@ -435,10 +437,11 @@ function getMaxResults(&$team){ /** * Berechnung des Saisonverlaufs für ein Team * - * @param teamObject $teama Referenz auf ein TeamObject A + * @param team $team Referenz auf ein TeamObject * @return array Saisonverlauf des Teams */ function getVerlaufTeam(&$team) { + $positionen = array(); $tableArray = $this->liga->calcTable($this->liga->options->keyValues['Rounds'], 'all', true); foreach($tableArray as $zeile){ if ($zeile['team'] == $team){ @@ -452,11 +455,12 @@ function getVerlaufTeam(&$team) { /** * Berechnung des Saisonverlaufs für zwei Teams * - * @param teamObject $teama Referenz auf ein TeamObject A - * @param teamObject $teamb Referenz auf ein TeamObject B + * @param team $teama Referenz auf ein TeamObject A + * @param team $teamb Referenz auf ein TeamObject B * @return array Saisonverlauf der Teams */ function getVerlaufTeams(&$teama, &$teamb) { + $positionen = array(); $tableArray = $this->liga->calcTable($this->liga->options->keyValues['Rounds'], 'all', true); foreach($tableArray as $zeile){ if ($zeile['team'] == $teama){ @@ -473,7 +477,7 @@ function getVerlaufTeams(&$teama, &$teamb) { * Ausgabe der Serie als HTML * * @author Marcus Schug
'; - $output = $serienArray['su'] . ' Spiele o. Niederlage'; + $output .= $serienArray['su'] . ' Spiele o. Niederlage'; } return $output; } // END SerieToHTML() diff --git a/lmo/addon/classlib/classes/team.class.php b/lmo/addon/classlib/classes/team.class.php index 74a10831..c03639af 100644 --- a/lmo/addon/classlib/classes/team.class.php +++ b/lmo/addon/classlib/classes/team.class.php @@ -50,7 +50,7 @@ class team * * @param string $name * @param string $kurz - * @param string $nr + * @param integer|string $nr * @param string $mittel * @return team */ diff --git a/lmo/addon/classlib/functions.php b/lmo/addon/classlib/functions.php index 488cf915..31852501 100644 --- a/lmo/addon/classlib/functions.php +++ b/lmo/addon/classlib/functions.php @@ -13,7 +13,7 @@ * const_array() * * @access public - * @param constant string + * @param string $constant durch Komma getrennte Werte * @return array */ function const_array($constant) { @@ -26,7 +26,7 @@ function const_array($constant) { * example: strBeforChar("fussballbl2004.l98",".") returns "fussballbl2004" * * @param string $str - * @param char $char + * @param string $char * @return string */ function strBeforChar($str,$char) { @@ -38,11 +38,12 @@ function strBeforChar($str,$char) { * example: strBeforChar("fussballbl2004.l98",".") returns "l98" * * @param string $str - * @param char $char + * @param string $char * @return string */ function strAfterChar($str,$char) { - return substr($str, strrpos($str, $char) + 1); + $pos = strrpos($str, $char); + return $pos === false ? '' : substr($str, $pos + 1); } /** @@ -75,7 +76,7 @@ function readLigaDir($dirName,&$dataArray) { while($data = $dir->read()){ $ext = strtolower( strAfterChar($data, '.')); if ($ext == 'l98') { - $name = trim(substr($data, 0, strrpos($data, $ext) -1)); + $name = trim(strBeforChar($data, '.')); $dataArray[] = array( 'path' => $dir->path, 'src' => $data, @@ -91,7 +92,7 @@ function readLigaDir($dirName,&$dataArray) { /** * unscharfe Teamsuche * - * @param array $teamNamesArray + * @param array|null $teamNamesArray * @param string $search * @return array */ @@ -103,8 +104,9 @@ function findTeamName(&$teamNamesArray, $search) { foreach($teamNamesArray as $teamName) { $match_with = strtolower(preg_replace($expr, '', $teamName)); if ($match_with == $match) { + // alle Treffer liefern: der Aufrufer erkennt daran eine mehrdeutige Suche + // (z.B. "TSV Muster" und "TSV Muster 2" unterscheiden sich nur in Ziffern) $results[] = $teamName; - break; } } } diff --git a/lmo/addon/classlib/html_output.php b/lmo/addon/classlib/html_output.php index 224dae7f..304dbcdd 100644 --- a/lmo/addon/classlib/html_output.php +++ b/lmo/addon/classlib/html_output.php @@ -34,12 +34,17 @@ */ function HTML_icon($img_name, $img_type, $img_size = 'small', $html = '', $alternative_text = '') { $subFolder = '/' . $img_type . '/' . $img_size . '/'; + $imgHTML = ''; + // Alternativtext erst nach allen Endungen, sonst endet die Suche schon bei der ersten foreach( const_array( CLASSLIB_IMG_TYPES ) as $extension) { - if ($imgHTML = findImage($img_name, $subFolder, $extension, $html, $alternative_text) ) { + if ($imgHTML = findImage($img_name, $subFolder, $extension, $html) ) { break; } } - if ($imgHTML == '') { + if ($imgHTML == '' && $alternative_text != '') { + $imgHTML = $alternative_text; + } + elseif ($imgHTML == '') { $imgHTML = substr($html, 6, strrpos($html, "'") - 6); } return $imgHTML; @@ -52,7 +57,7 @@ function HTML_icon($img_name, $img_type, $img_size = 'small', $html = '', $alter * @deprecated * @param string $ligaFile * @param string $team - * @param string $html + * @param string $htmlParameter * @param string $alternative_text * @return string */ @@ -67,7 +72,7 @@ function HTML_smallTeamIcon($ligaFile, $team, $htmlParameter = '', $alternative_ * @deprecated * @param string $ligaFile * @param string $team - * @param string $html + * @param string $htmlParameter * @param string $alternative_text * @return string */ @@ -81,7 +86,7 @@ function HTML_bigTeamIcon($ligaFile, $team, $htmlParameter = '', $alternative_te * * @deprecated * @param string $ligaFile - * @param string $html + * @param string $htmlParameter * @param string $alternative_text * @return string */ @@ -96,7 +101,7 @@ function HTML_smallLigaIcon($ligaFile, $htmlParameter = '', $alternative_text = * * @deprecated * @param string $ligaFile - * @param string $html + * @param string $htmlParameter * @param string $alternative_text * @return string */ @@ -111,9 +116,9 @@ function HTML_bigLigaIcon($ligaFile, $htmlParameter = '', $alternative_text = '' * Deprecated! Use HTML_icon() instead * * @deprecated - * @param string $ligaFile - * @param string $liganame + * @param string $spieler * @param string $htmlParameter + * @param string $alternative_text * @return string */ @@ -126,9 +131,9 @@ function HTML_smallSpielerIcon($spieler, $htmlParameter = '', $alternative_text * Deprecated! Use HTML_icon() instead * * @deprecated - * @param string $ligaFile - * @param string $liganame + * @param string $spieler * @param string $htmlParameter + * @param string $alternative_text * @return string */ function HTML_bigSpielerIcon($spieler, $htmlParameter = '', $alternative_text = '') { diff --git a/lmo/addon/classlib/ini.php b/lmo/addon/classlib/ini.php index fc8989d4..01f300de 100644 --- a/lmo/addon/classlib/ini.php +++ b/lmo/addon/classlib/ini.php @@ -33,15 +33,12 @@ // Weitere Klassen einbinden -// class pdf -//require_once(PATH_TO_ADDONDIR . '/classlib/classes/pdf/Cezpdf.php'); // Version 0.12.40 -//include_once(PATH_TO_ADDONDIR . '/classlib/classes/pdf/extensions/CezTableImage.php'); // classes for image manipulation if (file_exists(PATH_TO_ADDONDIR . '/classlib/classes/phpthumb/phpthumb.class.php') ){ require_once(PATH_TO_ADDONDIR . '/classlib/classes/phpthumb/phpthumb.class.php'); } if (!defined('LMO_VERSION_NR')) { - define('LMO_VERSION_NR', $json_a['version']); + define('LMO_VERSION_NR', lmo_installed_version()); } if (!defined('LMO_VERSION')) { define('LMO_VERSION', LMO_VERSION_NR); diff --git a/lmo/addon/classlib/update_addons.php b/lmo/addon/classlib/update_addons.php index 2161037a..cd77febc 100644 --- a/lmo/addon/classlib/update_addons.php +++ b/lmo/addon/classlib/update_addons.php @@ -21,7 +21,10 @@ function updateAddons($fileName) { //anzeigen zu lassen, um evtl. nicht gesetzte Variablen frühzeitig zu erkennen //error_reporting(E_ALL); $liga = basename($fileName, '.l98'); - $file = $liga . '.l98'; + // $file relativ zum Ligenverzeichnis wie im restlichen LMO; mit basename() wurden Ligen in + // Unterordnern nicht gefunden (HTML-Export und Statistik liefen dann ohne Daten) + $ligenDir = PATH_TO_LMO . '/' . $dirliga; + $file = strpos($fileName, $ligenDir) === 0 ? substr($fileName, strlen($ligenDir)) : $liga . '.l98'; $action = 'admin'; $array = array(); //Liga-Datei öffnen diff --git a/lmo/addon/mini/lmo-mininext.php b/lmo/addon/mini/lmo-mininext.php index cc28aed3..09207071 100644 --- a/lmo/addon/mini/lmo-mininext.php +++ b/lmo/addon/mini/lmo-mininext.php @@ -115,7 +115,7 @@ loadTemplatefile($mini_template); $team_a = NULL; $team_b = NULL; @@ -231,7 +231,7 @@ // determine games of the current league $spiele = $liga->allPartieForTeams($team_a, $team_b, true); - foreach($spiele as $spiel) { + foreach($spiele ?? array() as $spiel) { // null: die beiden treffen in dieser Liga nie aufeinander if ($spiel->hTore != -1 && $spiel->gTore != -1) { $archivSortDummy[] = $spiel->zeit; if ($spiel->heim == $team_a) { @@ -265,7 +265,7 @@ } if (!is_null($newTeam_a) && !is_null($newTeam_b)) { $spiele = $newLiga->allPartieForTeams($newTeam_a, $newTeam_b, true); - foreach($spiele as $spiel) { + foreach($spiele ?? array() as $spiel) { // null: die beiden treffen in dieser Liga nie aufeinander if ($spiel->hTore != -1 && $spiel->gTore != -1) { $archivSortDummy[] = $spiel->zeit; if ($spiel->heim == $newTeam_a) { diff --git a/lmo/addon/mini/lmo-minitab.php b/lmo/addon/mini/lmo-minitab.php index e5ee6c0a..341d4cf5 100644 --- a/lmo/addon/mini/lmo-minitab.php +++ b/lmo/addon/mini/lmo-minitab.php @@ -62,6 +62,12 @@ ? $cfgarray['mini']['standardTabellenPlatz'] : NULL)); +// Als Zahlen weiterrechnen: in der mitgelieferten Konfiguration sind die Werte leer (''), und +// PHP 8 bricht bei "Zahl - ''" mit einem TypeError ab. Standard wie oben: 2 Plaetze darueber/darunter. +$m_ueber = is_numeric($m_ueber) ? (int) $m_ueber : 2; +$m_unter = is_numeric($m_unter) ? (int) $m_unter : 2; +$m_platz = is_numeric($m_platz) ? (int) $m_platz : NULL; + //If IFRAME - complete HTML document if (basename($_SERVER['PHP_SELF']) == 'lmo-minitab.php') {?> @@ -94,12 +100,16 @@ //'Direkt Output:
'; $liga = new liga(); if (is_readable(PATH_TO_LMO . '/' . $dirliga . $m_liga) && $m_liga && $liga->loadFile(PATH_TO_LMO . '/' . $dirliga . $m_liga)) { - $template = new HTML_Template_IT( PATH_TO_TEMPLATEDIR . '/mini' ); + $template = new LMO_Template( PATH_TO_TEMPLATEDIR . '/mini' ); $template->loadTemplatefile($m_template . '.tpl.php'); $AnzahlTeams = $liga->teamCount(); $LigaTabelle = $liga->calcTable($liga->options->keyValues['Rounds']); - $LastGameDay = $liga->calcTable($liga->options->keyValues['Actual'] - 1); + // Tendenz = Platz am Vorspieltag minus Platz jetzt. Vor dem ersten Spieltag gibt es keinen + // Vorspieltag (calcTable(0) rechnet mit dem aktuellen Spieltag): Tendenz 0 fuer alle. + $LastGameDay = $liga->options->keyValues['Actual'] > 1 + ? $liga->calcTable($liga->options->keyValues['Actual'] - 1) + : $LigaTabelle; $Favorit = $liga->teamForNumber($liga->options->keyValues['favTeam']); $viewPosition = 1; if (is_null($m_platz) || $m_platz <= 0 || $m_platz > $AnzahlTeams ) { @@ -249,7 +259,7 @@ } $template->show(); } else { - echo getMessage($text['mini'][5] . ' ' . $mini_liga, true); + echo getMessage($text['mini'][5] . ' ' . $m_liga, true); } } //Falls IFRAME - komplettes HTML-Dokument diff --git a/lmo/addon/spieler/lmo-statadmin.php b/lmo/addon/spieler/lmo-statadmin.php index a2767771..37f59111 100644 --- a/lmo/addon/spieler/lmo-statadmin.php +++ b/lmo/addon/spieler/lmo-statadmin.php @@ -20,6 +20,32 @@ require_once(__DIR__ . '/../../init.php'); +/** + * Diese Datei wird direkt aufgerufen; init.php stellt bereit: + * + * @var \Lmo\Liga\Rohdaten\Liga $lmo_liga Rohdaten der geladenen Liga (lmo-openfile.php) + * @var array
| - "> + "> | -+ |
" title="">![' . $spalten[$i] . '](' . URL_TO_IMGDIR . '/spieler/' . rawurlencode($spalten[$i]) . '.png)
';
@@ -150,7 +182,7 @@
} //Ende der Suche nach Bildern für die Spaltenüberschriften
if ($spieler_extra_sortierspalte == 0) {
?>" title="">
" title="">![' . $spalten[$i] . '](' . URL_TO_IMGDIR . '/url.png "' . $text[46] . '"){kind=small}
';
+ if (!empty($pos) && $lmo_urls[$pos] != '' && $lmo_liga->einstellungen()->mitMannschaftslinks()) {
+ echo '';
}
//Spielerlinks
@@ -252,10 +284,10 @@
- ">
+ ">
| - "> + "> | - "> + "> | diff --git a/lmo/addon/ticker/ticker.php b/lmo/addon/ticker/ticker.php index 7f28cae4..2ff4aca2 100644 --- a/lmo/addon/ticker/ticker.php +++ b/lmo/addon/ticker/ticker.php @@ -20,19 +20,35 @@ require_once(__DIR__ . '/../../init.php'); +/** + * Diese Datei wird direkt aufgerufen; init.php stellt bereit: + * + * @var array
| - - | - |
-
-
-
-
-
-
-
-
