Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gruppen bei Gruppenliga nicht nutzbar #1656

Open
waiter1987 opened this issue Jan 10, 2024 · 25 comments
Open

Gruppen bei Gruppenliga nicht nutzbar #1656

waiter1987 opened this issue Jan 10, 2024 · 25 comments
Labels
bug Test by User Issue wurde zum testen freigegeben

Comments

@waiter1987
Copy link

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
Copy link
Contributor

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
Copy link
Author

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

@diddipoeler
Copy link
Owner

Bitte ein Update der Komponente machen.

@diddipoeler diddipoeler added the Test by User Issue wurde zum testen freigegeben label Jan 17, 2024
@waiter1987
Copy link
Author

waiter1987 commented Jan 17, 2024

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
Copy link
Contributor

sigal commented Jan 18, 2024

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
Copy link
Author

@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
Copy link
Author

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
Copy link
Contributor

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
Copy link
Owner

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
Copy link
Author

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

@waiter1987
Copy link
Author

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.
#1661 (comment)

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...

diddipoeler added a commit that referenced this issue Jan 20, 2024
@waiter1987
Copy link
Author

@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 added a commit that referenced this issue Jan 20, 2024
diddipoeler added a commit that referenced this issue Jan 20, 2024
@diddipoeler
Copy link
Owner

Bitte ein Update der Komponente machen.

@waiter1987
Copy link
Author

Perfekt. Ich schließe damit das Issue

@waiter1987
Copy link
Author

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 waiter1987 reopened this Jan 23, 2024
@waiter1987
Copy link
Author

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
Copy link
Owner

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
Copy link
Author

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
Copy link
Contributor

sigal commented Feb 8, 2024

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
Copy link
Author

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
Copy link
Contributor

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
Copy link
Author

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
Copy link
Author

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
Copy link
Owner

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

@ortwin20000
Copy link
Contributor

Gibt es hier weitere Infos?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Test by User Issue wurde zum testen freigegeben
Projects
None yet
Development

No branches or pull requests

4 participants