kickern/com_sportsmanager
1
0
Fork 0
mirror of https://github.com/Deutscher-Tischfussballbund/com_sportsmanager.git synced 2026-09-11 18:41:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Tabellenspalte richtig benennen: bei „Mehrere" sind es Saetze, keine Tore

Browse Source 
Aus dem Review: „Die Summe der Tore ist auch falsch." Sie ist es nicht — sie ist falsch
BESCHRIFTET. Unter punktetyp „Mehrere" ist begegnung.heim_punkte seit immer die Summe der
Satzstaende, und die Tabellenspalte heisst „Tore". Bisher fiel das nicht auf, weil die Tore
gar nicht bekannt waren; mit der Erfassung je Satz sind sie es, und die Spalte behauptet
etwas, was sie nicht zeigt.

Die Absicht war schon da: tabelleAnzeigen() fragt `punktetyp != 2 ? Tore : Saetze` — nur
kennt punktetyp ausschliesslich 0 und 1, die Bedingung war also nie wahr. Jetzt
`punktetyp == 0 ? Tore : Saetze`, an beiden Spaltenkoepfen (Absolut und Differenz/Verhaeltnis).

Gemessen an zwei Wettbewerben derselben Installation: bei punktetyp 1 stehen jetzt „Saetze
Absolut" und „Saetze", bei punktetyp 0 unveraendert dreimal „Tore". Keine Zahl aendert
sich, kein Sortierschluessel, keine Bestandsdaten.

Die Alternative — die echten Tore summieren, wo Satzergebnisse geführt werden — waere ein
Eingriff in Bestandsstaende und in die Tabellensortierung und gehoert in einen eigenen PR
mit Migration (Sven 2026-09-07).
This commit is contained in:
svennickel
2026-09-07 13:52:35 +02:00
parent 554ad25156
commit 099174c0ef
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/structure/components/com_sportsmanager/views/sportsmanager/view.html.php
+5 -2
View File
@@ -1788,11 +1788,14 @@ static function tabelleAnzeigen($veranstaltung, $modus, $teams, $spieltag, $spie
<strong><?php echo Text::_('COM_SPORTSMANAGER_PENALTY'); ?></strong></th>
<?php
}
// Die Spalte zaehlt bei „Einer" Tore und bei „Mehrere" Saetze — sie hiess
// bisher in beiden Faellen „Tore", weil die Bedingung auf punktetyp == 2
// prueft und punktetyp nur 0 und 1 kennt.
if ($veranstaltung->tabellenwertung != 3 && $veranstaltung->tabellenwertung != 6 && $veranstaltung->tabellenwertung != 9 && $veranstaltung->tabellenwertung != 23 && $veranstaltung->tabellenwertung != 26 && $veranstaltung->tabellenwertung != 29) {
if ($alleine_angezeigt && $praesentation == 0) {
?>
<th nowrap>
<strong><?php echo $modus->punktetyp != 2 ? Text::_('COM_SPORTSMANAGER_GOALS') : Text::_('COM_SPORTSMANAGER_SETS'); ?></strong><br/>
<strong><?php echo $modus->punktetyp == 0 ? Text::_('COM_SPORTSMANAGER_GOALS') : Text::_('COM_SPORTSMANAGER_SETS'); ?></strong><br/>
<small><?php echo Text::_('COM_SPORTSMANAGER_STAT_TOTAL'); ?></small>
</th>
<?php
@@ -1800,7 +1803,7 @@ static function tabelleAnzeigen($veranstaltung, $modus, $teams, $spieltag, $spie
?>
<th nowrap
title="<?php echo $veranstaltung->tabellenwertung == 1 || $veranstaltung->tabellenwertung == 4 || $veranstaltung->tabellenwertung == 7 || $veranstaltung->tabellenwertung == 10 || $veranstaltung->tabellenwertung == 21 || $veranstaltung->tabellenwertung == 24 || $veranstaltung->tabellenwertung == 27 ? Text::_('COM_SPORTSMANAGER_DIFFERENCE') : Text::_('COM_SPORTSMANAGER_RATIO'); ?>">
<strong><?php echo $modus->punktetyp != 2 ? Text::_('COM_SPORTSMANAGER_GOALS') : Text::_('COM_SPORTSMANAGER_SETS'); ?></strong>
<strong><?php echo $modus->punktetyp == 0 ? Text::_('COM_SPORTSMANAGER_GOALS') : Text::_('COM_SPORTSMANAGER_SETS'); ?></strong>
</th>
<?php
}