diddipoeler / sportsmanagement

Die neue Sports Management Komponente für Joomla 3/4/5
GNU General Public License v2.0
27 stars 27 forks source link

Gruppen bei Gruppenliga nicht nutzbar #1656

Open waiter1987 opened 9 months ago

waiter1987 commented 9 months ago

Hallo,

ich habe bei meiner Installation neu die EM 2024 hinterlegt und festgestellt, dass die angelegten Gruppen nicht benutzt werden können.

image image image image

ortwin20000 commented 9 months ago

Ich habe bei mir auch ein neues Projekt mit so ziemlich den gleichen Daten wie du angelegt und bei mir werden die Gruppen korrekt dargestellt. grafik

Ich kann nur bei mir keine Mannschaften dem Projekt zuweisen obwohl diese der Saison zugewiesen sind. grafik Ich habe wie bei dir das Land "Europa" erstellt und es der Liga zugewiesen und diese dem Projekt zugewiesen.

waiter1987 commented 9 months ago

Was ist da genau der Unterschied jetzt? Die EM 2024 ist neu angelegt, also nicht von JL importiert gewesen.

diddipoeler commented 9 months ago

Bitte ein Update der Komponente machen.

waiter1987 commented 9 months ago

Ich sehe in den Dateien (Reiter "Code") keine Änderungen. Was soll das Update bringen? Andreas und ich haben einzelne Codes auch geändert. Das würde es ja dann überschreiben... Würde gern zum Test nur die Dateien manuell austauschen wollen

sigal commented 9 months ago

Original Komponente.

Gruppenspiele

Gruppe A - F

Ich verstehe nicht, warum man die Gemeinschaft an den Änderungen nicht teilhaben läßt? (Pull request) Nicht jeder User kann einzelne Dateien so austauschen.

waiter1987 commented 9 months ago

@diddipoeler habe das Update gemacht. Jetzt habe ich aber den Fehler hier image

Taucht beim Aufruf eines Spieltags (Ergebnis-Tabellen) auf. Liegt höchstwahrscheinlich an der aktuellen Joomla-Version.

Das Update hat den Fehler jedenfalls behoben!

waiter1987 commented 9 months ago

Original Komponente.

Gruppenspiele

Gruppe A - F

Ich verstehe nicht, warum man die Gemeinschaft an den Änderungen nicht teilhaben läßt? (Pull request) Nicht jeder User kann einzelne Dateien so austauschen.

Ich bin glaube der falsche, der einen Pull Request einstellen sollte... Wir (Andreas und ich) haben versucht hier alles zu dokumentieren, was geändert wurde.

ortwin20000 commented 9 months ago

image Es dürfte an diesen Commits liegen. Wenn ich es richtig sehe will Diddi auf das Bootstrap Pagination umstellen und hat dafür den Code angepasst. Leider gibt es aber noch einige Stellen bei den Ansichten wo dies nicht richtig greift und schmeißt dann eine Fehlermeldung.

Wenn ich den Menüeintrag von Ergebnis-Tabelle-Ansicht nur auf Ergebnis-Ansicht ändere sollte es eigentlich klappen. Nur hab ich dann bei mir das Problem, dass mein Template Astroid bereits die Funktion für Pagination eingebaut hat und dann schmeißt es wieder einen Fehler.

image

diddipoeler commented 9 months ago

Original Komponente. Gruppenspiele Gruppe A - F Ich verstehe nicht, warum man die Gemeinschaft an den Änderungen nicht teilhaben läßt? (Pull request) Nicht jeder User kann einzelne Dateien so austauschen.

Ich bin glaube der falsche, der einen Pull Request einstellen sollte... Wir (Andreas und ich) haben versucht hier alles zu dokumentieren, was geändert wurde.

Warum der Falsche? Wer Quellcode ändern kann, kann auch einen Pull Request erstellen. Bitte auch nicht falsch verstehen. Aber dadurch erhalten wir ein einheitliche JSM Version, die für alle nutzbar ist.

Damit ich es vielleicht leichter habe, wären die geändert Zeilen sehr hilfreich. Alter Code / neuer Code.

waiter1987 commented 9 months ago

Habe das Update gemacht. Jetzt sind die Divisionen da. ich habe allerdings bei den Mannschaften nach wie vor die elende Liste hier: image

waiter1987 commented 9 months ago

Original Komponente. Gruppenspiele Gruppe A - F Ich verstehe nicht, warum man die Gemeinschaft an den Änderungen nicht teilhaben läßt? (Pull request) Nicht jeder User kann einzelne Dateien so austauschen.

Ich bin glaube der falsche, der einen Pull Request einstellen sollte... Wir (Andreas und ich) haben versucht hier alles zu dokumentieren, was geändert wurde.

Warum der Falsche? Wer Quellcode ändern kann, kann auch einen Pull Request erstellen. Bitte auch nicht falsch verstehen. Aber dadurch erhalten wir ein einheitliche JSM Version, die für alle nutzbar ist.

Damit ich es vielleicht leichter habe, wären die geändert Zeilen sehr hilfreich. Alter Code / neuer Code.

Dazu habe ich hier was angehangen. https://github.com/diddipoeler/sportsmanagement/issues/1661#issuecomment-1901095667

Zum Teil sind die Codes hier in den Issues schon drin. Einige Sachen sind mit "probiere mal bitte hier das mit dem zu tauschen" einhergegangen. Ich selber habe den Quellcode nicht geändert. Das war Andreas - dankenswerterweise...

Ich kann auch die geänderten Dateien irgendwo anhängen oder bei den Themen einzelne Issues machen... wie das am Einfachsten ist...

waiter1987 commented 9 months ago

@diddipoeler habe das Update gemacht. Jetzt habe ich aber den Fehler hier image

