Repository navigation
Conversation
- Removed legacy ITX and IT_Error classes, replacing them with a new LMO_Template class that extends HTML_Template_IT. - Updated init.php to utilize Composer's autoloader for dependency management. - Modified lmo-showmain2.php to instantiate LMO_Template instead of HTML_Template_IT. - Adjusted PHPUnit configuration to point to the correct vendor directory. - Enhanced GoldenMaster tests to handle Composer dependencies more efficiently by creating symlinks for the vendor directory. - Updated Normalizer to ignore vendor paths in include_path comparisons during tests.
- Implemented a new setup process for creating the first admin account if it doesn't exist. - Added language translations for initial setup messages in Bosnian, Czech, German, English, Spanish, French, Croatian, Italian, Hungarian, Dutch, Norwegian, Portuguese, Romanian, and Slovene. - Updated admin authentication and main admin pages to handle the new setup process. - Removed the old installer and integrated the setup logic directly into the application flow.
… views and snapshots, enhancing user experience by eliminating unnecessary warnings. Additionally, add language options to the admin footer for improved accessibility and localization.
SP aus der Ligadatei kommt als String, setSpielEnde() nimmt aber nur int an. Das Spielende blieb dadurch immer 0: Sonderpunkte fuer Verlaengerung und Elfmeterschiessen wurden nicht beruecksichtigt, und ein anschliessendes writeFile() schrieb SP=0 zurueck (Datenverlust). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
sortTable() (liga) und ligaHandball::sortTable() werteten eine Gruppe punktgleicher Teams erst aus, wenn danach ein Team mit anderer Punktzahl folgte. Die letzte Gruppe der Tabelle wurde nie nach dem direkten Vergleich sortiert. Beide nutzen jetzt die gemeinsame Methode sortTiedGroups(). Sie berechnet nur Gruppen neu, die kleiner als die ganze Tabelle sind; sonst wuerde calcTableforTeams() -> sortTable() bei einem Unentschieden im direkten Duell endlos rekursiv weiterrechnen. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
calcTableforTeams() buchte bei gTore == -2 die Punkte fuer die Niederlage (PointsForLost) beim Gast statt beim Heimteam: Das Heimteam bekam sie als Minuspunkte, der Gast als Pluspunkte (Kopierfehler aus dem Zweig darueber). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
factory() war nicht static; der dokumentierte Aufruf liga::factory($datei) endet unter PHP 8 mit einem Error. Ausserdem wurde aus LigaType=Handball der Klassenname "HandballLiga" gebildet, die Klasse heisst aber ligaHandball; dadurch kam immer ein liga-Objekt zurueck. Ohne LigaType gab es zusaetzlich eine Warnung (undefinierter Index). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Actual zaehlt ab 1, das Array $this->spieltage ab 0. Zurueckgegeben wurde bisher der Spieltag nach dem aktuellen (bzw. null beim letzten Spieltag). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Die Schleife las beim letzten Tabellenplatz $sort_pPkt[$i + 1] (undefinierter Index), die Torsummen fuer den direkten Vergleich wurden ohne Initialisierung hochgezaehlt und allPartieForTeams() kann null liefern. Unter PHP 8 brach jede Tabellenberechnung einer Fussball-Liga mit Warnungen ab. Ausserdem summierte der direkte Vergleich die Auswaertstore nicht (= statt +=), und nach einem Tausch wurde die Position des Paares falsch gesetzt. Die Regeln bleiben: Punkte, Spiele, Tordifferenz, Tore, dann direkter Vergleich (Tore, Auswaertstore im direkten Vergleich, alle Auswaertstore). sortDirectTable() gab statt einer Tabelle Debug-Text aus und liefert jetzt die normale Sortierung. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
'spiele' zaehlte jede angesetzte Partie, auch noch nicht gespielte. Diese nahmen ausserdem an der Suche nach hoechstem Heim-/Auswaertssieg und den meisten Toren teil (mit -1:-1 als Ergebnis). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
"Spiele ohne Niederlage" wurde mit = statt .= angehaengt und ersetzte so die vorherige Ausgabe (z. B. "2 Sieg(e)"); uebrig blieb nur die letzte Zeile. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fehlte das Zeichen, wurde strrpos() false und substr($str, 1) schnitt nur
das erste Zeichen ab ("ohnepunkt" -> "hnepunkt"). Jetzt wie strBeforChar():
leerer String.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Die Endung wird ohne Gross-/Kleinschreibung erkannt, der Name aber mit strrpos($data, 'l98') abgeschnitten. Bei "POKAL.L98" fand das nichts und der Name wurde "POKAL.L9". Jetzt wird alles vor dem letzten Punkt genommen. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
findImage() gibt ohne Treffer den Alternativtext zurueck. Ist er gesetzt, brach die Schleife ueber CLASSLIB_IMG_TYPES schon nach der ersten Endung (.gif) ab; Logos als .jpg oder .png wurden nie gefunden. Der Alternativtext wird jetzt erst verwendet, wenn keine Endung passt. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Neue Testsuite "unit" (tests/Unit), laeuft vor den Golden-Master-Tests. ClasslibTestCase laedt die classlib ohne init.php und baut Ligen im Speicher oder als kleine .l98-Datei auf. Abgedeckt: Hilfsfunktionen, team, sektion, optionsSektion, partie, spieltag, liga (Nachschlagen, gamesSorted, loadFile inkl. Pokal mit Hin- und Rueckspiel, calcTable mit Heim/Gast/Hin/Rueck, Strafen, Kegel, n.V./i.E., gruener Tisch, direkter Vergleich), ligaFussball, ligaHandball, factory(), stats und html_output.php. Die Tests sichern die vorangehenden Bugfixes ab. Nicht abgedeckt: writeFile() (bindet init.php ein) und loadFromDb() (braucht eine Datenbank). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
lmo-calctable.php verglich fuer die Gruppenbildung substr($tab0, 7, 9), also nur die letzte Ziffer der Punkte plus die Minuspunkte. Teams mit z.B. 59 und 49 Punkten galten als gleichauf, wenn sie in der Tabelle direkt untereinander standen; ihre Duelle flossen dann in den direkten Vergleich der tatsaechlich punktgleichen Teams ein. Nachgewiesen an der Primera Division 2013/14: Villarreal und Real Sociedad (je 59) gerieten in die Gruppe von Valencia und Celta (je 49), Celta stand dadurch faelschlich vor Valencia. Jetzt wird der komplette Punkte- und Minuspunkteblock verglichen.
Die classlib wich bei Punktgleichheit vom Kern ab: - Gleichstand wurde ueber Plus- und Minuspunkte erkannt, auch ohne Zwei-Punkte-Regel. Jetzt zaehlen die Minuspunkte nur bei MinusPoints=2. - Brachte der direkte Vergleich keine Entscheidung (Teams haben noch nicht gegeneinander gespielt oder das Duell war ausgeglichen), ging die Reihenfolge nach Tordifferenz verloren; entschieden hat dann der Objektvergleich der Teams. Jetzt bleibt die bisherige Reihenfolge. - Der direkte Vergleich wurde innerhalb des direkten Vergleichs erneut angewendet. Wie im Kern wird jetzt nur einmal ausgewertet. Die Sortierung der Gleichstandsgruppe steht in compareDirect() (Punkte, Minuspunkte, Tordifferenz, Tore; Kegel: Tore vor Tordifferenz), Handball hat eine eigene Variante (Punkte, Spiele, Tordifferenz). Gefunden mit echten Ligen (Serie A, Primera Division mit Wikipedia- Abschlusstabellen): die classlib lag dort in 6 Saisons falsch.
Der Rang aus dem direkten Vergleich ersetzt in $tab0 die erste Ziffer der
Tordifferenz. Ab Rang 10 wurde er zweistellig geschrieben und ueberschrieb
dabei die letzte Ziffer der Minuspunkte. Mit Minuspunkten (Zwei-Punkte-
Regel) stand das beste Team der Gruppe dann ganz unten, z.B. in einer
Handball-Liga nach einem Spieltag mit lauter Unentschieden.
Der Rang belegt jetzt immer genau eine Stelle; ab 10 folgen die Zeichen
hinter "9" (":", ";", ...), die als String richtig dahinter sortieren.
Fuer Gruppen mit weniger als zehn Teams aendert sich $tab0 nicht.
Wertung "beidseitiges Erg." (ET=3): laut Hilfetext zaehlt das Ergebnis fuer beide Mannschaften gleichermassen aus Sicht der Heimmannschaft. In der Praxis ist das die Spielwertung gegen beide Mannschaften (z.B. beide nicht angetreten), eingetragen als 0:2. Bisher bekamen beide Teams nur die Tore und das Spiel, aber weder Sieg, Remis oder Niederlage noch Punkte und Minuspunkte. In einer Liga mit Minuspunkten standen beide dadurch besser da als richtig. Jetzt wird das Ergebnis aus Heimsicht fuer beide gewertet, in der Tabelle und im direkten Vergleich (lmo-calctable1.php). Golden Master: geaendert sind die Ligen mit dieser Wertung (wertung, direct_wertung, direct_wertung_many) und die Ergebniseingabe im Admin-Test, die ein 1:1 beidseitig eintraegt. Spiele und Tore bleiben gleich, es aendern sich Bilanz, Punkte, Minuspunkte, Serien und die Reihenfolge.
Die Standardtabelle rechnet bis $endtab = $anzst, also bis zum letzten Spieltag der Saison, auch wenn erst wenige Spieltage gespielt sind. Eine Strafe "ab Spieltag 20" wurde dadurch schon nach Spieltag 5 abgezogen. Massgeblich ist jetzt der letzte Spieltag mit einem Ergebnis ($calc_lastst): die Strafe zaehlt, wenn er (bzw. der angezeigte Spieltag, falls frueher) mindestens X ist. Strafen ab Spieltag 0 zaehlen wie bisher sofort. Heim- und Auswaertstabelle bleiben ohne Strafen.
Straf-/Bonustore gibt man in lmo-adminteams.php als "x : y" ein, gespeichert werden sie negiert (TOR1 = -x, TOR2 = -y); "+2" bei den Gegentoren heisst also zwei Gegentore mehr. strafen() zog TOR2 mit abs() ab und machte aus zusaetzlichen Gegentoren weniger Gegentore. Jetzt wie lmo-calctable.php: mTor -= TOR2.
Die classlib buchte Strafen innerhalb der Partien eines Spieltags: - War das Team an Spieltag STDA spielfrei, entfiel die Strafe ganz. - Strafen zaehlten auch in Heim- und Auswaertstabelle. - Eine Strafe ab einem spaeteren Spieltag zaehlte schon, sobald die Tabelle bis dahin gerechnet wurde, auch ohne gespielte Partien. Jetzt entscheidet strafeFaellig(), ob eine Strafe in dieser Tabelle zaehlt (nicht in Heim-/Auswaertstabelle, Hin- und Rueckrunde wie lmo-calctable.php, erst wenn Spieltag STDA Ergebnisse hat). calcTable() bucht faellige Strafen nach jedem Spieltag genau einmal je Team, unabhaengig davon, ob es spielt; so stimmt auch die Platzierung je Spieltag (possp).
- sortTable() und sortDirectTable() brachen bei einer Tabelle ohne Zeilen mit einem TypeError ab (array_multisort ohne Sortierspalten) - getVerlaufTeam() und getVerlaufTeams() liefern fuer ein Team ausserhalb der Liga ein leeres Array statt einer Warnung - calcStatsForTeams(), removePartie() und drei isset() auf Schleifen- schluessel vereinfacht, ohne Aenderung am Verhalten
Nur Kommentare, kein Code: $ vor den Parameternamen, Tippfehler in Typen (interger, objekt, teamObject, char), und die tatsaechlichen Typen, wo der Kommentar etwas anderes versprach (z. B. team|null statt object bei den Suchfunktionen, Zeitstempel statt string bei Spieltag und Partie).
Die Hilfsskripte unter tests/GoldenMaster/bin bleiben auf Stufe 0: sie binden den alten Code ein und arbeiten mit dessen globalen Variablen. In den Tests vier Stellen praezisiert (Typangaben, Rueckruffunktion).
Vier Kommentare an das Verhalten angepasst: liga::factory() liefert null, wenn es die Datei nicht gibt; pokalRunde::getPartien() und getSpieltage() liefern je nach Aufruf ein Objekt, alle Eintraege oder false.
D1/D2 mit Text oder fehlenden Teilen (ab.cd.efgh, 23.08.) fuehrten in loadFile() zu einem TypeError in checkdate(). Das Datum gilt jetzt als fehlend (liga::dateToTime()). Dazu ohne Aenderung am Verhalten: Teamnummern aus der Datei als Zahl an teamForNumber(), strcmp() mit Strings, eine immer wahre Pruefung entfernt und Typangaben in Kommentaren an die tatsaechlichen Werte angepasst.
Phpstan classlib
lmo-openfile.php liest die Datei weiter in einzelne Variablen und bindet sie danach an ein Objekt (Lmo\Liga\LigaDaten): Eigenschaft und Variable sind dieselbe Stelle im Speicher. Alter Code laeuft unveraendert, neuer Code liest $ligaDaten->titel statt $titel. Erster Schritt, um Dateien einzeln von den losen Variablen zu loesen.
lmo-showcross.php benutzt keine losen Ligavariablen mehr und nennt am Dateianfang, welche Variablen sie erwartet. PHPStan prueft umgestellte Dateien auf Stufe 6 (phpstan-liga.neon.dist); vorher hatte die Datei auf Stufe 1 67 Meldungen. Snapshots unveraendert.
…ber LigaDaten lmo-showtable.php, lmo-showresults.php und lmo-showprogram.php benutzen keine losen Ligavariablen mehr (292 Stellen, mit dem Tokenizer ersetzt) und nennen am Dateianfang, welche Variablen sie erwarten. PHPStan prueft sie auf Stufe 6. Alle 365 Snapshots der Seiten unveraendert.
lmo-ligadaten.php stellt das Objekt bereit und verbindet es mit den alten Variablen. init.php bindet die Datei ein, ebenso die sechs Lader, die eine Ligadatei lesen und teils innerhalb einer Funktion laufen. Damit existiert $ligaDaten unabhaengig davon, welcher Lader laeuft oder ob einer laeuft. Ganze Golden-Master-Suite unveraendert.
3712 Stellen in 62 Dateien: $titel wird $ligaDaten->titel usw., mit dem PHP-Tokenizer ersetzt (auch in Zeichenketten). Unveraendert bleiben Ligavariablen in Funktionen und Klassen (51 Stellen, z. B. Parameter gleichen Namens); sie sind weiter mit dem Objekt verbunden. Geprueft: Syntax unter PHP 8.0 und 8.3, alle PHPStan-Pruefungen, alle drei Testsuiten (1193 Tests) ohne Abweichung.
Kopfkommentar mit @var hinter der Direktaufruf-Sperre, erzeugt aus den Meldungen von PHPStan: bekannte Variablen mit Beschreibung, Einstellungen aus cfg.txt, weitere Variablen des einbindenden Codes und solche, die die Datei selbst nur unter einer Bedingung setzt. Nur Kommentare, kein Code.
phpstan-liga.neon.dist (Stufe 6) und phpstan-liga-basis.neon.dist (Stufe 1) pruefen die Dateien mit Kopfkommentar. phpstan-bootstrap.php liefert die Konstanten, die init.php erst zur Laufzeit anlegt.
To dataclass
Die rund 60 Zeilen "if (!isset($x)) { $x = ...; }" aus lmo-openfile.php
sind LigaDaten::ergaenzeVorgabewerte(), die drei Zeilen am Anfang des
Laders LigaDaten::setzeStartwerte(). Im Lader bleibt, was nicht zur Liga
gehoert ($st, $stats) oder die Sprachtexte braucht (namePkt/nameTor).
Gleiches Verhalten: Snapshots der Seiten, des Pokals und des Admin-
Bereichs unveraendert.
…tlader lmo-openfilename.php brach mit "Division by zero" ab, wenn der Ligadatei die Anzahl der Spieltage fehlte: es rechnete mit der Anzahl der Partien, die es weder liest noch vorbelegt. lmo-openfiledat.php liess aktuellen Spieltag und LMO-Version leer. Beide rufen jetzt LigaDaten::ergaenzeGrunddaten() auf (Name, Version, aktueller Spieltag, Datumsanzeige, Groesse des Spielplans); der Hauptlader nutzt dieselbe Methode ueber ergaenzeVorgabewerte(). lmo-openfiledat.php oeffnet die Datei mit demselben vollstaendigen Pfad wie fuer filemtime().
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.