Repository navigation
Liga leser - #30
Closed
tombrain wants to merge 166 commits into
Closed
Liga leser#30tombrain wants to merge 166 commits into
tombrain wants to merge 166 commits into
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).
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().
Ligadaten vorgabewerte
Abschnitte, Schluessel und Werte einer .l98-Datei, zeilenweise wie bisher im Lader: Ein Abschnitt beginnt nur mit einer Zeile, die als Ganzes in eckigen Klammern steht. lmo-openfile.php holt die Eintraege von dort; die Aufbereitung der Werte ist unveraendert.
liga::loadFile() zerlegte die Datei an jeder eckigen Klammer. Stand eine in einem Liganamen, Mannschaftsnamen oder einer Notiz, ging der Rest des Abschnitts verloren. Kommentarzeilen (;) wurden als Schluessel gelesen. Die classlib benutzt jetzt denselben Leser wie der Kern (LigaDatei).
LigaDaten::OPTIONEN nennt je Schluessel das Feld, ob er fuer Liga oder Pokal gilt, und den Vorgabewert. setzeOption() ersetzt den switch mit 57 Faellen in lmo-openfile.php; ergaenzeGrunddaten() und ergaenzeVorgabewerte() nehmen die einfachen Vorgabewerte aus derselben Tabelle. Im Lader bleiben $st und $stats, die nicht zur Liga gehoeren. Zuordnung gegen den alten switch abgeglichen (57 Paare, 34 nur Liga, 3 nur Pokal); Snapshots unveraendert.
LigaDaten::optionenZumSpeichern() liefert die Zeilen des Abschnitts in der Reihenfolge der Tabelle OPTIONEN, mit den Bedingungen des bisherigen Codes (Liga/Pokal, Xtra*/Spez* nur mit Spez=1, namePkt/nameTor nur wenn gesetzt, Title = Name des Programms, Actual = gewaehlter Spieltag). lmo-savefile.php ersetzt damit 77 Zeilen fputs(). Alter und neuer Code Zeichen fuer Zeichen verglichen: 15 Ligadateien in je 10 Varianten, keine Abweichung.
bereiteMannschaftenVor() legt die Felder in voller Groesse an, setzeMannschaftswert() uebernimmt eine Zeile aus [Teams], [Teamm], [Teamk] oder [TeamN], ergaenzeMannschaftsnamen() leitet fehlende Kurz- und Mittelnamen ab, mannschaftenZumSpeichern() liefert die Abschnitte fuer lmo-savefile.php. Schluessel und Felder stehen in zwei Tabellen. Alter und neuer Lader Feld fuer Feld verglichen (17 Dateien, mit und ohne "nur ein Spieltag"), Speichern zeichengleich; Snapshots unveraendert.
bereiteSpieltageVor() legt die Felder in voller Groesse an (Liga 116x40, Pokal 7x64x7), setzeSpieltagswert() uebernimmt eine Zeile aus [RoundN] samt Umrechnung der Tore und Zusaetze fuer die Anzeige, dazu News und das Bereinigen des Spieltagsdatums. lmo-openfile.php enthaelt nur noch den Ablauf (107 statt 257 Zeilen). Ein unlesbares Datum im letzten Spieltag bricht das Laden nicht mehr ab. Alter und neuer Lader Feld fuer Feld verglichen: 34 Dateien in je vier Varianten, keine Abweichung; alle drei Testsuiten unveraendert.
newsZumSpeichern() und spieltageZumSpeichern() ergaenzen Options und Mannschaften; abschnitteZumSpeichern() liefert die Datei in der Schreibreihenfolge. lmo-savefile.php schreibt nur noch Abschnitte und Zeilen (82 statt 200 Zeilen). Erhalten bleibt, dass das Speichern die Liga veraendert: im Pokal steht anzsp danach auf den Paarungen der letzten Runde, fehlende Wertungen (msieg) sind mit 0 belegt. Alter und neuer Code verglichen: 37 Dateien (5 Pokale) in 18 Varianten, 666 Faelle, Datei zeichengleich und Zustand der Liga danach gleich.
…auf fuer alle Lader Der Lader reichte die Zeilen als Text mit | als Trenner weiter; ein | in einer Notiz, einem Namen oder einer Adresse schnitt den Rest ab. Jetzt gehen die Eintraege aus LigaDatei direkt an LigaDaten::lies(), das den ganzen Ablauf enthaelt (Options mit Vorgabewerten, News, Mannschaften, Spieltage). lmo-openfilename.php und lmo-openfiledat.php benutzen denselben Leser und setzeOptionen() mit ihren wenigen Schluesseln. Verglichen mit dem alten Stand: Hauptlader weicht nur in den zwei vorher abgeschnittenen Feldern ab, die abgespeckten Lader gar nicht (174 Faelle). Alle drei Testsuiten unveraendert.
setzeStartwerte() leert alle Felder ausser den dreien, die auch aus dem Seitenaufruf kommen. Bisher blieben Werte der zuvor geladenen Liga stehen, wenn die naechste Datei sie nicht nennt. Nur noch Zeilen aus [RoundN] belegen ein fehlendes Handicap mit 0; bisher tat das jede Zeile der Datei und legte so den Eintrag -1 an. Beide Staende der Klasse verglichen: einziger Unterschied ist der entfernte Eintrag -1. Alle drei Testsuiten unveraendert.
Lmo\Liga\Rohdaten\Liga haelt Einstellungen, News, Mannschaften, Spieltage und Partien so, wie sie in der Datei stehen: Werte unveraendert, nichts ergaenzt, abgeleitet oder aufgefuellt, unbekannte Schluessel und Abschnitte bleiben erhalten. lies()/schreibe() und ausEintraegen()/abschnitte() sind Gegenstuecke; ein mit new angelegtes Objekt ist eine gueltige leere Liga. LigaDatei bekommt dafuer alsText() und schreibe(). Jede vom LMO gespeicherte Ligadatei (Repo, Demo, erzeugte Testligen) kommt nach Laden und Speichern zeichengleich wieder heraus. Noch nicht eingebunden: der LMO liest und schreibt weiter ueber LigaDaten.
Laden: die Datei wird immer zuerst in Rohdaten\Liga gelesen, daraus erzeugt LigaDaten die aufbereiteten Felder fuer den alten Code (liesRohdaten()). Speichern: alsRohdaten() rechnet die Felder zurueck, die Rohdaten werden geschrieben. Dadurch bleiben beim Speichern erhalten, was der LMO nicht kennt: Einstellungen in [Options] (auch stats) und ganze Abschnitte. Unbekannte Werte einer Mannschaft oder Partie gehen weiter verloren, weil der alte Code Mannschaften und Partien in seinen Feldern verschiebt. Verglichen mit dem vorigen Stand: Laden weicht nur bei einer Datei ab, die nicht mit einem Spieltag endet; Speichern nur um den erhaltenen unbekannten Abschnitt. Alle drei Testsuiten unveraendert.
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.