Taucht beim Aufruf eines Spieltags (Ergebnis-Tabellen) auf. Liegt höchstwahrscheinlich an der aktuellen Joomla-Version.

Das Update hat den Fehler jedenfalls behoben!

Das ist auch noch offen :-(

Komme hier nicht drauf image

https://sport.daniel-frase.de/index.php/1-bundesliga-2023-2024-spieltag/resultsranking/0/399:2023-2024/4617:1-bundesliga-2023-2024/0/0

diddipoeler commented 9 months ago

Bitte ein Update der Komponente machen.

waiter1987 commented 9 months ago

Perfekt. Ich schließe damit das Issue

waiter1987 commented 9 months ago

Ich habe das Update gemacht. Es werden im Backend die Gruppen angezeigt, aber nicht im Frontend!

https://sport.daniel-frase.de/index.php/uefa-champions-league-2023-2024/results/0/0/4632:champions-league-2023-2024/0/0 Hier sollen die betreffenden Gruppen angezeigt werden!

Wenn ich das bspw. auf Gr. A nur anzeigen lassen, sieht es so aus, als wären da gar keine Spiele drin https://sport.daniel-frase.de/index.php/uefa-champions-league-2023-2024/uefa-champions-league-2023-2024-gruppe-a/resultsranking/0/399:2023-2024/4632:champions-league-2023-2024/3113:gruppe-a/0

Ich vermute hier eine Einstellungssache: image

Allerdings habe ich den Afrika-Cup neu angelegt. Das lädt er im Frontend gar nicht. :-( https://sport.daniel-frase.de/index.php/nationalmannschaften/kontinentalmeisterschaften/afrika-caf/afrika-cup-2023/results/0/399:2023-2024/4661:afrika-cup-2024/0/0 image

waiter1987 commented 8 months ago

Bgzl. des Aufrufens im Frontend hab ich in der Datenbank eine Änderung vorgenommen. Da wo die Maus ist habe ich das "NULL" rausgenommen. Jetzt ist das im Frontend aufrufbar. Gruppen gehen dabei aber nach wie vor nicht.

image

diddipoeler commented 8 months ago

Also bei mir sieht das so aus: https://www.fussballineuropa.de/index.php/component/sportsmanagement/ranking/0/0/150819:caf-afrika-cup-kamerun-2024/0/0/0?Itemid=-1

In welcher Tabelle hast denn die NULL rausgenommen ?

waiter1987 commented 8 months ago

In der Projects in der Spalte Extensions image

Ich weiß halt nicht, ob das mit den Gruppen ein Thema ist, was ich mit J5 habe... 🤷‍♀️

sigal commented 8 months ago

Schon die Einstellungen in den Templates [ranking] - [results] Reiter "Gruppeneinstellungen" überprüft?

2024-02-08-Sports Management_ Template   Tabellen Anzeige   bearbeiten

2024-02-08 13_Sports Management_ Template   Ergebnis Anzeige   bearbeiten

waiter1987 commented 8 months ago

Hab ich genauso... habe aber eben noch was anderes bemerkt...

Bei den Mannschaften ist die Gruppe zugeordnet image

Im Spieltag ist das nicht mehr der Fall! image

Selbst wenn ich am Beispiel Ägypten jetzt auf Gruppe A händisch ändere, passiert nichts!

Das ist der Screenshot vom Menüeintrag image

Das der Frontend Gruppe A

Und bei Ergebnisse haben alle Spielfrei image

Ich bin gerade maximal verwirrt...

ortwin20000 commented 8 months ago

Ich habe mal mit deinen Daten die beiden Templates ranking und results geöffnet und abgespeichert. Bei beiden Templates auch sichergestellt, dass die Gruppeneinstellung auf Gruppenstufe 1 gesetzt ist. Wenn ich dann den gleichen Menüeintrag wie bei dir erstelle erhalte ich auch keinen Einträge aber wenn ich in den Spieltagen die Gruppe der einzelnen Spiele setze werden sie mir angezeigt.

image

Warum auch immer werden mir aber darunter die Überschriften der anderen Gruppen auch angezeigt.

Das passiert mir auch bei der resultsranking-Ansicht: image

waiter1987 commented 8 months ago

So... Dank @ortwin20000 sind wir ein klein wenig weiter.

Es zählt nur, wenn im Spieltag die Gruppen bestätigt werden. Ob das so gewollt ist oder nicht, kann ich nicht sagen. Ich habe es getestet. Im Spieltag sieht es so aus, als wären keine Gruppen zugeordnet. Wenn ich das einschränke (Gruppeneinschränkung im Spieltag) und die Zuordnung mache (beim Spiel), dann zählt es auch in der Tabelle!

Allerdings gibt es ein Thema beim Menüpunkt. Ich wähle Gr. A aus. Es werden alle Gruppen angezeigt. Und es zählt erst, wenn ich den Spieltag oben wechsle.

Das liegt am SEF. Wenn ich das ausschalte, geht das!

waiter1987 commented 8 months ago

Spieltag sieht so aus. Sieht so aus, als wäre kein Spieltag bei den Spielen drin image

Wenn ich oben (wo die Maus ist) den Spieltag eingrenze, zeigt er die Spiele an, mit den Mannschaften, die die Gruppenzuordnung hat image

Ohne die Zuordnung der Gruppe sieht die Tabelle so aus (hier im Modul): image

Wenn die Gruppen zugeordnet und gespeichert sind... image

Sieht es so aus: image

diddipoeler commented 8 months ago

Bitte bei den Gruppen keine Programmtechnischen Änderungen vornehmen. Das musste ich so machen, da es historisch gewachsen ist. Erläuterung kommt morgen.

ortwin20000 commented 8 months ago

Gibt es hier weitere Infos?